Pass script state to Web API object->script wrappable converter.

Source/bindings/v8/CallbackPromiseAdapter.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 29 matching lines @@
 // This class provides an easy way to convert from a Script-exposed
 // class (i.e. a class that has a toV8() overload) that uses Promises
 // to a WebKit API class that uses WebCallbacks. You can define
 // seperate Success and Error classes, but this example just uses one
 // object for both.
 //
 // To use:
 //
 // class MyClass ... {
 //    typedef blink::WebMyClass WebType;
-//    static PassRefPtr<MyClass> from(blink::WebMyClass* webInstance) {
+//    static PassRefPtr<MyClass> from(NewScriptState*,
+//                                    blink::WebMyClass* webInstance) {
 //        // convert/create as appropriate, but often it's just:
 //        return MyClass::create(adoptPtr(webInstance));
 //    }
 //
 // Now when calling into a WebKit API that requires a WebCallbacks<blink::WebMyClass, blink::WebMyClass>*:
 //
 //    // call signature: callSomeMethod(WebCallbacks<MyClass, MyClass>* callbacks)
 //    webObject->callSomeMethod(new CallbackPromiseAdapter<MyClass, MyClass>(resolver, scriptExecutionContext));
 //
 // Note that this class does not manage its own lifetime. In this
 // example that ownership of the WebCallbacks instance is being passed
 // in and it is up to the callee to free the WebCallbacks instace.
 template<typename S, typename T>
 class CallbackPromiseAdapter FINAL : public blink::WebCallbacks<typename S::WebType, typename T::WebType> {
 public:
     CallbackPromiseAdapter(PassRefPtr<ScriptPromiseResolver> resolver, ExecutionContext* context)
         : m_resolver(resolver)
         , m_scriptState(NewScriptState::current(toIsolate(context)))
     {
     }
     virtual ~CallbackPromiseAdapter() { }
 
     virtual void onSuccess(typename S::WebType* result) OVERRIDE
     {
         NewScriptState::Scope scope(m_scriptState.get());
-        m_resolver->resolve(S::from(result));
+        m_resolver->resolve(S::from(m_scriptState.get(), result));
     }
     virtual void onError(typename T::WebType* error) OVERRIDE
     {
         NewScriptState::Scope scope(m_scriptState.get());
         m_resolver->reject(T::from(error));

[michaeln1, 2014/04/11 23:46:29]

Since this is a generic template, should it also anticipate that the errorT may
need this treatment too? If its not needed, the ::from function can just ignore
it.

     }
 private:
     RefPtr<ScriptPromiseResolver> m_resolver;
     RefPtr<NewScriptState> m_scriptState;
     WTF_MAKE_NONCOPYABLE(CallbackPromiseAdapter);
 };
 
 } // namespace WebCore
 
 #endif