Implement "create an element" when sync for Custom Element V1

third_party/WebKit/Source/bindings/core/v8/custom/V8HTMLElementCustom.cpp

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "bindings/core/v8/V8HTMLElement.h"
 
 #include "bindings/core/v8/DOMWrapperWorld.h"
 #include "bindings/core/v8/ExceptionState.h"
 #include "bindings/core/v8/ScriptCustomElementDefinition.h"
 #include "bindings/core/v8/V8Binding.h"
@@ skipping 37 matching lines @@
 
     ExceptionState exceptionState(
         ExceptionState::ConstructionContext,
         "HTMLElement",
         info.Holder(),
         isolate);
 
     Element* element;
     if (definition->constructionStack().isEmpty()) {
         // This is an element being created with 'new' from script
-        element = window->document()->createElement(
-            definition->descriptor().localName(),
-            AtomicString(),
-            exceptionState);
-        if (exceptionState.throwIfNeeded())
-            return;
+        // TODO(kojii): We can't call HTMLElementFactory or

[dominicc (has gone to gerrit), 2016/06/09 06:39:23]

This is good!

+        // Document::createElement because they will invoke custom element
+        // constructors. HTMLElement is enough for now, but type extesion
+        // will require to hook HTMLElementFactory.
+        element = HTMLElement::create(
+            QualifiedName(nullAtom, definition->descriptor().localName(), HTMLNames::xhtmlNamespaceURI),
+            *window->document());
+        element->setCustomElementState(CustomElementState::Undefined);
     } else {
         element = definition->constructionStack().last();
         if (element) {
             // This is an element being upgraded that has called super
             definition->constructionStack().last().clear();
         } else {
             // During upgrade an element has invoked the same constructor
             // before calling 'super' and that invocation has poached the
             // element.
             exceptionState.throwDOMException(
@@ skipping 10 matching lines @@
         wrapperType,
         info.Holder());
     // If the element had a wrapper, we now update and return that
     // instead.
     v8SetReturnValue(info, wrapper);
 
     v8CallOrCrash(wrapper->SetPrototype(
         scriptState->context(),
         definition->prototype()));
 
-    // TODO(dominicc): These elements should be 'undefined', not
-    // 'uncustomized', on creation. Investigate why some elements are
-    // running around uncustomized.
-    if (element->getCustomElementState() == CustomElementState::Uncustomized)
-        element->setCustomElementState(CustomElementState::Undefined);
     // TODO(dominicc): Move this to the exactly correct place when
     // https://github.com/whatwg/html/issues/1297 is closed.
     element->setCustomElementState(CustomElementState::Custom);
 }
 
 } // namespace blink