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

Unified Diff: chrome/browser/browsing_data/browsing_data_remover_browsertest.cc

Issue 2664253006: Clears out external protocol data when cookies and site data is cleared. (Closed)
Patch Set: a Created 3 years, 10 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/browsing_data/browsing_data_remover_browsertest.cc
diff --git a/chrome/browser/browsing_data/browsing_data_remover_browsertest.cc b/chrome/browser/browsing_data/browsing_data_remover_browsertest.cc
index bb4b12a12e613fed106455072ef3a34521aaa3d7..69a1b303ce94188adc17a21c35ec4ccc596f49d6 100644
--- a/chrome/browser/browsing_data/browsing_data_remover_browsertest.cc
+++ b/chrome/browser/browsing_data/browsing_data_remover_browsertest.cc
@@ -16,6 +16,7 @@
#include "chrome/browser/browsing_data/browsing_data_remover_test_util.h"
#include "chrome/browser/browsing_data/cache_counter.h"
#include "chrome/browser/chrome_notification_types.h"
+#include "chrome/browser/external_protocol/external_protocol_handler.h"
#include "chrome/browser/profiles/profile.h"
#include "chrome/browser/ui/browser.h"
#include "chrome/browser/ui/tabs/tab_strip_model.h"
@@ -274,6 +275,20 @@ IN_PROC_BROWSER_TEST_F(BrowsingDataRemoverBrowserTest, Cache) {
EXPECT_EQ(0, GetCacheSize());
}
+IN_PROC_BROWSER_TEST_F(BrowsingDataRemoverBrowserTest,
+ ExternalProtocolHandlerPrefs) {
+ Profile* profile = browser()->profile();
+ base::DictionaryValue prefs;
+ prefs.SetBoolean("tel", true);
+ profile->GetPrefs()->Set(prefs::kExcludedSchemes, prefs);
+ ExternalProtocolHandler::BlockState block_state =
+ ExternalProtocolHandler::GetBlockState("tel", profile);
+ ASSERT_EQ(ExternalProtocolHandler::BLOCK, block_state);
+ RemoveAndWait(BrowsingDataRemover::REMOVE_SITE_DATA);
+ block_state = ExternalProtocolHandler::GetBlockState("tel", profile);
+ ASSERT_EQ(ExternalProtocolHandler::UNKNOWN, block_state);
+}
+
// Verify that TransportSecurityState data is cleared for REMOVE_CACHE.
IN_PROC_BROWSER_TEST_F(BrowsingDataRemoverTransportSecurityStateBrowserTest,
ClearTransportSecurityState) {

Powered by Google App Engine
This is Rietveld 408576698