diff options
author | bauerb@chromium.org <bauerb@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-08-06 13:33:04 +0000 |
---|---|---|
committer | bauerb@chromium.org <bauerb@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-08-06 13:33:04 +0000 |
commit | d814a88597a01bf8249e36983f55a5f880154642 (patch) | |
tree | b38b92bb3f3dd40285f1a5d652f3c8c6609e4279 /chrome/browser/plugins/plugin_finder_unittest.cc | |
parent | bc00c1d1ae16e1cd82430320d89daaaaea5ded67 (diff) | |
download | chromium_src-d814a88597a01bf8249e36983f55a5f880154642.zip chromium_src-d814a88597a01bf8249e36983f55a5f880154642.tar.gz chromium_src-d814a88597a01bf8249e36983f55a5f880154642.tar.bz2 |
Make element removal methods in DictionaryValue and ListValue take scoped_ptr's as outparams.
TBR=pneubeck@chromium.org,scottbyer@chromium.org
BUG=263894
Review URL: https://chromiumcodereview.appspot.com/21030009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@215885 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/plugins/plugin_finder_unittest.cc')
-rw-r--r-- | chrome/browser/plugins/plugin_finder_unittest.cc | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/chrome/browser/plugins/plugin_finder_unittest.cc b/chrome/browser/plugins/plugin_finder_unittest.cc index a6317a6..5afe3d1 100644 --- a/chrome/browser/plugins/plugin_finder_unittest.cc +++ b/chrome/browser/plugins/plugin_finder_unittest.cc @@ -15,7 +15,7 @@ TEST(PluginFinderTest, JsonSyntax) { scoped_ptr<DictionaryValue> plugin_list( PluginFinder::LoadBuiltInPluginList()); ASSERT_TRUE(plugin_list.get()); - base::Value* version = NULL; + scoped_ptr<base::Value> version; ASSERT_TRUE(plugin_list->Remove("x-version", &version)); EXPECT_EQ(base::Value::TYPE_INTEGER, version->GetType()); @@ -69,5 +69,4 @@ TEST(PluginFinderTest, JsonSyntax) { << "Invalid security status \"" << status_str << "\""; } } - delete version; } |