ServiceWorker: register() should return the existing registration object if available

public/platform/WebServiceWorkerRegistrationProxy.h

Index: public/platform/WebServiceWorkerRegistrationProxy.h
diff --git a/public/platform/WebServiceWorkerRegistrationProxy.h b/public/platform/WebServiceWorkerRegistrationProxy.h
index 57d1404938faaaaa664fd941fc641a816e82052d..a6eca04ea1a5c84d026814265b9c9db96a6f6ebd 100644
--- a/public/platform/WebServiceWorkerRegistrationProxy.h
+++ b/public/platform/WebServiceWorkerRegistrationProxy.h
@@ -5,8 +5,11 @@
 #ifndef WebServiceWorkerRegistrationProxy_h
 #define WebServiceWorkerRegistrationProxy_h
 
+#include "WebCommon.h"

[tkent, 2014/08/14 05:33:15]

should be "public/platform/WebCommon.h"

[nhiroki, 2014/08/14 06:15:19]

Done.

+
 namespace blink {
 
+class ServiceWorkerRegistration;
 class WebServiceWorker;
 
 // A proxy interface, passed via WebServiceWorkerRegistration.setProxy() from
@@ -14,7 +17,7 @@ class WebServiceWorker;
 // embedder.
 class WebServiceWorkerRegistrationProxy {
 public:
-    WebServiceWorkerRegistrationProxy() { }
+    WebServiceWorkerRegistrationProxy() : m_private(0) { }
     virtual ~WebServiceWorkerRegistrationProxy() { }
 
     // Notifies that the registration entered the installation process.
@@ -25,6 +28,14 @@ public:
     virtual void setInstalling(WebServiceWorker*) = 0;
     virtual void setWaiting(WebServiceWorker*) = 0;
     virtual void setActive(WebServiceWorker*) = 0;
+
+#if INSIDE_BLINK
+    BLINK_PLATFORM_EXPORT WebServiceWorkerRegistrationProxy(ServiceWorkerRegistration*);
+    BLINK_PLATFORM_EXPORT ServiceWorkerRegistration* unwrap() const;

[tkent, 2014/08/14 05:33:15]

We usually define |operator ServiceWorkerRegistration*() const| instead of
defining a normal function.

[nhiroki, 2014/08/14 06:15:19]

Done.

+#endif
+
+protected:
+    ServiceWorkerRegistration* m_private;
 };
 
 } // namespace blink