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

Unified Diff: chrome/browser/extensions/api/declarative_content/content_rules_registry_unittest.cc

Issue 16295003: Update chrome/ to use scoped_refptr<T>::get() rather than implicit "operator T*" (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebased Created 7 years, 7 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/api/declarative_content/content_rules_registry_unittest.cc
diff --git a/chrome/browser/extensions/api/declarative_content/content_rules_registry_unittest.cc b/chrome/browser/extensions/api/declarative_content/content_rules_registry_unittest.cc
index f37f1ec29e14a4fed7473bb5a9ee606470a2ca28..6c7a7e2cd083df46c8f1c9e86fdee95e32b8ea5a 100644
--- a/chrome/browser/extensions/api/declarative_content/content_rules_registry_unittest.cc
+++ b/chrome/browser/extensions/api/declarative_content/content_rules_registry_unittest.cc
@@ -40,13 +40,13 @@ TEST_F(DeclarativeContentRulesRegistryTest, ActiveRulesDoesntGrow) {
scoped_refptr<ContentRulesRegistry> registry(
new ContentRulesRegistry(env.profile(), NULL /*ui_part*/));
- EXPECT_EQ(0u, active_rules(*registry).size());
+ EXPECT_EQ(0u, active_rules(*registry.get()).size());
content::LoadCommittedDetails load_details;
content::FrameNavigateParams navigate_params;
scoped_ptr<WebContents> tab = env.MakeTab();
registry->DidNavigateMainFrame(tab.get(), load_details, navigate_params);
- EXPECT_EQ(0u, active_rules(*registry).size());
+ EXPECT_EQ(0u, active_rules(*registry.get()).size());
// Add a rule.
linked_ptr<RulesRegistry::Rule> rule(new RulesRegistry::Rule);
@@ -73,24 +73,24 @@ TEST_F(DeclarativeContentRulesRegistryTest, ActiveRulesDoesntGrow) {
registry->AddRulesImpl(extension->id(), rules);
registry->DidNavigateMainFrame(tab.get(), load_details, navigate_params);
- EXPECT_EQ(0u, active_rules(*registry).size());
+ EXPECT_EQ(0u, active_rules(*registry.get()).size());
std::vector<std::string> css_selectors;
css_selectors.push_back("input");
registry->Apply(tab.get(), css_selectors);
- EXPECT_EQ(1u, active_rules(*registry).size());
+ EXPECT_EQ(1u, active_rules(*registry.get()).size());
// Closing the tab should erase its entry from active_rules_.
tab.reset();
- EXPECT_EQ(0u, active_rules(*registry).size());
+ EXPECT_EQ(0u, active_rules(*registry.get()).size());
tab = env.MakeTab();
registry->Apply(tab.get(), css_selectors);
- EXPECT_EQ(1u, active_rules(*registry).size());
+ EXPECT_EQ(1u, active_rules(*registry.get()).size());
// Navigating the tab should erase its entry from active_rules_ if
// it no longer matches.
registry->DidNavigateMainFrame(tab.get(), load_details, navigate_params);
- EXPECT_EQ(0u, active_rules(*registry).size());
+ EXPECT_EQ(0u, active_rules(*registry.get()).size());
}
} // namespace

Powered by Google App Engine
This is Rietveld 408576698