Make Tracing Service not use outgoing InterfaceProvider, update conventions

services/tracing/data_sink.cc

Index: services/tracing/data_sink.cc
diff --git a/services/tracing/trace_data_sink.cc b/services/tracing/data_sink.cc
similarity index 73%
rename from services/tracing/trace_data_sink.cc
rename to services/tracing/data_sink.cc
index b278956648b15949b06e450e304daa325fb76997..dcd71844bd594b06956a5809e93530c4dcdccd1e 100644
--- a/services/tracing/trace_data_sink.cc
+++ b/services/tracing/data_sink.cc
@@ -2,7 +2,7 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#include "services/tracing/trace_data_sink.h"
+#include "services/tracing/data_sink.h"
 
 #include <utility>
 
@@ -13,16 +13,16 @@ using mojo::common::BlockingCopyFromString;
 
 namespace tracing {
 
-TraceDataSink::TraceDataSink(mojo::ScopedDataPipeProducerHandle pipe)
+DataSink::DataSink(mojo::ScopedDataPipeProducerHandle pipe)
     : pipe_(std::move(pipe)), empty_(true) {}
 
-TraceDataSink::~TraceDataSink() {
+DataSink::~DataSink() {
   if (pipe_.is_valid())
     pipe_.reset();
   DCHECK(!pipe_.is_valid());
 }
 
-void TraceDataSink::AddChunk(const std::string& json) {
+void DataSink::AddChunk(const std::string& json) {
   if (!empty_)
     BlockingCopyFromString(",", pipe_);
   empty_ = false;