Clean up comments about null state wrappers

Summary:
Updating state with a null wrapper is neither desirable, nor possible. The underlying task was closed, just cleaning up comments.

Changelog: [Internal] comments only

Reviewed By: mdvacca

Differential Revision: D21186545

fbshipit-source-id: d14ddd59d42e8fd91c6e7fd50037311d4e8d0b60
This commit is contained in:
Joshua Gross
2020-04-23 15:39:45 -07:00
committed by Facebook GitHub Bot
parent 78b1c85747
commit 17adf549e2
2 changed files with 0 additions and 2 deletions
@@ -140,7 +140,6 @@ public class ReactModalHostManager extends ViewGroupManager<ReactModalHostView>
@Override
public Object updateState(
ReactModalHostView view, ReactStylesDiffMap props, @Nullable StateWrapper stateWrapper) {
// TODO T55794595: Add support for updating state with null stateWrapper
Point modalSize = ModalHostHelper.getModalHostSize(view.getContext());
view.updateState(stateWrapper, modalSize.x, modalSize.y);
return null;
@@ -83,7 +83,6 @@ public class ReactTextViewManager
@Override
public Object updateState(
ReactTextView view, ReactStylesDiffMap props, @Nullable StateWrapper stateWrapper) {
// TODO T55794595: Add support for updating state with null stateWrapper
ReadableNativeMap state = stateWrapper.getState();
ReadableMap attributedString = state.getMap("attributedString");
ReadableMap paragraphAttributes = state.getMap("paragraphAttributes");