Added SuspendableScriptExecutor::createAndRun to replace ctor+run

Source/web/SuspendableScriptExecutor.h

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef SuspendableScriptExecutor_h
 #define SuspendableScriptExecutor_h
 
 #include "core/dom/ActiveDOMObject.h"
 #include "wtf/OwnPtr.h"
 #include "wtf/Vector.h"
 
 namespace blink {
 
 class LocalFrame;
 class ScriptSourceCode;
 class WebScriptExecutionCallback;
 
 class SuspendableScriptExecutor final : public ActiveDOMObject {
 public:
+    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();
 
-    // this method must be called only once
     void run();
-
-    virtual void resume() override;
-
-    virtual void contextDestroyed() override;
-
-private:
     void executeAndDestroySelf();
 
     LocalFrame* m_frame;
     int m_worldID;
     Vector<ScriptSourceCode> m_sources;
     int m_extensionGroup;
     bool m_userGesture;
     WebScriptExecutionCallback* m_callback;
 };
 
 } // namespace blink
 
 #endif // SuspendableScriptExecutor_h