Make it possible to have multiple InspectorBackends

sky/viewer/services/inspector_impl.cc

Index: sky/viewer/services/inspector_impl.cc
diff --git a/sky/viewer/services/inspector_impl.cc b/sky/viewer/services/inspector_impl.cc
index a5343fa9600723ac64b2513f202966330e1700ab..4c8d086f3ed98ed58d565e0bcd090da8482667ae 100644
--- a/sky/viewer/services/inspector_impl.cc
+++ b/sky/viewer/services/inspector_impl.cc
@@ -4,10 +4,12 @@
 
 #include "sky/viewer/services/inspector_impl.h"
 
+#include "base/bind.h"
 #include "sky/engine/public/web/WebDocument.h"
 #include "sky/engine/public/web/WebElement.h"
 #include "sky/engine/public/web/WebFrame.h"
 #include "sky/engine/public/web/WebView.h"
+#include "sky/services/inspector/inspector.mojom.h"
 #include "sky/viewer/document_view.h"
 
 namespace sky {
@@ -19,9 +21,21 @@ InspectorServiceImpl::InspectorServiceImpl(DocumentView* view)
 InspectorServiceImpl::~InspectorServiceImpl() {
 }
 
+void Ignored() {}

[abarth-chromium, 2014/11/11 02:53:57]

Can you move this to around line 15 and put it in an anonymous namespace?

+
 void InspectorServiceImpl::Inject() {
   if (!view_)
     return;
+
+  mojo::ServiceProviderPtr inpector_service_provider;

[abarth-chromium, 2014/11/11 02:53:57]

s/inpector_service_provider/inspector_service_provider/

+  view_->shell()->ConnectToApplication("mojo:sky_inspector_server",
+                                       GetProxy(&inpector_service_provider));
+  InspectorServerPtr inspector;
+  ConnectToService(inpector_service_provider.get(), &inspector);
+  inspector->Listen(9898, base::Bind(&Ignored));
+  // Listen drops existing agents/backends, wait before registering new ones.
+  inspector.WaitForIncomingMethodCall();
+
   view_->web_view()->injectModule("/sky/framework/inspector/inspector.sky");
 }