De-client tracing.TraceController interface

services/tracing/main.cc

Index: services/tracing/main.cc
diff --git a/services/tracing/main.cc b/services/tracing/main.cc
index dc2dec56f0998c569ff4f791597db6e67e7f671a..c0f9b87cad36281c19925bf42071dd730876a4e4 100644
--- a/services/tracing/main.cc
+++ b/services/tracing/main.cc
@@ -3,21 +3,46 @@
 // found in the LICENSE file.
 
 #include "base/bind.h"
+#include "base/memory/scoped_vector.h"
 #include "mojo/application/application_runner_chromium.h"
 #include "mojo/common/weak_binding_set.h"
+#include "mojo/common/weak_interface_ptr_set.h"
 #include "mojo/public/c/system/main.h"
 #include "mojo/public/cpp/application/application_delegate.h"
 #include "mojo/public/cpp/application/application_impl.h"
+#include "mojo/public/cpp/bindings/strong_binding.h"
 #include "services/tracing/trace_data_sink.h"
 #include "services/tracing/tracing.mojom.h"
 
 namespace tracing {
 
+namespace {
+
+class CollectorImpl : public TraceDataCollector {
+ public:
+  CollectorImpl(mojo::InterfaceRequest<TraceDataCollector> request,
+                TraceDataSink* sink)
+      : sink_(sink), binding_(this, request.Pass()) {}
+
+  ~CollectorImpl() override {}
+
+  // tracing::TraceDataCollector implementation.
+  void DataCollected(const mojo::String& json) override {
+    sink_->AddChunk(json.To<std::string>());
+  }
+
+ private:
+  TraceDataSink* sink_;
+  mojo::Binding<TraceDataCollector> binding_;
+
+  DISALLOW_COPY_AND_ASSIGN(CollectorImpl);
+};
+
+}  // namespace
+
 class TracingApp : public mojo::ApplicationDelegate,
                    public mojo::InterfaceFactory<TraceCoordinator>,
-                   public tracing::TraceCoordinator,
-                   public mojo::InterfaceFactory<TraceDataCollector>,
-                   public tracing::TraceDataCollector {
+                   public TraceCoordinator {
  public:
   TracingApp() {}
   ~TracingApp() override {}
@@ -27,7 +52,15 @@ class TracingApp : public mojo::ApplicationDelegate,
   bool ConfigureIncomingConnection(
       mojo::ApplicationConnection* connection) override {
     connection->AddService<TraceCoordinator>(this);
-    connection->AddService<TraceDataCollector>(this);
+
+    // If someone connects to us they may want to use the TraceCoordinator
+    // interface and/or they may want to expose themselves to be traced. Attempt
+    // to connect to the TraceController interface to see if the application
+    // connecting to us wants to be traced. They can refuse the connection or
+    // close the pipe if not.
+    TraceControllerPtr controller_ptr;
+    connection->ConnectToService(&controller_ptr);
+    controller_ptrs_.AddInterfacePtr(controller_ptr.Pass());
     return true;
   }
 
@@ -37,21 +70,20 @@ class TracingApp : public mojo::ApplicationDelegate,
     coordinator_bindings_.AddBinding(this, request.Pass());
   }
 
-  // mojo::InterfaceFactory<TraceDataCollector> implementation.
-  void Create(mojo::ApplicationConnection* connection,
-              mojo::InterfaceRequest<TraceDataCollector> request) override {
-    collector_bindings_.AddBinding(this, request.Pass());
-  }
-
   // tracing::TraceCoordinator implementation.
   void Start(mojo::ScopedDataPipeProducerHandle stream,
              const mojo::String& categories) override {
     sink_.reset(new TraceDataSink(stream.Pass()));
-    collector_bindings_.ForAllBindings([categories](
-        TraceController* controller) { controller->StartTracing(categories); });
+    controller_ptrs_.ForAllPtrs(
+        [categories, this](TraceController* controller) {
+          TraceDataCollectorPtr ptr;
+          collector_impls_.push_back(
+              new CollectorImpl(GetProxy(&ptr), sink_.get()));
+          controller->StartTracing(categories, ptr.Pass());
+        });
   }
   void StopAndFlush() override {
-    collector_bindings_.ForAllBindings(
+    controller_ptrs_.ForAllPtrs(
         [](TraceController* controller) { controller->StopTracing(); });
 
     // TODO: We really should keep track of how many connections we have here
@@ -59,20 +91,18 @@ class TracingApp : public mojo::ApplicationDelegate,
     // pipe closure on all pipes.
     base::MessageLoop::current()->PostDelayedTask(
         FROM_HERE,
-        base::Bind(&TraceDataSink::Flush, base::Unretained(sink_.get())),
+        base::Bind(&TracingApp::AllDataCollected, base::Unretained(this)),
         base::TimeDelta::FromSeconds(1));
   }
 
-  // tracing::TraceDataCollector implementation.
-  void DataCollected(const mojo::String& json) override {
-    if (sink_)
-      sink_->AddChunk(json.To<std::string>());
+  void AllDataCollected() {
+    collector_impls_.clear();
+    sink_->Flush();
   }
-  // tracing::TraceDataCollector implementation.
-  void EndTracing() override {}
 
   scoped_ptr<TraceDataSink> sink_;
-  mojo::WeakBindingSet<TraceDataCollector> collector_bindings_;
+  ScopedVector<CollectorImpl> collector_impls_;
+  mojo::WeakInterfacePtrSet<TraceController> controller_ptrs_;
   mojo::WeakBindingSet<TraceCoordinator> coordinator_bindings_;
 
   DISALLOW_COPY_AND_ASSIGN(TracingApp);