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

Unified Diff: chrome/browser/cocoa/clear_browsing_data_controller_unittest.mm

Issue 524026: Adds "Delete Local Storage" option to "Clear Browsing Data" for Windows and G... (Closed) Base URL: http://src.chromium.org/svn/trunk/src/
Patch Set: '' Created 10 years, 11 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/cocoa/clear_browsing_data_controller_unittest.mm
===================================================================
--- chrome/browser/cocoa/clear_browsing_data_controller_unittest.mm (revision 36077)
+++ chrome/browser/cocoa/clear_browsing_data_controller_unittest.mm (working copy)
@@ -26,6 +26,7 @@
prefs->SetBoolean(prefs::kDeleteBrowsingHistory, true);
prefs->SetBoolean(prefs::kDeleteDownloadHistory, false);
prefs->SetBoolean(prefs::kDeleteCache, true);
+ prefs->SetBoolean(prefs::kDeleteLocalStorage, true);
prefs->SetBoolean(prefs::kDeleteCookies, false);
prefs->SetBoolean(prefs::kDeletePasswords, true);
prefs->SetBoolean(prefs::kDeleteFormData, false);
@@ -49,6 +50,7 @@
EXPECT_TRUE([controller_ clearBrowsingHistory]);
EXPECT_FALSE([controller_ clearDownloadHistory]);
EXPECT_TRUE([controller_ emptyCache]);
+ EXPECT_TRUE([controller_ deleteLocalStorage]);
EXPECT_FALSE([controller_ deleteCookies]);
EXPECT_TRUE([controller_ clearSavedPasswords]);
EXPECT_FALSE([controller_ clearFormData]);
@@ -60,6 +62,7 @@
// Check that the remove-mask matches the initial properties:
EXPECT_EQ(BrowsingDataRemover::REMOVE_HISTORY |
BrowsingDataRemover::REMOVE_CACHE |
+ BrowsingDataRemover::REMOVE_LOCAL_STORAGE |
BrowsingDataRemover::REMOVE_PASSWORDS,
[controller_ removeMask]);
}
@@ -69,6 +72,7 @@
[controller_ setClearBrowsingHistory:false];
[controller_ setClearDownloadHistory:true];
[controller_ setEmptyCache:false];
+ [controller_ setDeleteLocalStorage:false];
[controller_ setDeleteCookies:true];
[controller_ setClearSavedPasswords:false];
[controller_ setClearFormData:true];
@@ -84,6 +88,7 @@
[controller_ setClearBrowsingHistory:false];
[controller_ setClearDownloadHistory:false];
[controller_ setEmptyCache:false];
+ [controller_ setDeleteLocalStorage:false];
[controller_ setDeleteCookies:false];
[controller_ setClearSavedPasswords:false];
[controller_ setClearFormData:false];
@@ -105,6 +110,7 @@
// Make sure the rest of the prefs didn't change:
EXPECT_TRUE(prefs->GetBoolean(prefs::kDeleteCache));
+ EXPECT_TRUE(prefs->GetBoolean(prefs::kDeleteLocalStorage));
EXPECT_FALSE(prefs->GetBoolean(prefs::kDeleteCookies));
EXPECT_TRUE(prefs->GetBoolean(prefs::kDeletePasswords));
EXPECT_FALSE(prefs->GetBoolean(prefs::kDeleteFormData));
« no previous file with comments | « chrome/browser/cocoa/clear_browsing_data_controller.mm ('k') | chrome/browser/gtk/clear_browsing_data_dialog_gtk.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698