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

Unified Diff: chrome/browser/ui/cocoa/content_settings/collected_cookies_mac.mm

Issue 10536017: Refactoring CookiesTreeModel to support multiple data sources. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixed all comments by Evan. Created 8 years, 6 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/ui/cocoa/content_settings/collected_cookies_mac.mm
diff --git a/chrome/browser/ui/cocoa/content_settings/collected_cookies_mac.mm b/chrome/browser/ui/cocoa/content_settings/collected_cookies_mac.mm
index 0d516b30faf6fdf9972cb41c37d86e5067784c70..9bc2cdb965eb590ad4cc88d815593bbb95b981a6 100644
--- a/chrome/browser/ui/cocoa/content_settings/collected_cookies_mac.mm
+++ b/chrome/browser/ui/cocoa/content_settings/collected_cookies_mac.mm
@@ -383,30 +383,47 @@ void CollectedCookiesMac::OnSheetDidEnd(NSWindow* sheet) {
const LocalSharedObjectsContainer& allowed_lsos =
content_settings->allowed_local_shared_objects();
- allowedTreeModel_.reset(
- new CookiesTreeModel(allowed_lsos.cookies()->Clone(),
- allowed_lsos.databases()->Clone(),
- allowed_lsos.local_storages()->Clone(),
- allowed_lsos.session_storages()->Clone(),
- allowed_lsos.appcaches()->Clone(),
- allowed_lsos.indexed_dbs()->Clone(),
- allowed_lsos.file_systems()->Clone(),
- NULL,
- allowed_lsos.server_bound_certs()->Clone(),
- true));
+
+ {
+ ContainerMap apps_map;
+ // TODO(nasko): once the app node of the tree is no longer visible, the
+ // name won't matter and should be changed to empty string.
+ apps_map[""] = new LocalDataContainer(
+ "Site Data", "",
+ allowed_lsos.cookies()->Clone(),
+ allowed_lsos.databases()->Clone(),
+ allowed_lsos.local_storages()->Clone(),
+ allowed_lsos.session_storages()->Clone(),
+ allowed_lsos.appcaches()->Clone(),
+ allowed_lsos.indexed_dbs()->Clone(),
+ allowed_lsos.file_systems()->Clone(),
+ NULL,
+ allowed_lsos.server_bound_certs()->Clone());
+
+ allowedTreeModel_.reset(new CookiesTreeModel(apps_map, true));
+ }
+
const LocalSharedObjectsContainer& blocked_lsos =
content_settings->blocked_local_shared_objects();
- blockedTreeModel_.reset(
- new CookiesTreeModel(blocked_lsos.cookies()->Clone(),
- blocked_lsos.databases()->Clone(),
- blocked_lsos.local_storages()->Clone(),
- blocked_lsos.session_storages()->Clone(),
- blocked_lsos.appcaches()->Clone(),
- blocked_lsos.indexed_dbs()->Clone(),
- blocked_lsos.file_systems()->Clone(),
- NULL,
- blocked_lsos.server_bound_certs()->Clone(),
- true));
+
+ {
+ ContainerMap apps_map;
+ // TODO(nasko): once the app node of the tree is no longer visible, the
+ // name won't matter and should be changed to empty string.
+ apps_map[""] = new LocalDataContainer(
+ "Site Data", "",
+ blocked_lsos.cookies()->Clone(),
+ blocked_lsos.databases()->Clone(),
+ blocked_lsos.local_storages()->Clone(),
+ blocked_lsos.session_storages()->Clone(),
+ blocked_lsos.appcaches()->Clone(),
+ blocked_lsos.indexed_dbs()->Clone(),
+ blocked_lsos.file_systems()->Clone(),
+ NULL,
+ blocked_lsos.server_bound_certs()->Clone());
+
+ blockedTreeModel_.reset(new CookiesTreeModel(apps_map, true));
+ }
// Convert the model's icons from Skia to Cocoa.
std::vector<gfx::ImageSkia> skiaIcons;

Powered by Google App Engine
This is Rietveld 408576698