Expose navigator.connect client side API in blink.

public/platform/WebNavigatorConnectProvider.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 WebNavigatorConnectProvider_h
+#define WebNavigatorConnectProvider_h
+
+#include "WebCallbacks.h"
+
+namespace blink {
+
+class WebMessagePortChannel;
+class WebURL;
+
+class WebNavigatorConnectProvider {
+public:
+    virtual ~WebNavigatorConnectProvider() { }
+
+    // Initiates a connection to the given URL. When succesfull the service
+    // can communicate with the client over the given channel.
+    // Ownership of the WebCallbacks is transferred to the provider.
+    virtual void connect(const WebURL&, WebMessagePortChannel*, WebCallbacks<void, void>*) { }

[scheib, 2014/12/05 00:16:40]

Probably leave this pure virtual. The platform default impl will return NULL for
provider until implemented, but if an implementation exists it should probably
compile fail unless it overrides connect.

[Marijn Kruisselbrink, 2014/12/05 09:23:43]

Ideally, yes. But I've been told that while blink and chromium are still
separate repositories, it's better to avoid marking methods as pure virtual (as
well avoid marking implementations of these as override) to make it easier to
change things. Pure virtual methods (and marking methods as override) makes
changes to these interface even harder than they already are in the split repo
world.

[scheib, 2014/12/05 17:39:35]

OK with me, check with the platform OWNER I suppose. ;) I had pushback on recent
patch.

+};
+
+} // namespace blink
+
+#endif // WebNavigatorConnectProvider_h