tracing: Make ConvertableToTraceFormat move-only scoped_ptr

base/trace_event/memory_dump_session_state.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 "base/trace_event/memory_dump_session_state.h"
 
 namespace base {
 namespace trace_event {
 
-MemoryDumpSessionState::MemoryDumpSessionState(
-    const scoped_refptr<StackFrameDeduplicator>& stack_frame_deduplicator,
-    const scoped_refptr<TypeNameDeduplicator>& type_name_deduplicator)
-    : stack_frame_deduplicator_(stack_frame_deduplicator),
-      type_name_deduplicator_(type_name_deduplicator) {}
+MemoryDumpSessionState::MemoryDumpSessionState() {}
 
-MemoryDumpSessionState::~MemoryDumpSessionState() {
+MemoryDumpSessionState::~MemoryDumpSessionState() {}
+
+void MemoryDumpSessionState::SetStackFrameDeduplicator(
+    scoped_ptr<StackFrameDeduplicator> stack_frame_deduplicator) {

[petrcermak, 2016/02/26 10:21:03]

Perhaps DCHECK(stack_frame_deduplicator_ == nullptr)?

[Primiano Tucci (use gerrit), 2016/02/26 22:52:49]

Done.

+  stack_frame_deduplicator_ = std::move(stack_frame_deduplicator);
+}
+
+void MemoryDumpSessionState::SetTypeNameDeduplicator(
+    scoped_ptr<TypeNameDeduplicator> type_name_deduplicator) {
+  type_name_deduplicator_ = std::move(type_name_deduplicator);

[petrcermak, 2016/02/26 10:21:03]

ditto

[Primiano Tucci (use gerrit), 2016/02/26 22:52:50]

Done.

 }
 
 }  // namespace trace_event
 }  // namespace base