Make ValueDeserializer::Deserialize return scoped_ptr

chrome/browser/component_updater/component_installers_unittest.cc

Index: chrome/browser/component_updater/component_installers_unittest.cc
diff --git a/chrome/browser/component_updater/component_installers_unittest.cc b/chrome/browser/component_updater/component_installers_unittest.cc
index 10c5834824c0de15de08aad657a23c7c132f6633..e6630636171e76629c8ddb00599e214cd1291506 100644
--- a/chrome/browser/component_updater/component_installers_unittest.cc
+++ b/chrome/browser/component_updater/component_installers_unittest.cc
@@ -75,7 +75,7 @@ TEST(ComponentInstallerTest, PepperFlashCheck) {
   JSONFileValueDeserializer deserializer(manifest);
   std::string error;
   scoped_ptr<base::DictionaryValue> root(static_cast<base::DictionaryValue*>(
-      deserializer.Deserialize(NULL, &error)));
+      deserializer.Deserialize(NULL, &error).release()));

[Lei Zhang, 2015/10/12 22:42:21]

Can we provide some easier way to convert the return value to a
scoped_ptr<base::DictionaryValue>?

We also need to do the same in:
chrome/browser/extensions/api/developer_private/extension_info_generator_unittest.cc
chrome/browser/ui/cocoa/extensions/extension_install_prompt_test_utils.mm
chrome/browser/themes/browser_theme_pack_unittest.cc
chrome/browser/extensions/user_script_listener_unittest.cc
chrome/browser/extensions/extension_icon_manager_unittest.cc
chrome/browser/extensions/extension_action_icon_factory_unittest.cc

and it may also make the code prettier in:
chrome/browser/extensions/external_pref_loader.cc
chrome/browser/extensions/extension_service_unittest.cc

   ASSERT_TRUE(root);
   ASSERT_TRUE(root->IsType(base::Value::TYPE_DICTIONARY));