Update client side navigator.connect API to use ServicePortCollection [1/3]

public/platform/modules/navigator_services/WebServicePortProvider.h

Index: public/platform/modules/navigator_services/WebServicePortProvider.h
diff --git a/public/platform/modules/navigator_services/WebServicePortProvider.h b/public/platform/modules/navigator_services/WebServicePortProvider.h
new file mode 100644
index 0000000000000000000000000000000000000000..d75c99e7aa48a89a28da3e4790f24c19df6f21f8
--- /dev/null
+++ b/public/platform/modules/navigator_services/WebServicePortProvider.h
@@ -0,0 +1,67 @@
+// Copyright 2015 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 WebServicePortProvider_h
+#define WebServicePortProvider_h
+
+#include "public/platform/WebCallbacks.h"
+#include "public/platform/WebVector.h"
+#include "public/platform/modules/navigator_services/WebServicePort.h"
+
+namespace blink {
+
+class WebString;
+class WebURL;
+typedef WebVector<class WebMessagePortChannel*> WebMessagePortChannelArray;

[tkent, 2015/06/25 00:07:57]

typdef -> using

+
+typedef WebCallbacks<WebServicePortID, void> WebServicePortConnectCallbacks;

[tkent, 2015/06/25 00:07:57]

typedef -> using

[Marijn Kruisselbrink, 2015/06/25 01:07:26]

Done

+
+// One instance of this class is associated with each ServicePortCollection.
+// When a ServicePortCollection is destroyed the WebServicePortProvider
+// associated with it is also destroyed. All communication from ServicePorts and
+// the ServicePortCollection to embedding code goes via this interface.
+class WebServicePortProvider {
+public:
+    virtual void destroy() { BLINK_ASSERT_NOT_REACHED(); }
+
+    // Initiates a connection from the given origin to the given URL. When
+    // successful the service can communicate with the client over the given
+    // channel. The origin isn't passed as WebSecurityOrigin because that would
+    // be a layering violation (platform/ code shouldn't depend on web/ code).
+    // Ownership of the WebServicePortConnectCallbacks is transferred to the provider.
+    virtual void connect(const WebURL&, const WebString& origin, WebServicePortConnectCallbacks*) { BLINK_ASSERT_NOT_REACHED(); }
+
+    // Called when javascript code calls postMessage on a ServicePort that is owned
+    // by the ServicePortCollection this provider is associated with.
+    // Passes ownership of WebMessagePortChannelArray.
+    virtual void postMessage(WebServicePortID, const WebString&, WebMessagePortChannelArray*) { BLINK_ASSERT_NOT_REACHED(); }
+
+    // Called when a ServicePort owned by the ServicePortCollection this
+    // provider is associated with is closed (explicitly or via garbage
+    // collection).
+    virtual void closePort(WebServicePortID) { BLINK_ASSERT_NOT_REACHED(); }
+protected:

[tkent, 2015/06/25 00:07:57]

add a blank line before protected:

[Marijn Kruisselbrink, 2015/06/25 01:07:26]

Done

+    virtual ~WebServicePortProvider() { }

[tkent, 2015/06/25 00:07:57]

nit: now we allow to omit a space between { and }.

[Marijn Kruisselbrink, 2015/06/25 01:07:26]

Done

+};
+
+} // namespace blink
+
+#if INSIDE_BLINK
+
+namespace WTF {
+
+template<typename T> struct OwnedPtrDeleter;
+template<> struct OwnedPtrDeleter<blink::WebServicePortProvider> {
+    static void deletePtr(blink::WebServicePortProvider* provider)
+    {
+        if (provider)
+            provider->destroy();
+    }
+};
+
+} // namespace WTF
+
+#endif // INSIDE_BLINK
+
+#endif // WebServicePortProvider_h