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

public/platform/WebServiceWorkerRegistrationProxy.h

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #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
 // blink to the embedder, to talk to the ServiceWorkerRegistration object from
 // embedder.
 class WebServiceWorkerRegistrationProxy {
 public:
-    WebServiceWorkerRegistrationProxy() { }
+    WebServiceWorkerRegistrationProxy() : m_private(0) { }
     virtual ~WebServiceWorkerRegistrationProxy() { }
 
     // Notifies that the registration entered the installation process.
     // The installing worker should be accessible via
     // WebServiceWorkerRegistration.installing.
     virtual void dispatchUpdateFoundEvent() = 0;
 
     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
 
 #endif // WebServiceWorkerRegistrationProxy_h