Fix ScriptCustomElementDefinition::createElementSync to use the given document

third_party/WebKit/Source/core/dom/custom/CustomElementDefinition.h

Index: third_party/WebKit/Source/core/dom/custom/CustomElementDefinition.h
diff --git a/third_party/WebKit/Source/core/dom/custom/CustomElementDefinition.h b/third_party/WebKit/Source/core/dom/custom/CustomElementDefinition.h
index 0bf974f57d4b278841c4f18eb7313547dce5ed41..a33c68e7e1a523901fe048b54b53cc9f24e4263f 100644
--- a/third_party/WebKit/Source/core/dom/custom/CustomElementDefinition.h
+++ b/third_party/WebKit/Source/core/dom/custom/CustomElementDefinition.h
@@ -48,6 +48,7 @@ public:
         return m_constructionStack;
     }
 
+    HTMLElement* createElementForConstructor(Document&);
     virtual HTMLElement* createElementSync(Document&, const QualifiedName&) = 0;
     virtual HTMLElement* createElementSync(Document&, const QualifiedName&, ExceptionState&) = 0;
     HTMLElement* createElementAsync(Document&, const QualifiedName&);
@@ -73,6 +74,18 @@ public:
     void enqueueAttributeChangedCallback(Element*, const QualifiedName&,
         const AtomicString& oldValue, const AtomicString& newValue);
 
+    class CORE_EXPORT ConstructionStackScope final {

[kochi, 2016/08/25 08:18:11]

ScopedConstructionStack?
(I don't have strong opinion here for naming, just my preference)

[kojii, 2016/08/25 09:01:17]

We have CEReactionsScope, if not strong, allow me to take the consistency with
it.

[kochi, 2016/08/25 09:12:19]

Acknowledged.

+        STACK_ALLOCATED();
+        WTF_MAKE_NONCOPYABLE(ConstructionStackScope);

[tkent, 2016/08/25 09:03:35]

nit: Let's use DISALLOW_COPY_AND_ASSIGN(ConstructionStackScope) for new class.

+    public:
+        ConstructionStackScope(CustomElementDefinition*, Element*);
+        ~ConstructionStackScope();
+
+    private:
+        ConstructionStack& m_constructionStack;
+        Member<Element> m_element;
+        size_t m_depth;

[kochi, 2016/08/25 08:18:11]

Looks like m_element and m_depth are only used for DCHECK?

If so, enclose them '#ifdef DCHECK_IS_ON() ~ #endif'.

[kojii, 2016/08/25 09:01:17]

Done.

+    };
 protected:
     virtual bool runConstructor(Element*) = 0;