Remove the wrong optimization in replacing a text node with innerHTML

third_party/WebKit/LayoutTests/shadow-dom/slotchange.html

Index: third_party/WebKit/LayoutTests/shadow-dom/slotchange.html
diff --git a/third_party/WebKit/LayoutTests/shadow-dom/slotchange.html b/third_party/WebKit/LayoutTests/shadow-dom/slotchange.html
index 4ad226efe299da91bd645515265719f2bf85d667..e5387e4f688e5550446b4f039168db01aff49e4d 100644
--- a/third_party/WebKit/LayoutTests/shadow-dom/slotchange.html
+++ b/third_party/WebKit/LayoutTests/shadow-dom/slotchange.html
@@ -309,3 +309,24 @@ async_test((test) => {
   n.c1.remove();
 }, "slotchange event: Fallback slot's parant slot is assinged to another slot.");
 </script>
+
+<div id="test8">
+  <div id="host1">

[kochi, 2016/11/30 04:20:37]

Not critial, but all these subtrees have duplicate
#host1's in document tree.

+    <template id="shadowroot" data-mode="open">
+      <slot id="s1"></slot>
+    </template>
+  </div>
+</div>

[kochi, 2016/11/30 04:20:37]

This #test8 subtree isn't used?

[hayato, 2016/11/30 05:22:55]

Done.

+
+<script>
+async_test((test) => {
+  const n = createTestTree(window.test1);

[kochi, 2016/11/30 04:20:37]

nit: other tests use just 'test1', and no local 'test1' in this function, so
'window.' is unnecessary? It's fine if this is to avoid confusion with the next
line ('n.test1').

Or should this be 'test8'?

[hayato, 2016/11/30 05:22:55]

Recently, I am using `window.` prefix explicitly for new code to suppress
eslint's warnings.
Yeah, it is inconsistent within this file, but that might be okay.

+  removeWhiteSpaceOnlyTextNodes(n.test1);
+  n.host1.innerHTML = 'hello';
+
+  window.setTimeout(() => {
+    doneIfSlotChange([n.s1], test);
+    n.host1.innerHTML = 'world';
+  }, 0);
+}, 'slotchange event: Replacing a text node with innerHTML.');
+</script>