Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(328)

Unified Diff: chrome/browser/themes/theme_syncable_service_unittest.cc

Issue 11275088: Remove implicit scoped_refptr operator T* Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 8 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/themes/theme_syncable_service_unittest.cc
diff --git a/chrome/browser/themes/theme_syncable_service_unittest.cc b/chrome/browser/themes/theme_syncable_service_unittest.cc
index 3c474c213a819d6f01be4aff465dd9df534d45e6..c00a2894f2aadf690a7b927c7e74f1f125f1d4a6 100644
--- a/chrome/browser/themes/theme_syncable_service_unittest.cc
+++ b/chrome/browser/themes/theme_syncable_service_unittest.cc
@@ -96,7 +96,7 @@ class FakeThemeService : public ThemeService {
}
virtual string GetThemeID() const OVERRIDE {
- if (theme_extension_)
+ if (theme_extension_.get())
return theme_extension_->id();
else
return "";
@@ -139,7 +139,7 @@ scoped_refptr<extensions::Extension> MakeThemeExtension(
extensions::Extension::Create(
extension_path, extensions::Extension::EXTERNAL_PREF_DOWNLOAD, source,
extensions::Extension::NO_FLAGS, &error);
- EXPECT_TRUE(extension);
+ EXPECT_TRUE(extension.get());
EXPECT_EQ("", error);
return extension;
}
@@ -187,7 +187,7 @@ class ThemeSyncableServiceTest : public testing::Test {
// find it.
theme_extension_ = MakeThemeExtension(FilePath(kExtensionFilePath),
kCustomThemeName, kCustomThemeUrl);
- service->AddExtension(theme_extension_);
+ service->AddExtension(theme_extension_.get());
ASSERT_EQ(1u, service->extensions()->size());
}
« no previous file with comments | « chrome/browser/task_manager/task_manager_unittest.cc ('k') | chrome/browser/thumbnails/thumbnail_tab_helper.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698