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

Side by Side Diff: chrome/browser/safe_browsing/safe_browsing_service_browsertest.cc

Issue 2396133003: Change the logic how Subesource Filter propagates activation. (Closed)
Patch Set: fix tests 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 unified diff | Download patch
« no previous file with comments | « no previous file | components/subresource_filter/content/browser/content_subresource_filter_driver_factory.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 // 4 //
5 // This test creates a safebrowsing service using test safebrowsing database 5 // This test creates a safebrowsing service using test safebrowsing database
6 // and a test protocol manager. It is used to test logics in safebrowsing 6 // and a test protocol manager. It is used to test logics in safebrowsing
7 // service. 7 // service.
8 8
9 #include "chrome/browser/safe_browsing/safe_browsing_service.h" 9 #include "chrome/browser/safe_browsing/safe_browsing_service.h"
10 10
(...skipping 908 matching lines...) Expand 10 before | Expand all | Expand 10 after
919 SetupResponseForUrl(bad_url, malware_full_hash); 919 SetupResponseForUrl(bad_url, malware_full_hash);
920 920
921 EXPECT_CALL(observer_, OnSafeBrowsingHit(IsUnsafeResourceFor(bad_url))) 921 EXPECT_CALL(observer_, OnSafeBrowsingHit(IsUnsafeResourceFor(bad_url)))
922 .Times(1); 922 .Times(1);
923 content::WebContents* web_contents = 923 content::WebContents* web_contents =
924 browser()->tab_strip_model()->GetActiveWebContents(); 924 browser()->tab_strip_model()->GetActiveWebContents();
925 subresource_filter::ContentSubresourceFilterDriverFactory* driver_factory = 925 subresource_filter::ContentSubresourceFilterDriverFactory* driver_factory =
926 subresource_filter::ContentSubresourceFilterDriverFactory:: 926 subresource_filter::ContentSubresourceFilterDriverFactory::
927 FromWebContents(web_contents); 927 FromWebContents(web_contents);
928 928
929 EXPECT_EQ(0U, driver_factory->activation_set().size()); 929 EXPECT_EQ(0U,
930 driver_factory->safe_browsing_blacklisted_patterns_set().size());
930 chrome::NavigateParams params(browser(), bad_url, ui::PAGE_TRANSITION_LINK); 931 chrome::NavigateParams params(browser(), bad_url, ui::PAGE_TRANSITION_LINK);
931 ui_test_utils::NavigateToURL(&params); 932 ui_test_utils::NavigateToURL(&params);
932 933 EXPECT_EQ(1U,
933 EXPECT_EQ(1U, driver_factory->activation_set().size()); 934 driver_factory->safe_browsing_blacklisted_patterns_set().size());
934 EXPECT_TRUE(got_hit_report()); 935 EXPECT_TRUE(got_hit_report());
935 } 936 }
936 937
937 IN_PROC_BROWSER_TEST_F(SafeBrowsingServiceTest, SocEngReportingBlacklistEmpty) { 938 IN_PROC_BROWSER_TEST_F(SafeBrowsingServiceTest, SocEngReportingBlacklistEmpty) {
938 // Tests that URLS which doesn't belong to the SOCIAL_ENGINEERING_ADS threat 939 // Tests that URLS which doesn't belong to the SOCIAL_ENGINEERING_ADS threat
939 // type aren't seen by the Subresource Filter. 940 // type aren't seen by the Subresource Filter.
941 subresource_filter::testing::ScopedSubresourceFilterFeatureToggle
942 scoped_feature_toggle(
943 base::FeatureList::OVERRIDE_ENABLE_FEATURE,
944 subresource_filter::kActivationStateEnabled,
945 subresource_filter::kActivationScopeNoSites,
946 subresource_filter::kActivationListSocialEngineeringAdsInterstitial);
947
940 GURL bad_url = embedded_test_server()->GetURL(kMalwarePage); 948 GURL bad_url = embedded_test_server()->GetURL(kMalwarePage);
941 949
942 SBFullHashResult malware_full_hash; 950 SBFullHashResult malware_full_hash;
943 GenUrlFullHashResult(bad_url, MALWARE, &malware_full_hash); 951 GenUrlFullHashResult(bad_url, MALWARE, &malware_full_hash);
944 SetupResponseForUrl(bad_url, malware_full_hash); 952 SetupResponseForUrl(bad_url, malware_full_hash);
945 953
946 EXPECT_CALL(observer_, OnSafeBrowsingHit(IsUnsafeResourceFor(bad_url))) 954 EXPECT_CALL(observer_, OnSafeBrowsingHit(IsUnsafeResourceFor(bad_url)))
947 .Times(1); 955 .Times(1);
948 content::WebContents* web_contents = 956 content::WebContents* web_contents =
949 browser()->tab_strip_model()->GetActiveWebContents(); 957 browser()->tab_strip_model()->GetActiveWebContents();
950 subresource_filter::ContentSubresourceFilterDriverFactory* driver_factory = 958 subresource_filter::ContentSubresourceFilterDriverFactory* driver_factory =
951 subresource_filter::ContentSubresourceFilterDriverFactory:: 959 subresource_filter::ContentSubresourceFilterDriverFactory::
952 FromWebContents(web_contents); 960 FromWebContents(web_contents);
953 961
954 EXPECT_EQ(0U, driver_factory->activation_set().size()); 962 EXPECT_EQ(0U,
963 driver_factory->safe_browsing_blacklisted_patterns_set().size());
955 chrome::NavigateParams params(browser(), bad_url, ui::PAGE_TRANSITION_LINK); 964 chrome::NavigateParams params(browser(), bad_url, ui::PAGE_TRANSITION_LINK);
956 ui_test_utils::NavigateToURL(&params); 965 ui_test_utils::NavigateToURL(&params);
957 966 EXPECT_EQ(0U,
958 EXPECT_EQ(0U, driver_factory->activation_set().size()); 967 driver_factory->safe_browsing_blacklisted_patterns_set().size());
959 EXPECT_TRUE(got_hit_report()); 968 EXPECT_TRUE(got_hit_report());
960 } 969 }
961 970
962 IN_PROC_BROWSER_TEST_F(SafeBrowsingServiceTest, 971 IN_PROC_BROWSER_TEST_F(SafeBrowsingServiceTest,
963 SubResourceHitWithMainFrameReferrer) { 972 SubResourceHitWithMainFrameReferrer) {
964 GURL first_url = embedded_test_server()->GetURL(kEmptyPage); 973 GURL first_url = embedded_test_server()->GetURL(kEmptyPage);
965 GURL second_url = embedded_test_server()->GetURL(kMalwarePage); 974 GURL second_url = embedded_test_server()->GetURL(kMalwarePage);
966 GURL bad_url = embedded_test_server()->GetURL(kMalwareImg); 975 GURL bad_url = embedded_test_server()->GetURL(kMalwareImg);
967 976
968 SBFullHashResult malware_full_hash; 977 SBFullHashResult malware_full_hash;
(...skipping 797 matching lines...) Expand 10 before | Expand all | Expand 10 after
1766 content::Source<SafeBrowsingDatabaseManager>( 1775 content::Source<SafeBrowsingDatabaseManager>(
1767 sb_factory_->test_safe_browsing_service()->database_manager().get())); 1776 sb_factory_->test_safe_browsing_service()->database_manager().get()));
1768 BrowserThread::PostTask( 1777 BrowserThread::PostTask(
1769 BrowserThread::IO, FROM_HERE, 1778 BrowserThread::IO, FROM_HERE,
1770 base::Bind(&SafeBrowsingDatabaseManagerCookieTest::ForceUpdate, 1779 base::Bind(&SafeBrowsingDatabaseManagerCookieTest::ForceUpdate,
1771 base::Unretained(this))); 1780 base::Unretained(this)));
1772 observer.Wait(); 1781 observer.Wait();
1773 } 1782 }
1774 1783
1775 } // namespace safe_browsing 1784 } // namespace safe_browsing
OLDNEW
« no previous file with comments | « no previous file | components/subresource_filter/content/browser/content_subresource_filter_driver_factory.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698