minidump: Change the ownership model

minidump/minidump_thread_writer.cc

Index: minidump/minidump_thread_writer.cc
diff --git a/minidump/minidump_thread_writer.cc b/minidump/minidump_thread_writer.cc
index 1dc3bc2812844668de1a370fed1c7b4d1030a7c3..a316da4ea0a71a5df99ab0b9a78d0aec71c4fb1d 100644
--- a/minidump/minidump_thread_writer.cc
+++ b/minidump/minidump_thread_writer.cc
@@ -34,16 +34,17 @@ const MINIDUMP_THREAD* MinidumpThreadWriter::MinidumpThread() const {
   return &thread_;
 }
 
-void MinidumpThreadWriter::SetStack(MinidumpMemoryWriter* stack) {
+void MinidumpThreadWriter::SetStack(scoped_ptr<MinidumpMemoryWriter> stack) {
   DCHECK_EQ(state(), kStateMutable);
 
-  stack_ = stack;
+  stack_ = stack.Pass();
 }
 
-void MinidumpThreadWriter::SetContext(MinidumpContextWriter* context) {
+void MinidumpThreadWriter::SetContext(
+    scoped_ptr<MinidumpContextWriter> context) {
   DCHECK_EQ(state(), kStateMutable);
 
-  context_ = context;
+  context_ = context.Pass();
 }
 
 bool MinidumpThreadWriter::Freeze() {
@@ -78,9 +79,9 @@ std::vector<internal::MinidumpWritable*> MinidumpThreadWriter::Children() {
 
   std::vector<MinidumpWritable*> children;
   if (stack_) {
-    children.push_back(stack_);
+    children.push_back(stack_.get());
   }
-  children.push_back(context_);
+  children.push_back(context_.get());
 
   return children;
 }
@@ -112,17 +113,18 @@ void MinidumpThreadListWriter::SetMemoryListWriter(
   memory_list_writer_ = memory_list_writer;
 }
 
-void MinidumpThreadListWriter::AddThread(MinidumpThreadWriter* thread) {
+void MinidumpThreadListWriter::AddThread(
+    scoped_ptr<MinidumpThreadWriter> thread) {
   DCHECK_EQ(state(), kStateMutable);
 
-  threads_.push_back(thread);
-
   if (memory_list_writer_) {
     MinidumpMemoryWriter* stack = thread->Stack();
     if (stack) {
       memory_list_writer_->AddExtraMemory(stack);
     }
   }
+
+  threads_.push_back(thread.release());
 }
 
 bool MinidumpThreadListWriter::Freeze() {