Test invisible-static elements upgrading when created with HTML parser

Invisible-static custom elements are still upgraded when they are
constructed through HTML parser. To make it clear, we are adding test
to confirm this behavior. This CL also make the previous test's
custom element constructor to not append a child to verify upgrades.

Previous change that defer upgrades of invisible="static" elements:
https://chromium-review.googlesource.com/c/chromium/src/+/1204373

Bug: 873057
Change-Id: If25afa4c1c77ecd70988d3d29cc5ab020247b8ea
Reviewed-on: https://chromium-review.googlesource.com/1235265
Commit-Queue: Rakina Zata Amni <rakina@chromium.org>
Reviewed-by: Kent Tamura <tkent@chromium.org>
Cr-Commit-Position: refs/heads/master@{#592710}

third_party/WebKit/LayoutTests/invisible_dom/invisible-static.html

@@ -13,10 +13,17 @@
 <script>
 'use strict';
 
+let upgraded = false;
+let connected = false;
+
 class TestElement extends HTMLElement {
   constructor() {
     super();
-    this.innerHTML = "upgraded";
+    upgraded = true;
+  }
+
+  connectedCallback() {
+    connected = true;
   }
 }
 
@@ -29,51 +36,98 @@ function setUp() {
   normal.innerHTML = basicInvisible.innerHTML = staticInvisible.innerHTML = "";
   basicInvisible.invisible = "invisible";
   staticInvisible.invisible = "static";
+  connected = upgraded = false;
 }
 
 test(() => {
   setUp();
   normal.innerHTML = testElementString;
+  assert_true(upgraded);
+  assert_true(connected);
+
+  setUp();
   basicInvisible.innerHTML = testElementString;
+  assert_true(upgraded);
+  assert_true(connected);
+
+  setUp();
   staticInvisible.innerHTML = testElementString;
-  assert_equals(normal.innerHTML, testElementUpgradedString);
-  assert_equals(basicInvisible.innerHTML, testElementUpgradedString);
-  assert_equals(staticInvisible.innerHTML, testElementString);
+  assert_false(upgraded);
+  assert_false(connected);
 }, "Custom elements inside invisible-static subtree is not upgraded.");
 
 test(() => {
   setUp();
   staticInvisible.removeAttribute("invisible");
   staticInvisible.innerHTML = testElementString;
-  assert_equals(staticInvisible.innerHTML, testElementUpgradedString);
+  assert_true(upgraded);
+  assert_true(connected);
 }, "Previously-static subtree should not block custom element upgrade");
 
 test(() => {
   setUp();
   staticInvisible.innerHTML = testElementString;
   staticInvisible.invisible = "invisible";
-  assert_equals(staticInvisible.innerHTML, testElementUpgradedString);
+  assert_true(upgraded);
+  assert_true(connected);
 }, "Making an element not invisible='static' upgrades the custom elements inside");
 
 test(() => {
   setUp();
   staticInvisible.innerHTML = testElementString;
-  assert_equals(staticInvisible.innerHTML, testElementString, "Normally not upgraded");
+  assert_false(upgraded);
+  assert_false(connected);
   customElements.upgrade(staticInvisible);
-  assert_equals(staticInvisible.innerHTML, testElementUpgradedString, "After forcing got upgraded");
+  assert_true(upgraded);
+  assert_true(connected);
 }, "Upgrade by customElements.upgrade is not deferred");
 
 test(() => {
   setUp();
   staticInvisible.innerHTML = "<another-element></another-element>";
+  let anotherUpgraded = false;
+  let anotherConnected = false;
   customElements.define("another-element", class extends HTMLElement {
     constructor() {
       super();
-      this.innerHTML = "upgraded";
+      anotherUpgraded = true;
+    }
+
+    connectedCallback() {
+      anotherConnected = true;
     }
   });
-  assert_equals(staticInvisible.innerHTML, "<another-element></another-element>");
+  assert_false(anotherUpgraded);
+  assert_false(anotherConnected);
   staticInvisible.invisible = "invisible";
-  assert_equals(staticInvisible.innerHTML, "<another-element>upgraded</another-element>");
+  assert_true(anotherUpgraded);
+  assert_true(anotherConnected);
 }, "Upgrade after defined is deferred");
+
+setUp();
 </script>
+
+<test-element id="parserEl" invisible="static"></test-element>
+
+<script>
+test(() => {
+  assert_true(upgraded);
+  assert_true(connected);
+}, "Invisible static custom element is upgraded if constructed through HTML parser.");
+
+setUp();
+</script>
+
+<div>
+  <test-element></test-element>
+</div>
+
+<script>
+test(() => {
+  assert_true(upgraded);
+  assert_true(connected);
+}, "Custom element under invisible static element made through HTML parser is upgraded.");
+</script>
+
+
+</body>