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..730ac6b1d018f55ad7c3c14ddb7da59d72499427 100644
--- a/extensions/browser/sandboxed_unpacker.cc
+++ b/extensions/browser/sandboxed_unpacker.cc
@@ -684,11 +684,9 @@ void SandboxedUnpacker::ReportSuccess(
         base::TimeTicks::Now() - crx_unpack_start_time_);
   DCHECK(!temp_dir_.GetPath().empty());
 
-  // Client takes ownership of temporary directory and extension.
-  // TODO(https://crbug.com/699528): we should consider transferring the
-  // ownership of original_manifest to the client as well.
+  // Client takes ownership of temporary directory, manifest, and extension.
   client_->OnUnpackSuccess(temp_dir_.Take(), extension_root_,
-                           original_manifest.get(), extension_.get(),
+                           std::move(original_manifest), extension_.get(),
                            install_icon);
   extension_ = NULL;
 }