Commit 0f1cfccc authored by Mihai Sardarescu's avatar Mihai Sardarescu Committed by Commit Bot

[unified-consent] Turn off Safe Browsing Extended Reporting on clearing primary account.

This CL turns off Safe Browsing Extended Reporting when the user clears
the primary account.

Bug: 865526
Change-Id: I92f42f1ae1e24e0948db00848e17cd6b61c2c4e3
Reviewed-on: https://chromium-review.googlesource.com/1146573Reviewed-by: default avatarThomas Tangl <tangltom@chromium.org>
Commit-Queue: Mihai Sardarescu <msarda@chromium.org>
Cr-Commit-Position: refs/heads/master@{#577141}
parent 4767e8db
......@@ -97,6 +97,7 @@ void UnifiedConsentService::OnPrimaryAccountCleared(
// services.
pref_service_->SetBoolean(prefs::kUrlKeyedAnonymizedDataCollectionEnabled,
false);
service_client_->SetSafeBrowsingExtendedReportingEnabled(false);
service_client_->SetSpellCheckEnabled(false);
switch (GetMigrationState()) {
......
......@@ -250,13 +250,13 @@ TEST_F(UnifiedConsentServiceTest, ClearPrimaryAccountDisablesSomeServices) {
EXPECT_FALSE(pref_service_.GetBoolean(
prefs::kUrlKeyedAnonymizedDataCollectionEnabled));
EXPECT_FALSE(spell_check_enabled_);
EXPECT_FALSE(safe_browsing_extended_reporting_enabled_);
// Consent is not revoked for the following services.
EXPECT_TRUE(alternate_error_pages_enabled_);
EXPECT_TRUE(metrics_reporting_enabled_);
EXPECT_TRUE(network_predictions_enabled_);
EXPECT_TRUE(safe_browsing_enabled_);
EXPECT_TRUE(safe_browsing_extended_reporting_enabled_);
EXPECT_TRUE(search_suggest_enabled_);
}
#endif // !defined(OS_CHROMEOS)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment