Custom Elements: add prototype check for "failed" elements

third_party/WebKit/LayoutTests/custom-elements/spec/state-failed-upgrade.html

Index: third_party/WebKit/LayoutTests/custom-elements/spec/state-failed-upgrade.html
diff --git a/third_party/WebKit/LayoutTests/custom-elements/spec/state-failed-upgrade.html b/third_party/WebKit/LayoutTests/custom-elements/spec/state-failed-upgrade.html
index 995ee272ea2a87d5ddfcf3a51a08282c01718bd2..363c8bcd866abc9c7415b216fb7590b313cad584 100644
--- a/third_party/WebKit/LayoutTests/custom-elements/spec/state-failed-upgrade.html
+++ b/third_party/WebKit/LayoutTests/custom-elements/spec/state-failed-upgrade.html
@@ -36,6 +36,13 @@ test_with_window(w => {
   // "failed" is not "defined"
   // https://dom.spec.whatwg.org/#concept-element-defined
   assert_false(element.matches(':defined'));
+  // "failed" element should implement HTMLUnknownElement only in "creating an element for a token"
+  // https://html.spec.whatwg.org/#create-an-element-for-the-token
+  assert_equals(Object.getPrototypeOf(element), w.HTMLElement.prototype);

[kojii, 2016/09/06 08:10:55]

Replace this one to "instanceof" too?

[dominicc (has gone to gerrit), 2016/09/06 16:54:07]

I think the assert_equals checks are better because they're more precise; it
means we're returning something whose prototype is exactly HTMLUnknownElement?

instanceof HTMLElement in particular won't work because HTMLUnknownElement and
HTMLElement are both instanceof HTMLElement.

+  // 2. If element's custom element state is "failed", then abort these steps.
+  // https://html.spec.whatwg.org/multipage/scripting.html#upgrades
+  container.appendChild(element);
+  assert_equals(constructorCount, 1, 'constructor should be invoked once');
 });
 </script>
 </body>