Decouple ScriptPromise creation from ScriptPromiseResolver.

Source/bindings/v8/ScriptPromise.h

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 20 matching lines @@
 #ifndef ScriptPromise_h
 #define ScriptPromise_h
 
 #include "bindings/v8/ScopedPersistent.h"
 #include "bindings/v8/ScriptValue.h"
 #include "bindings/v8/V8ScriptRunner.h"
 #include <v8.h>
 
 namespace WebCore {
 
+class ExecutionContext;
+
 // ScriptPromise is the class for representing Promise values in C++ world.
 // ScriptPromise holds a Promise.
 // So holding a ScriptPromise as a member variable in DOM object causes
 // memory leaks since it has a reference from C++ to V8.
 //
 class ScriptPromise {
 public:
+    // Constructs an empty promise.
     ScriptPromise()
         : m_promise()
     {
     }
 
     explicit ScriptPromise(const ScriptValue& promise)
         : m_promise(promise)
     {
         ASSERT(!m_promise.hasNoValue());
     }
@@ skipping 27 matching lines @@
     bool hasNoValue() const
     {
         return m_promise.hasNoValue();
     }
 
     void clear()
     {
         m_promise.clear();
     }
 
+    // Creates a pending promise.

[abarth-chromium, 2013/10/15 17:40:51]

These comments aren't needed anymore.  We can see from the name what it does.

+    static ScriptPromise createPending();
+    // Creates a pending promise.
+    static ScriptPromise createPending(ExecutionContext*);
+
 private:
     ScriptValue m_promise;
 };
 
 } // namespace WebCore
 
 
 #endif // ScriptPromise_h