Make ValueDeserializer::Deserialize return scoped_ptr

chrome/browser/component_updater/recovery_component_installer.cc

Index: chrome/browser/component_updater/recovery_component_installer.cc
diff --git a/chrome/browser/component_updater/recovery_component_installer.cc b/chrome/browser/component_updater/recovery_component_installer.cc
index 3657d6458dc7cc6f3c6ec177e04c57e3825b0800..e038a06a19b2d76fa9ac3d95c53ce4a963ad6edd 100644
--- a/chrome/browser/component_updater/recovery_component_installer.cc
+++ b/chrome/browser/component_updater/recovery_component_installer.cc
@@ -118,10 +118,9 @@ base::CommandLine GetRecoveryInstallCommandLine(
 scoped_ptr<base::DictionaryValue> ReadManifest(const base::FilePath& manifest) {
   JSONFileValueDeserializer deserializer(manifest);
   std::string error;
-  scoped_ptr<base::Value> root(deserializer.Deserialize(NULL, &error));
+  scoped_ptr<base::Value> root = deserializer.Deserialize(NULL, &error);
   if (root.get() && root->IsType(base::Value::TYPE_DICTIONARY)) {

[Lei Zhang, 2015/10/14 16:46:14]

Is this redundant?

-    return scoped_ptr<base::DictionaryValue>(
-        static_cast<base::DictionaryValue*>(root.release()));
+    return base::DictionaryValue::From(root.Pass());
   }
   return scoped_ptr<base::DictionaryValue>();
 }