tracing: Make ConvertableToTraceFormat move-only scoped_ptr

base/trace_event/heap_profiler_heap_dump_writer_unittest.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <stddef.h>
 
 #include <set>
 #include <string>
 
 #include "base/json/json_reader.h"
 #include "base/macros.h"
-#include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/trace_event/heap_profiler_allocation_context.h"
 #include "base/trace_event/heap_profiler_heap_dump_writer.h"
 #include "base/trace_event/heap_profiler_stack_frame_deduplicator.h"
 #include "base/trace_event/heap_profiler_type_name_deduplicator.h"
 #include "base/trace_event/trace_event_argument.h"
 #include "base/values.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace {
@@ skipping 10 matching lines @@
 const char kBool[] = "bool";
 const char kString[] = "string";
 
 }  // namespace
 
 namespace base {
 namespace trace_event {
 namespace internal {
 
 scoped_ptr<const Value> WriteAndReadBack(const std::set<Entry>& entries) {
-  scoped_refptr<TracedValue> traced_value = Serialize(entries);
+  scoped_ptr<TracedValue> traced_value = Serialize(entries);
   std::string json;
   traced_value->AppendAsTraceFormat(&json);
   return JSONReader::Read(json);
 }
 
 scoped_ptr<const DictionaryValue> WriteAndReadBackEntry(Entry entry) {
   std::set<Entry> input_entries;
   input_entries.insert(entry);
 
   scoped_ptr<const Value> json_dict = WriteAndReadBack(input_entries);
@@ skipping 137 matching lines @@
   // At this point the heap looks like this:
   //
   // |        | CrWidget <- BrMain | Init <- RenMain | Sum |
   // +--------+--------------------+-----------------+-----+
   // | int    |                 10 |               0 |  10 |
   // | bool   |                 18 |              30 |  48 |
   // | string |                  0 |              19 |  19 |
   // +--------+--------------------+-----------------+-----+
   // | Sum    |                 28 |              49 |  77 |
 
-  auto sf_deduplicator = make_scoped_refptr(new StackFrameDeduplicator);
-  auto tn_deduplicator = make_scoped_refptr(new TypeNameDeduplicator);
+  auto sf_deduplicator = make_scoped_ptr(new StackFrameDeduplicator);
+  auto tn_deduplicator = make_scoped_ptr(new TypeNameDeduplicator);
   HeapDumpWriter writer(sf_deduplicator.get(), tn_deduplicator.get());
   const std::set<Entry>& dump = writer.Summarize(bytes_by_context);
 
   // Get the indices of the backtraces and types by adding them again to the
   // deduplicator. Because they were added before, the same number is returned.
   StackFrame bt0[] = {kRendererMain, kInitialize};
   StackFrame bt1[] = {kBrowserMain, kCreateWidget};
   int bt_renderer_main = sf_deduplicator->Insert(bt0, bt0 + 1);
   int bt_browser_main = sf_deduplicator->Insert(bt1, bt1 + 1);
   int bt_renderer_main_initialize = sf_deduplicator->Insert(bt0, bt0 + 2);
@@ skipping 43 matching lines @@
   bytes_by_context[ctx] = 512;
 
   // 1 MiB in BrowserMain -> CreateWidget -> GetBitmap.
   ctx.backtrace.frames[2] = kGetBitmap;
   bytes_by_context[ctx] = 1024 * 1024;
 
   // 0.5 KiB in BrowserMain -> CreateWidget -> Initialize.
   ctx.backtrace.frames[2] = kInitialize;
   bytes_by_context[ctx] = 512;
 
-  auto sf_deduplicator = make_scoped_refptr(new StackFrameDeduplicator);
-  auto tn_deduplicator = make_scoped_refptr(new TypeNameDeduplicator);
+  auto sf_deduplicator = make_scoped_ptr(new StackFrameDeduplicator);
+  auto tn_deduplicator = make_scoped_ptr(new TypeNameDeduplicator);
   HeapDumpWriter writer(sf_deduplicator.get(), tn_deduplicator.get());
   const std::set<Entry>& dump = writer.Summarize(bytes_by_context);
 
   // Get the indices of the backtraces and types by adding them again to the
   // deduplicator. Because they were added before, the same number is returned.
   StackFrame bt0[] = {kBrowserMain, kCreateWidget, kGetBitmap};
   StackFrame bt1[] = {kBrowserMain, kCreateWidget, kInitialize};
   int bt_browser_main = sf_deduplicator->Insert(bt0, bt0 + 1);
   int bt_create_widget = sf_deduplicator->Insert(bt0, bt0 + 2);
   int bt_get_bitmap = sf_deduplicator->Insert(bt0, bt0 + 3);
@@ skipping 11 matching lines @@
   AssertSizeEq(dump, bt_create_widget, -1, 1024 * 1024 + 512 + 512);
   AssertSizeEq(dump, bt_browser_main, -1, 1024 * 1024 + 512 + 512);
 
   // Initialize was not significant, it should not have been dumped.
   AssertNotDumped(dump, bt_initialize, -1);
 }
 
 }  // namespace internal
 }  // namespace trace_event
 }  // namespace base