Implement WindowTimers.set{Timeout,Interval} without [Custom]

Source/bindings/core/v8/ScheduledAction.h

Index: Source/bindings/core/v8/ScheduledAction.h
diff --git a/Source/bindings/core/v8/ScheduledAction.h b/Source/bindings/core/v8/ScheduledAction.h
index e41c145e6afe92c1b281254567df317eb3cfe64a..f66f4797d79660171ae2ec437841503b669555cd 100644
--- a/Source/bindings/core/v8/ScheduledAction.h
+++ b/Source/bindings/core/v8/ScheduledAction.h
@@ -47,13 +47,17 @@ class WorkerGlobalScope;
 class ScheduledAction {
     WTF_MAKE_NONCOPYABLE(ScheduledAction);
 public:
-    ScheduledAction(ScriptState*, v8::Handle<v8::Function>, int argc, v8::Handle<v8::Value> argv[], v8::Isolate*);
-    ScheduledAction(ScriptState*, const String&, const KURL&, v8::Isolate*);
+    static PassOwnPtr<ScheduledAction> create(ScriptState*, const ScriptValue&, const Vector<ScriptValue>&);

[Yuta Kitamura, 2014/11/14 08:24:06]

The meaning of the second and third arguments is not immediately
obvious; you could put parameter names for these.

[Jens Widell, 2014/11/14 10:19:27]

Done.

+    static PassOwnPtr<ScheduledAction> create(ScriptState*, const String&);

[Yuta Kitamura, 2014/11/14 08:24:06]

Ditto for the second argument.

[Jens Widell, 2014/11/14 10:19:27]

Done.

+
     ~ScheduledAction();
 
     void execute(ExecutionContext*);
 
 private:
+    ScheduledAction(ScriptState*, const ScriptValue&, const Vector<ScriptValue>&);
+    ScheduledAction(ScriptState*, const String&);

[Yuta Kitamura, 2014/11/14 08:24:06]

Ditto.

[Jens Widell, 2014/11/14 10:19:27]

Done.

+
     void execute(LocalFrame*);
     void execute(WorkerGlobalScope*);
     void createLocalHandlesForArgs(Vector<v8::Handle<v8::Value> >* handles);