Decouple ScriptPromise creation from ScriptPromiseResolver.

Source/bindings/v8/ScriptPromise.h

Index: Source/bindings/v8/ScriptPromise.h
diff --git a/Source/bindings/v8/ScriptPromise.h b/Source/bindings/v8/ScriptPromise.h
index c4f3e5dd07f88e8295eb7265bb111c55e873b965..e3d0da7841ef54273335507a94222b122729e2d5 100644
--- a/Source/bindings/v8/ScriptPromise.h
+++ b/Source/bindings/v8/ScriptPromise.h
@@ -45,6 +45,7 @@ namespace WebCore {
 //
 class ScriptPromise {
 public:
+    // Constructs an empty promise.
     ScriptPromise()
         : m_promise()
     {
@@ -92,6 +93,11 @@ public:
         m_promise.clear();
     }
 
+    // Creates a pending promise.
+    static ScriptPromise create();
+    // Creates a pending promise.
+    static ScriptPromise create(ScriptExecutionContext*);

[abarth-chromium, 2013/10/11 18:18:02]

Rather than write comments like these, we prefer if you put those ideas into the
names of the functions.  These function should be called:

ScriptPromise::createPending

Then all the callers will know that they're creating pending promises.

[yhirano, 2013/10/14 02:49:53]

Done.

+
 private:
     ScriptValue m_promise;
 };