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

Unified Diff: third_party/WebKit/Source/bindings/templates/callback_function.h.tmpl

Issue 2617653002: bindings: Don't throw a TypeError when 'null' is passed to nullable callback function (Closed)
Patch Set: Created 3 years, 11 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/templates/callback_function.h.tmpl
diff --git a/third_party/WebKit/Source/bindings/templates/callback_function.h.tmpl b/third_party/WebKit/Source/bindings/templates/callback_function.h.tmpl
index 383cf40d9cee5ea88ece6b873343e7e0e8b9e8fb..133cfa1a53548c0e1b106792068d867b51404a4f 100644
--- a/third_party/WebKit/Source/bindings/templates/callback_function.h.tmpl
+++ b/third_party/WebKit/Source/bindings/templates/callback_function.h.tmpl
@@ -18,9 +18,7 @@ class {{forward_declaration}};
class {{exported}}{{cpp_class}} final : public GarbageCollectedFinalized<{{cpp_class}}>, public TraceWrapperBase {
public:
- static {{cpp_class}}* create(ScriptState* scriptState, v8::Local<v8::Function> callback) {
- return new {{cpp_class}}(scriptState, callback);
- }
+ static {{cpp_class}}* create(ScriptState* scriptState, v8::Local<v8::Value> callback);
~{{cpp_class}}() = default;

Powered by Google App Engine
This is Rietveld 408576698