Added SuspendableScriptExecutor::createAndRun to replace ctor+run

Source/web/SuspendableScriptExecutor.h

Index: Source/web/SuspendableScriptExecutor.h
diff --git a/Source/web/SuspendableScriptExecutor.h b/Source/web/SuspendableScriptExecutor.h
index b2d8489c72f355115ae3020fb9c54c4093a7896b..0ddefd23fd8b3b8474e7790123f1fb0715ebfdca 100644
--- a/Source/web/SuspendableScriptExecutor.h
+++ b/Source/web/SuspendableScriptExecutor.h
@@ -17,17 +17,16 @@ class WebScriptExecutionCallback;
 
 class SuspendableScriptExecutor final : public ActiveDOMObject {
 public:
-    SuspendableScriptExecutor(LocalFrame*, int worldID, const Vector<ScriptSourceCode>& sources, int extensionGroup, bool userGesture, WebScriptExecutionCallback*);
-    virtual ~SuspendableScriptExecutor();
-
-    // this method must be called only once
-    void run();
+    static void createAndRun(LocalFrame*, int worldID, const Vector<ScriptSourceCode>& sources, int extensionGroup, bool userGesture, WebScriptExecutionCallback*);
 
     virtual void resume() override;

[aandrey, 2014/10/24 09:22:00]

should these be private as well?
what if I call resume() twice?

[kozyatinskiy1, 2014/10/24 09:43:39]

This method is public in base class and someone always can cast pointer to base
class and call this method. This method uses by notifier to notify this object
about context resumed event.
Now we can receive pointer to this object and can't call method resume manually.

-
     virtual void contextDestroyed() override;
 
 private:
+    SuspendableScriptExecutor(LocalFrame*, int worldID, const Vector<ScriptSourceCode>& sources, int extensionGroup, bool userGesture, WebScriptExecutionCallback*);
+    virtual ~SuspendableScriptExecutor();
+
+    void run();
     void executeAndDestroySelf();
 
     LocalFrame* m_frame;