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

Unified Diff: third_party/WebKit/Source/bindings/tests/results/core/LongCallbackFunction.h

Issue 2458573002: bindings: Store ScriptState in generated callback functions (Closed)
Patch Set: Created 4 years, 2 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/tests/results/core/LongCallbackFunction.h
diff --git a/third_party/WebKit/Source/bindings/tests/results/core/LongCallbackFunction.h b/third_party/WebKit/Source/bindings/tests/results/core/LongCallbackFunction.h
index bf566ee024e77c561d1e40b720c93cc487d2778a..55f25c263f7249b8df452933ded96a420414c9bf 100644
--- a/third_party/WebKit/Source/bindings/tests/results/core/LongCallbackFunction.h
+++ b/third_party/WebKit/Source/bindings/tests/results/core/LongCallbackFunction.h
@@ -22,9 +22,9 @@ class ScriptState;
class CORE_EXPORT LongCallbackFunction final : public GarbageCollectedFinalized<LongCallbackFunction>,
public TraceWrapperBase {
public:
- static LongCallbackFunction* create(v8::Isolate* isolate, v8::Local<v8::Function> callback)
+ static LongCallbackFunction* create(ScriptState* scriptState, v8::Local<v8::Function> callback)
{
- return new LongCallbackFunction(isolate, callback);
+ return new LongCallbackFunction(scriptState, callback);
}
~LongCallbackFunction() = default;
@@ -32,7 +32,7 @@ public:
DECLARE_TRACE();
DECLARE_TRACE_WRAPPERS();
- bool call(ScriptState* scriptState, ScriptWrappable* scriptWrappable, int num1, int num2, int& returnValue);
+ bool call(ScriptWrappable* scriptWrappable, int num1, int num2, int& returnValue);
v8::Local<v8::Function> v8Value(v8::Isolate* isolate)
{
@@ -46,7 +46,9 @@ public:
}
private:
- LongCallbackFunction(v8::Isolate* isolate, v8::Local<v8::Function>);
+ LongCallbackFunction(ScriptState*, v8::Local<v8::Function>);
+
+ RefPtr<ScriptState> m_scriptState;
ScopedPersistent<v8::Function> m_callback;
};

Powered by Google App Engine
This is Rietveld 408576698