Enable connection to Mojo services from Blink

third_party/WebKit/public/platform/Platform.h

Index: third_party/WebKit/public/platform/Platform.h
diff --git a/third_party/WebKit/public/platform/Platform.h b/third_party/WebKit/public/platform/Platform.h
index 4264a4cd22d5e6a7a3757f12025b6c572370b9dc..c92b047ec714d442efdfaf92e59ab583c8576dd0 100644
--- a/third_party/WebKit/public/platform/Platform.h
+++ b/third_party/WebKit/public/platform/Platform.h
@@ -56,6 +56,7 @@
 #include "base/metrics/user_metrics_action.h"
 #include "cc/resources/shared_bitmap.h"
 #include "cc/surfaces/frame_sink_id.h"
+#include "services/service_manager/public/interfaces/connector.mojom-blink.h"

[Sam McNally, 2016/12/20 07:29:38]

We shouldn't exposing blink-variant mojoms outside blink.

[blundell, 2016/12/20 07:39:43]

Can you elaborate? I understand that in general, the Blink variants are for
usage strictly within Blink. This exposure is that so that the embedder can give
Blink its service connection. I can do this via an untyped handle, but just
curious on what the exact reasoning is in this case.

[Sam McNally, 2016/12/20 08:57:18]

Blink variants of mojoms include WTF types. Using them in blink public
interfaces leaks them into the rest of the codebase. See
https://bugs.chromium.org/p/chromium/issues/detail?id=621177 for another case
where this occurred.

[blundell, 2016/12/20 09:01:57]

Got it, thanks. In making this change, I'll also be able to reland this CL :).

 
 namespace gpu {
 class GpuMemoryBufferManager;
@@ -604,6 +605,9 @@ class BLINK_PLATFORM_EXPORT Platform {
 
   virtual InterfaceProvider* interfaceProvider();
 
+  virtual void bindServiceConnector(
+      service_manager::mojom::blink::ConnectorRequest);
+
   // Platform events -----------------------------------------------------
   // Device Orientation, Device Motion, Device Light, Battery, Gamepad.