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

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: Refactoring common code and style fixes. 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..93510bc99d270dee542d69fc48a3c96f0aefd59c 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,46 @@ 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));
+
+ string16 name = ASCIIToUTF16("Site Data");
+ string16 browser_id;
+
+ {
+ ContainerMap apps_map;
+ apps_map[name] = new LocalDataContainer(
+ name, browser_id,
+ 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;
+ apps_map[name] = new LocalDataContainer(
+ name, browser_id,
+ 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