Clear body instead of document; use textContent instead of innerHTML

facebook · Apr 28, 2020 · 0cc80fa · 0cc80fa
1 parent 39b2042
commit 0cc80fa
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 10 deletions.
diff --git a/packages/react-art/src/ReactARTHostConfig.js b/packages/react-art/src/ReactARTHostConfig.js
@@ -427,12 +427,8 @@ export function unhideTextInstance(textInstance, text): void {
   // Noop
 }
 
-export function clearContainer(container: Container): void {
-  // TODO This doesn't work for anything other than SVG.
-  // Is that okay?
-  while (container.lastChild != null) {
-    container.lastChild.eject();
-  }
+export function clearContainer(container) {
+  // TODO Implement this
 }
 
 export function DEPRECATED_mountResponderInstance(

diff --git a/packages/react-dom/src/client/ReactDOMHostConfig.js b/packages/react-dom/src/client/ReactDOMHostConfig.js
@@ -636,11 +636,11 @@ export function unhideTextInstance(
 
 export function clearContainer(container: Container): void {
   if (container.nodeType === ELEMENT_NODE) {
-    ((container: any): Element).innerHTML = '';
+    ((container: any): Element).textContent = '';
   } else if (container.nodeType === DOCUMENT_NODE) {
-    const documentElement = ((container: any): Document).documentElement;
-    if (documentElement != null) {
-      documentElement.innerHTML = '';
+    const body = ((container: any): Document).body;
+    if (body != null) {
+      body.textContent = '';
     }
   }
 }