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

Unified Diff: chrome/browser/safe_browsing/safe_browsing_service_browsertest.cc

Issue 2396133003: Change the logic how Subesource Filter propagates activation. (Closed)
Patch Set: fix browser test Created 4 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/safe_browsing/safe_browsing_service_browsertest.cc
diff --git a/chrome/browser/safe_browsing/safe_browsing_service_browsertest.cc b/chrome/browser/safe_browsing/safe_browsing_service_browsertest.cc
index 7f03120c7117e491e86af228ddd6861b7ce7eb27..c0a86bb432356789a1f849de35ed6d4f2b20ce97 100644
--- a/chrome/browser/safe_browsing/safe_browsing_service_browsertest.cc
+++ b/chrome/browser/safe_browsing/safe_browsing_service_browsertest.cc
@@ -926,11 +926,10 @@ IN_PROC_BROWSER_TEST_F(SafeBrowsingServiceTest,
subresource_filter::ContentSubresourceFilterDriverFactory::
FromWebContents(web_contents);
- EXPECT_EQ(0U, driver_factory->activation_set().size());
+ EXPECT_EQ(0U, driver_factory->hits_set().size());
chrome::NavigateParams params(browser(), bad_url, ui::PAGE_TRANSITION_LINK);
ui_test_utils::NavigateToURL(&params);
-
- EXPECT_EQ(1U, driver_factory->activation_set().size());
+ EXPECT_EQ(1U, driver_factory->hits_set().size());
EXPECT_TRUE(got_hit_report());
}
@@ -951,11 +950,10 @@ IN_PROC_BROWSER_TEST_F(SafeBrowsingServiceTest, SocEngReportingBlacklistEmpty) {
subresource_filter::ContentSubresourceFilterDriverFactory::
FromWebContents(web_contents);
- EXPECT_EQ(0U, driver_factory->activation_set().size());
+ EXPECT_EQ(0U, driver_factory->hits_set().size());
chrome::NavigateParams params(browser(), bad_url, ui::PAGE_TRANSITION_LINK);
ui_test_utils::NavigateToURL(&params);
-
- EXPECT_EQ(0U, driver_factory->activation_set().size());
+ EXPECT_EQ(0U, driver_factory->hits_set().size());
EXPECT_TRUE(got_hit_report());
}

Powered by Google App Engine
This is Rietveld 408576698