tracing: Make ConvertableToTraceFormat move-only scoped_ptr

cc/debug/benchmark_instrumentation.cc

Index: cc/debug/benchmark_instrumentation.cc
diff --git a/cc/debug/benchmark_instrumentation.cc b/cc/debug/benchmark_instrumentation.cc
index 5a04e214ed1a6d8eff90bef0bf1ddad25b877ddd..9a282e7f4ad0de37b3232961184c9d4f497588fd 100644
--- a/cc/debug/benchmark_instrumentation.cc
+++ b/cc/debug/benchmark_instrumentation.cc
@@ -20,15 +20,11 @@ void IssueImplThreadRenderingStatsEvent(const RenderingStats& stats) {
 }
 
 void IssueDisplayRenderingStatsEvent() {
-  scoped_refptr<base::trace_event::TracedValue> record_data =
-      new base::trace_event::TracedValue();
+  auto record_data = make_scoped_ptr(new base::trace_event::TracedValue());

[danakj, 2016/02/25 23:10:07]

current guidance is "don't spell smart pointers as auto".

we haz plans to improve that but for now spell it out please. you could write
scoped_ptr<T> value(new T); to keep it more terse while writing the type.

[Primiano Tucci (use gerrit), 2016/02/26 07:01:04]

Ok changed all the places in the CL where I did auto ... = make_scoped to be
more verbose.

FWIW btw, despite this CL the rest of the codebase seems to have plenty of auto
.* = make_scoped (I might have contributed quite to that a bit in the past :-/)

   record_data->SetInteger("frame_count", 1);
   TRACE_EVENT_INSTANT1(
-      "benchmark",
-      "BenchmarkInstrumentation::DisplayRenderingStats",
-      TRACE_EVENT_SCOPE_THREAD,
-      "data",
-      scoped_refptr<base::trace_event::ConvertableToTraceFormat>(record_data));
+      "benchmark", "BenchmarkInstrumentation::DisplayRenderingStats",
+      TRACE_EVENT_SCOPE_THREAD, "data", std::move(record_data));
 }
 
 }  // namespace benchmark_instrumentation