Transfer DictionaryValue ownership in SandboxedUnpacker::OnUnpackSuccess()

extensions/browser/sandboxed_unpacker.cc

Index: extensions/browser/sandboxed_unpacker.cc
diff --git a/extensions/browser/sandboxed_unpacker.cc b/extensions/browser/sandboxed_unpacker.cc
index 5f36c51e44bff3140696ad2143c6f475c5777275..8c6a262c3836fecb97f5008dd9eeff6efb106e68 100644
--- a/extensions/browser/sandboxed_unpacker.cc
+++ b/extensions/browser/sandboxed_unpacker.cc
@@ -685,10 +685,8 @@ void SandboxedUnpacker::ReportSuccess(
   DCHECK(!temp_dir_.GetPath().empty());
 
   // Client takes ownership of temporary directory and extension.

[Noel Gordon, 2017/03/30 06:35:41]

Maybe this comment needs updating too?  Perhaps:

// Client takes ownership of temporary directory, manifest, and extension.

or similar.

[lazyboy, 2017/03/30 07:01:51]

Done.

-  // TODO(https://crbug.com/699528): we should consider transferring the
-  // ownership of original_manifest to the client as well.
   client_->OnUnpackSuccess(temp_dir_.Take(), extension_root_,
-                           original_manifest.get(), extension_.get(),
+                           std::move(original_manifest), extension_.get(),
                            install_icon);
   extension_ = NULL;
 }