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

Unified Diff: chrome/browser/extensions/script_badge_controller_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/extensions/script_badge_controller_unittest.cc
diff --git a/chrome/browser/extensions/script_badge_controller_unittest.cc b/chrome/browser/extensions/script_badge_controller_unittest.cc
index 6a6075fd63521a97e5f009252fb8d9cbefcad50f..940033a259f4cdfdd2a1c3b29c11470b9e010cfd 100644
--- a/chrome/browser/extensions/script_badge_controller_unittest.cc
+++ b/chrome/browser/extensions/script_badge_controller_unittest.cc
@@ -79,7 +79,7 @@ class ScriptBadgeControllerTest : public ChromeRenderViewHostTestHarness {
.Set("page_action", DictionaryBuilder()
.Set("default_title", "Hello")))
.Build();
- extension_service_->AddExtension(extension);
+ extension_service_->AddExtension(extension.get());
return extension;
}
@@ -140,7 +140,7 @@ TEST_F(ScriptBadgeControllerTest, ExecutionMakesBadgeVisible) {
GURL(""),
val);
EXPECT_THAT(script_badge_controller_->GetCurrentActions(),
- testing::ElementsAre(GetScriptBadge(*extension)));
+ testing::ElementsAre(GetScriptBadge(*extension.get())));
EXPECT_THAT(location_bar_updated.events, testing::Gt(0));
};
@@ -170,7 +170,7 @@ TEST_F(ScriptBadgeControllerTest, FragmentNavigation) {
val);
EXPECT_THAT(script_badge_controller_->GetCurrentActions(),
- testing::ElementsAre(GetScriptBadge(*extension)));
+ testing::ElementsAre(GetScriptBadge(*extension.get())));
EXPECT_EQ(1, location_bar_updated.events);
}
@@ -186,7 +186,7 @@ TEST_F(ScriptBadgeControllerTest, FragmentNavigation) {
NavigateAndCommit(GURL("http://www.google.com#hash"));
EXPECT_THAT(script_badge_controller_->GetCurrentActions(),
- testing::ElementsAre(GetScriptBadge(*extension)));
+ testing::ElementsAre(GetScriptBadge(*extension.get())));
EXPECT_EQ(0, location_bar_updated.events);
}
@@ -219,7 +219,7 @@ TEST_F(ScriptBadgeControllerTest, GetAttentionMakesBadgeVisible) {
.Set("permissions", ListBuilder()
.Append("tabs")))
.Build();
- extension_service_->AddExtension(extension);
+ extension_service_->AddExtension(extension.get());
// Establish a page id.
NavigateAndCommit(GURL("http://www.google.com"));
@@ -240,7 +240,7 @@ TEST_F(ScriptBadgeControllerTest, GetAttentionMakesBadgeVisible) {
script_badge_controller_->GetAttentionFor(extension->id());
EXPECT_THAT(script_badge_controller_->GetCurrentActions(),
- testing::ElementsAre(GetScriptBadge(*extension)));
+ testing::ElementsAre(GetScriptBadge(*extension.get())));
EXPECT_THAT(initial_badge_display.events, testing::Gt(0));
CountingNotificationObserver subsequent_get_attention_call;
@@ -253,7 +253,7 @@ TEST_F(ScriptBadgeControllerTest, GetAttentionMakesBadgeVisible) {
script_badge_controller_->GetAttentionFor(extension->id());
EXPECT_THAT(script_badge_controller_->GetCurrentActions(),
- testing::ElementsAre(GetScriptBadge(*extension)));
+ testing::ElementsAre(GetScriptBadge(*extension.get())));
EXPECT_EQ(0, subsequent_get_attention_call.events);
};
« no previous file with comments | « chrome/browser/extensions/permissions_updater.cc ('k') | chrome/browser/extensions/script_bubble_controller_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698