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

Issues Search
    My Issues | Starred     Open | Closed | All

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

Issue 2418813002: [Reland] Refactoring of SBER preference usage (Closed)
Patch Set: Sync 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
« no previous file with comments | « chrome/browser/safe_browsing/safe_browsing_blocking_page_test.cc ('k') | chrome/browser/safe_browsing/safe_browsing_service.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/safe_browsing/safe_browsing_blocking_page_unittest.cc
diff --git a/chrome/browser/safe_browsing/safe_browsing_blocking_page_unittest.cc b/chrome/browser/safe_browsing/safe_browsing_blocking_page_unittest.cc
index 64ba51d4b76785f8dc940be8f2d96c0d9eaed1da..e2dfe45a74c1483bd4dec3243fb1cf00fbd71aa1 100644
--- a/chrome/browser/safe_browsing/safe_browsing_blocking_page_unittest.cc
+++ b/chrome/browser/safe_browsing/safe_browsing_blocking_page_unittest.cc
@@ -11,9 +11,9 @@
#include "chrome/browser/safe_browsing/test_safe_browsing_service.h"
#include "chrome/browser/safe_browsing/threat_details.h"
#include "chrome/browser/safe_browsing/ui_manager.h"
-#include "chrome/common/pref_names.h"
#include "chrome/test/base/chrome_render_view_host_test_harness.h"
#include "components/prefs/pref_service.h"
+#include "components/safe_browsing_db/safe_browsing_prefs.h"
#include "content/public/browser/interstitial_page.h"
#include "content/public/browser/navigation_entry.h"
#include "content/public/browser/render_process_host.h"
@@ -231,8 +231,7 @@ TEST_F(SafeBrowsingBlockingPageTest, MalwarePageDontProceed) {
// Enable malware details.
Profile* profile = Profile::FromBrowserContext(
web_contents()->GetBrowserContext());
- profile->GetPrefs()->SetBoolean(
- prefs::kSafeBrowsingExtendedReportingEnabled, true);
+ profile->GetPrefs()->SetBoolean(GetExtendedReportingPrefName(), true);
// Start a load.
controller().LoadURL(GURL(kBadURL), content::Referrer(),
@@ -266,8 +265,7 @@ TEST_F(SafeBrowsingBlockingPageTest, MalwarePageProceed) {
// Enable malware reports.
Profile* profile = Profile::FromBrowserContext(
web_contents()->GetBrowserContext());
- profile->GetPrefs()->SetBoolean(
- prefs::kSafeBrowsingExtendedReportingEnabled, true);
+ profile->GetPrefs()->SetBoolean(GetExtendedReportingPrefName(), true);
// Start a load.
controller().LoadURL(GURL(kBadURL), content::Referrer(),
@@ -300,8 +298,7 @@ TEST_F(SafeBrowsingBlockingPageTest, PageWithMalwareResourceDontProceed) {
// Enable malware reports.
Profile* profile = Profile::FromBrowserContext(
web_contents()->GetBrowserContext());
- profile->GetPrefs()->SetBoolean(
- prefs::kSafeBrowsingExtendedReportingEnabled, true);
+ profile->GetPrefs()->SetBoolean(GetExtendedReportingPrefName(), true);
// Navigate somewhere.
Navigate(kGoogleURL, 0, true);
@@ -336,8 +333,7 @@ TEST_F(SafeBrowsingBlockingPageTest, PageWithMalwareResourceProceed) {
// Enable malware reports.
Profile* profile = Profile::FromBrowserContext(
web_contents()->GetBrowserContext());
- profile->GetPrefs()->SetBoolean(
- prefs::kSafeBrowsingExtendedReportingEnabled, true);
+ profile->GetPrefs()->SetBoolean(GetExtendedReportingPrefName(), true);
// Navigate somewhere.
Navigate(kGoodURL, 0, true);
@@ -370,8 +366,7 @@ TEST_F(SafeBrowsingBlockingPageTest,
// Enable malware reports.
Profile* profile = Profile::FromBrowserContext(
web_contents()->GetBrowserContext());
- profile->GetPrefs()->SetBoolean(
- prefs::kSafeBrowsingExtendedReportingEnabled, true);
+ profile->GetPrefs()->SetBoolean(GetExtendedReportingPrefName(), true);
// Navigate somewhere.
Navigate(kGoogleURL, 0, true);
@@ -412,8 +407,7 @@ TEST_F(SafeBrowsingBlockingPageTest,
// Enable malware reports.
Profile* profile = Profile::FromBrowserContext(
web_contents()->GetBrowserContext());
- profile->GetPrefs()->SetBoolean(
- prefs::kSafeBrowsingExtendedReportingEnabled, true);
+ profile->GetPrefs()->SetBoolean(GetExtendedReportingPrefName(), true);
// Navigate somewhere.
Navigate(kGoogleURL, 0, true);
@@ -469,8 +463,7 @@ TEST_F(SafeBrowsingBlockingPageTest, PageWithMultipleMalwareResourceProceed) {
// Enable malware reports.
Profile* profile = Profile::FromBrowserContext(
web_contents()->GetBrowserContext());
- profile->GetPrefs()->SetBoolean(
- prefs::kSafeBrowsingExtendedReportingEnabled, true);
+ profile->GetPrefs()->SetBoolean(GetExtendedReportingPrefName(), true);
// Navigate somewhere else.
Navigate(kGoodURL, 0, true);
@@ -521,8 +514,7 @@ TEST_F(SafeBrowsingBlockingPageTest, NavigatingBackAndForth) {
// Enable malware reports.
Profile* profile = Profile::FromBrowserContext(
web_contents()->GetBrowserContext());
- profile->GetPrefs()->SetBoolean(
- prefs::kSafeBrowsingExtendedReportingEnabled, true);
+ profile->GetPrefs()->SetBoolean(GetExtendedReportingPrefName(), true);
// Navigate somewhere.
Navigate(kGoodURL, 0, true);
@@ -573,8 +565,7 @@ TEST_F(SafeBrowsingBlockingPageTest, ProceedThenDontProceed) {
// Enable malware reports.
Profile* profile = Profile::FromBrowserContext(
web_contents()->GetBrowserContext());
- profile->GetPrefs()->SetBoolean(
- prefs::kSafeBrowsingExtendedReportingEnabled, true);
+ profile->GetPrefs()->SetBoolean(GetExtendedReportingPrefName(), true);
// Start a load.
controller().LoadURL(GURL(kBadURL), content::Referrer(),
@@ -609,8 +600,7 @@ TEST_F(SafeBrowsingBlockingPageTest, MalwareReportsDisabled) {
// Disable malware reports.
Profile* profile = Profile::FromBrowserContext(
web_contents()->GetBrowserContext());
- profile->GetPrefs()->SetBoolean(
- prefs::kSafeBrowsingExtendedReportingEnabled, false);
+ profile->GetPrefs()->SetBoolean(GetExtendedReportingPrefName(), false);
// Start a load.
controller().LoadURL(GURL(kBadURL), content::Referrer(),
@@ -644,8 +634,7 @@ TEST_F(SafeBrowsingBlockingPageTest, MalwareReportsToggling) {
// Disable malware reports.
Profile* profile = Profile::FromBrowserContext(
web_contents()->GetBrowserContext());
- profile->GetPrefs()->SetBoolean(
- prefs::kSafeBrowsingExtendedReportingEnabled, false);
+ profile->GetPrefs()->SetBoolean(GetExtendedReportingPrefName(), false);
// Start a load.
controller().LoadURL(GURL(kBadURL), content::Referrer(),
@@ -659,20 +648,17 @@ TEST_F(SafeBrowsingBlockingPageTest, MalwareReportsToggling) {
base::RunLoop().RunUntilIdle();
- EXPECT_FALSE(profile->GetPrefs()->GetBoolean(
- prefs::kSafeBrowsingExtendedReportingEnabled));
+ EXPECT_FALSE(IsExtendedReportingEnabled(*profile->GetPrefs()));
// Simulate the user check the report agreement checkbox.
sb_interstitial->controller()->SetReportingPreference(true);
- EXPECT_TRUE(profile->GetPrefs()->GetBoolean(
- prefs::kSafeBrowsingExtendedReportingEnabled));
+ EXPECT_TRUE(IsExtendedReportingEnabled(*profile->GetPrefs()));
// Simulate the user uncheck the report agreement checkbox.
sb_interstitial->controller()->SetReportingPreference(false);
- EXPECT_FALSE(profile->GetPrefs()->GetBoolean(
- prefs::kSafeBrowsingExtendedReportingEnabled));
+ EXPECT_FALSE(IsExtendedReportingEnabled(*profile->GetPrefs()));
}
// Test that extended reporting option is not shown on blocking an HTTPS main
@@ -681,8 +667,7 @@ TEST_F(SafeBrowsingBlockingPageTest, ExtendedReportingNotShownOnSecurePage) {
// Enable malware details.
Profile* profile = Profile::FromBrowserContext(
web_contents()->GetBrowserContext());
- profile->GetPrefs()->SetBoolean(
- prefs::kSafeBrowsingExtendedReportingEnabled, true);
+ profile->GetPrefs()->SetBoolean(GetExtendedReportingPrefName(), true);
// Start a load.
controller().LoadURL(GURL(kBadHTTPSURL), content::Referrer(),
@@ -715,8 +700,7 @@ TEST_F(SafeBrowsingBlockingPageTest,
// Enable malware details.
Profile* profile = Profile::FromBrowserContext(
web_contents()->GetBrowserContext());
- profile->GetPrefs()->SetBoolean(
- prefs::kSafeBrowsingExtendedReportingEnabled, true);
+ profile->GetPrefs()->SetBoolean(GetExtendedReportingPrefName(), true);
// Commit a load.
content::WebContentsTester::For(web_contents())
@@ -750,8 +734,7 @@ TEST_F(SafeBrowsingBlockingPageTest,
// Enable malware details.
Profile* profile = Profile::FromBrowserContext(
web_contents()->GetBrowserContext());
- profile->GetPrefs()->SetBoolean(
- prefs::kSafeBrowsingExtendedReportingEnabled, true);
+ profile->GetPrefs()->SetBoolean(GetExtendedReportingPrefName(), true);
// Commit a load.
content::WebContentsTester::For(web_contents())
@@ -785,8 +768,7 @@ TEST_F(SafeBrowsingBlockingPageTest,
// Enable malware details.
Profile* profile = Profile::FromBrowserContext(
web_contents()->GetBrowserContext());
- profile->GetPrefs()->SetBoolean(
- prefs::kSafeBrowsingExtendedReportingEnabled, true);
+ profile->GetPrefs()->SetBoolean(GetExtendedReportingPrefName(), true);
// Commit a load.
content::WebContentsTester::For(web_contents())
@@ -821,8 +803,7 @@ TEST_F(SafeBrowsingBlockingPageTest,
// Enable malware details.
Profile* profile = Profile::FromBrowserContext(
web_contents()->GetBrowserContext());
- profile->GetPrefs()->SetBoolean(
- prefs::kSafeBrowsingExtendedReportingEnabled, true);
+ profile->GetPrefs()->SetBoolean(GetExtendedReportingPrefName(), true);
// Commit a load.
content::WebContentsTester::For(web_contents())
« no previous file with comments | « chrome/browser/safe_browsing/safe_browsing_blocking_page_test.cc ('k') | chrome/browser/safe_browsing/safe_browsing_service.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698