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

Unified Diff: third_party/WebKit/Source/bindings/core/v8/ScriptCustomElementDefinitionBuilder.cpp

Issue 2024073002: Add callbacks to ScriptCustomElementDefinition (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix when no callbacks, cleanup, and reset-result Created 4 years, 7 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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/bindings/core/v8/ScriptCustomElementDefinitionBuilder.cpp
diff --git a/third_party/WebKit/Source/bindings/core/v8/ScriptCustomElementDefinitionBuilder.cpp b/third_party/WebKit/Source/bindings/core/v8/ScriptCustomElementDefinitionBuilder.cpp
index 39da9922a20331c868b3889186b2f20796ec9a0a..023099adac36ad7963f18279216d23057f571020 100644
--- a/third_party/WebKit/Source/bindings/core/v8/ScriptCustomElementDefinitionBuilder.cpp
+++ b/third_party/WebKit/Source/bindings/core/v8/ScriptCustomElementDefinitionBuilder.cpp
@@ -60,17 +60,21 @@ bool ScriptCustomElementDefinitionBuilder::checkConstructorNotRegistered()
return true;
}
-bool ScriptCustomElementDefinitionBuilder::checkPrototype()
+bool ScriptCustomElementDefinitionBuilder::getValue(
+ const v8::Local<v8::Object>& object, const char* name,
+ v8::Local<v8::Value>& value) const
{
v8::Isolate* isolate = m_scriptState->isolate();
v8::Local<v8::Context> context = m_scriptState->context();
- v8::Local<v8::String> prototypeString =
- v8AtomicString(isolate, "prototype");
+ v8::Local<v8::String> nameString = v8AtomicString(isolate, name);
+ return v8Call(object->Get(context, nameString), value);
+}
+
+bool ScriptCustomElementDefinitionBuilder::checkPrototype()
+{
v8::Local<v8::Value> prototypeValue;
- if (!v8Call(
- m_constructor->Get(context, prototypeString), prototypeValue)) {
+ if (!getValue(m_constructor, "prototype", prototypeValue))
return false;
- }
if (!prototypeValue->IsObject()) {
m_exceptionState.throwTypeError(
"constructor prototype is not an object");
@@ -82,6 +86,46 @@ bool ScriptCustomElementDefinitionBuilder::checkPrototype()
return true;
}
+bool ScriptCustomElementDefinitionBuilder::getCallable(const char* name,
+ v8::Local<v8::Object>& callback) const
+{
+ v8::Local<v8::Value> value;
+ if (!getValue(m_prototype, name, value))
+ return false;
+ if (value->IsUndefined())
+ return true;
+ if (!value->IsObject()) {
+ m_exceptionState.throwTypeError(
+ String::format("\"%s\" is not an object", name));
+ return false;
+ }
+ callback = value.As<v8::Object>();
+ if (!callback->IsCallable()) {
+ m_exceptionState.throwTypeError(
+ String::format("\"%s\" is not callable", name));
+ return false;
+ }
+ return true;
+}
+
+bool ScriptCustomElementDefinitionBuilder::cacheProperties()
+{
+ v8::Local<v8::Value> observedAttributesValue;
+ if (!getValue(m_constructor, "observedAttributes", observedAttributesValue))
yosin_UTC9 2016/06/01 06:33:33 Can we have const char* for these bar string liter
kojii 2016/06/01 07:19:27 Done.
+ return false;
+ if (!observedAttributesValue->IsUndefined()) {
+ m_observedAttributes = toImplArray<Vector<AtomicString>>(
+ observedAttributesValue, 0, m_scriptState->isolate(),
+ m_exceptionState);
+ if (m_exceptionState.hadException())
+ return false;
+ }
+
+ return getCallable("connectedCallback", m_connectedCallback)
+ && getCallable("disconnectedCallback", m_disconnectedCallback)
+ && getCallable("attributeChangedCallback", m_attributeChangedCallback);
+}
+
CustomElementDefinition* ScriptCustomElementDefinitionBuilder::build(
const CustomElementDescriptor& descriptor)
{
@@ -90,7 +134,11 @@ CustomElementDefinition* ScriptCustomElementDefinitionBuilder::build(
m_registry,
descriptor,
m_constructor,
- m_prototype);
+ m_prototype,
+ m_connectedCallback,
+ m_disconnectedCallback,
+ m_attributeChangedCallback,
+ m_observedAttributes);
}
} // namespace blink

Powered by Google App Engine
This is Rietveld 408576698