Service Worker: Use ScriptPromise instead of ScriptPromise&.

Source/modules/serviceworkers/FetchEvent.h

Index: Source/modules/serviceworkers/FetchEvent.h
diff --git a/Source/modules/serviceworkers/FetchEvent.h b/Source/modules/serviceworkers/FetchEvent.h
index f81d01ddd897daf4253c88507dc1708fa0a8f08e..fdd47b39ded97d85cd132273c06dab3052986c88 100644
--- a/Source/modules/serviceworkers/FetchEvent.h
+++ b/Source/modules/serviceworkers/FetchEvent.h
@@ -5,6 +5,7 @@
 #ifndef FetchEvent_h
 #define FetchEvent_h
 
+#include "bindings/core/v8/ScriptPromise.h"
 #include "modules/EventModules.h"
 #include "modules/ModulesExport.h"
 #include "modules/fetch/Request.h"
@@ -18,6 +19,7 @@ namespace blink {
 class ExceptionState;
 class Request;
 class RespondWithObserver;
+class ScriptPromise;

[nhiroki, 2015/07/29 08:52:18]

nit: ScriptState? (This file already included ScriptPromise.h)

[jungkees, 2015/07/29 09:12:23]

Yeah, right. I finally removed the forward declaration for this case.

I just wonder what'd be the best practice for it. IMHO, this header file doesn't
need the *definition* of the ScriptPromise, so it seems a forward declaration is
enough here. But in that case, I'll need to include ScriptPromise.h file in
FetchEvent.cpp anyway where the definition is needed, right?

 
 // A fetch event is dispatched by the client to a service worker's script
 // context. RespondWithObserver can be used to notify the client about the
@@ -32,7 +34,7 @@ public:
     Request* request() const;
     bool isReload() const;
 
-    void respondWith(ScriptState*, ScriptPromise&, ExceptionState&);
+    void respondWith(ScriptState*, ScriptPromise, ExceptionState&);
 
     const AtomicString& interfaceName() const override;