From ad8e04ac88be37d5ccb6c2cf61f52b224dca493c Mon Sep 17 00:00:00 2001 From: "thakis@chromium.org" Date: Mon, 1 Nov 2010 04:16:27 +0000 Subject: Convert implicit scoped_refptr constructor calls to explicit ones, part 1 This CL was created automatically by this clang rewriter: http://codereview.appspot.com/2776043/ . I manually fixed a few rough spots of the rewriter output (doh1-3) and fixed all presubmit errors. BUG=28083 TEST=None Review URL: http://codereview.chromium.org/4192012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@64573 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/browser/themes/browser_theme_pack.cc | 2 +- chrome/browser/themes/browser_theme_pack_unittest.cc | 4 ++-- chrome/browser/themes/browser_theme_provider.cc | 4 ++-- 3 files changed, 5 insertions(+), 5 deletions(-) (limited to 'chrome/browser/themes') diff --git a/chrome/browser/themes/browser_theme_pack.cc b/chrome/browser/themes/browser_theme_pack.cc index 56cbdbf..c0b0081 100644 --- a/chrome/browser/themes/browser_theme_pack.cc +++ b/chrome/browser/themes/browser_theme_pack.cc @@ -368,7 +368,7 @@ BrowserThemePack* BrowserThemePack::BuildFromExtension( scoped_refptr BrowserThemePack::BuildFromDataPack( FilePath path, const std::string& expected_id) { DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); - scoped_refptr pack = new BrowserThemePack; + scoped_refptr pack(new BrowserThemePack); pack->data_pack_.reset(new base::DataPack); if (!pack->data_pack_->Load(path)) { diff --git a/chrome/browser/themes/browser_theme_pack_unittest.cc b/chrome/browser/themes/browser_theme_pack_unittest.cc index 5da6acf..5d4be22 100644 --- a/chrome/browser/themes/browser_theme_pack_unittest.cc +++ b/chrome/browser/themes/browser_theme_pack_unittest.cc @@ -410,8 +410,8 @@ TEST_F(BrowserThemePackTest, CanBuildAndReadPack) { ASSERT_TRUE(extension.get()); ASSERT_EQ("", error); - scoped_refptr pack = - BrowserThemePack::BuildFromExtension(extension.get()); + scoped_refptr pack( + BrowserThemePack::BuildFromExtension(extension.get())); ASSERT_TRUE(pack.get()); ASSERT_TRUE(pack->WriteToDisk(file)); VerifyStarGazing(pack.get()); diff --git a/chrome/browser/themes/browser_theme_provider.cc b/chrome/browser/themes/browser_theme_provider.cc index 134d954..ae40eef 100644 --- a/chrome/browser/themes/browser_theme_provider.cc +++ b/chrome/browser/themes/browser_theme_provider.cc @@ -602,8 +602,8 @@ void BrowserThemeProvider::SaveThemeID(const std::string& id) { } void BrowserThemeProvider::BuildFromExtension(const Extension* extension) { - scoped_refptr pack = - BrowserThemePack::BuildFromExtension(extension); + scoped_refptr pack( + BrowserThemePack::BuildFromExtension(extension)); if (!pack.get()) { // TODO(erg): We've failed to install the theme; perhaps we should tell the // user? http://crbug.com/34780 -- cgit v1.1