Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(545)

Side by Side Diff: third_party/WebKit/Source/core/dom/custom/CustomElementDefinition.h

Issue 2054433002: Implement "create an element" when sync for Custom Element V1 (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@async-ce
Patch Set: Prefer ExceptionState over throw Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CustomElementDefinition_h 5 #ifndef CustomElementDefinition_h
6 #define CustomElementDefinition_h 6 #define CustomElementDefinition_h
7 7
8 #include "bindings/core/v8/ScriptValue.h" 8 #include "bindings/core/v8/ScriptValue.h"
9 #include "core/CoreExport.h" 9 #include "core/CoreExport.h"
10 #include "core/dom/custom/CustomElementDescriptor.h" 10 #include "core/dom/custom/CustomElementDescriptor.h"
11 #include "platform/heap/Handle.h" 11 #include "platform/heap/Handle.h"
12 #include "wtf/Noncopyable.h" 12 #include "wtf/Noncopyable.h"
13 13
14 namespace blink { 14 namespace blink {
15 15
16 class ScriptState;
dominicc (has gone to gerrit) 2016/06/10 06:13:28 Thanks for cleaning this up.
17 class Element; 16 class Element;
17 class ExceptionState;
18 class HTMLElement;
19 class QualifiedName;
18 20
19 class CORE_EXPORT CustomElementDefinition 21 class CORE_EXPORT CustomElementDefinition
20 : public GarbageCollectedFinalized<CustomElementDefinition> { 22 : public GarbageCollectedFinalized<CustomElementDefinition> {
21 WTF_MAKE_NONCOPYABLE(CustomElementDefinition); 23 WTF_MAKE_NONCOPYABLE(CustomElementDefinition);
22 public: 24 public:
23 CustomElementDefinition(const CustomElementDescriptor&); 25 CustomElementDefinition(const CustomElementDescriptor&);
24 virtual ~CustomElementDefinition(); 26 virtual ~CustomElementDefinition();
25 27
26 DECLARE_VIRTUAL_TRACE(); 28 DECLARE_VIRTUAL_TRACE();
27 29
28 const CustomElementDescriptor& descriptor() { return m_descriptor; } 30 const CustomElementDescriptor& descriptor() { return m_descriptor; }
29 31
30 // TODO(yosin): To support Web Modules, introduce an abstract 32 // TODO(yosin): To support Web Modules, introduce an abstract
31 // class |CustomElementConstructor| to allow us to have JavaScript 33 // class |CustomElementConstructor| to allow us to have JavaScript
32 // and C++ constructors and ask the binding layer to convert 34 // and C++ constructors and ask the binding layer to convert
33 // |CustomElementConstructor| to |ScriptValue|. Replace 35 // |CustomElementConstructor| to |ScriptValue|. Replace
34 // |getConstructorForScript()| by |getConstructor() -> 36 // |getConstructorForScript()| by |getConstructor() ->
35 // CustomElementConstructor|. 37 // CustomElementConstructor|.
36 virtual ScriptValue getConstructorForScript() = 0; 38 virtual ScriptValue getConstructorForScript() = 0;
37 39
38 using ConstructionStack = HeapVector<Member<Element>, 1>; 40 using ConstructionStack = HeapVector<Member<Element>, 1>;
39 ConstructionStack& constructionStack() 41 ConstructionStack& constructionStack()
40 { 42 {
41 return m_constructionStack; 43 return m_constructionStack;
42 } 44 }
43 45
46 virtual HTMLElement* createElementByRunningConstructor(Document&, const Qual ifiedName&) = 0;
dominicc (has gone to gerrit) 2016/06/10 06:13:28 Maybe call this createElementSync to match the nam
kojii 2016/06/10 19:01:45 Done.
47 virtual HTMLElement* createElementByRunningConstructor(Document&, const Qual ifiedName&, ExceptionState&) = 0;
48
44 void upgrade(Element*); 49 void upgrade(Element*);
45 50
46 protected: 51 protected:
47 virtual bool runConstructor(Element*) = 0; 52 virtual bool runConstructor(Element*) = 0;
48 53
54 static void errorForConstructorResult(Element*, Document&, const QualifiedNa me&, ExceptionState&);
55
49 private: 56 private:
50 const CustomElementDescriptor m_descriptor; 57 const CustomElementDescriptor m_descriptor;
51 ConstructionStack m_constructionStack; 58 ConstructionStack m_constructionStack;
52 }; 59 };
53 60
54 } // namespace blink 61 } // namespace blink
55 62
56 #endif // CustomElementDefinition_h 63 #endif // CustomElementDefinition_h
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698