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

Unified Diff: chrome/browser/content_settings/local_shared_objects_container.cc

Issue 454623003: Store in memory cookies in a hash set instead of a list. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix total ordering. Created 6 years, 4 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
« no previous file with comments | « chrome/browser/browsing_data/canonical_cookie_hash.cc ('k') | chrome/chrome_browser.gypi » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/content_settings/local_shared_objects_container.cc
diff --git a/chrome/browser/content_settings/local_shared_objects_container.cc b/chrome/browser/content_settings/local_shared_objects_container.cc
index 2b5a8bdef63591d6aefda1235691811bd4bc71bf..51b3d1b31dfb0e68be30bb7b6b309cde2a0b9228 100644
--- a/chrome/browser/content_settings/local_shared_objects_container.cc
+++ b/chrome/browser/content_settings/local_shared_objects_container.cc
@@ -12,6 +12,7 @@
#include "chrome/browser/browsing_data/browsing_data_indexed_db_helper.h"
#include "chrome/browser/browsing_data/browsing_data_local_storage_helper.h"
#include "chrome/browser/browsing_data/browsing_data_service_worker_helper.h"
+#include "chrome/browser/browsing_data/canonical_cookie_hash.h"
#include "chrome/browser/browsing_data/cookies_tree_model.h"
#include "chrome/browser/profiles/profile.h"
#include "content/public/browser/storage_partition.h"
@@ -86,16 +87,15 @@ size_t LocalSharedObjectsContainer::GetObjectCountForDomain(
// to be a third party regarding the domain of the provided |origin|.
// E.g. if the origin is "http://foo.com" then all cookies with domain foo.com,
// a.foo.com, b.a.foo.com or *.foo.com will be counted.
- typedef CannedBrowsingDataCookieHelper::OriginCookieListMap
- OriginCookieListMap;
- const OriginCookieListMap& origin_cookies_list_map =
- cookies()->origin_cookie_list_map();
- for (OriginCookieListMap::const_iterator it =
- origin_cookies_list_map.begin();
- it != origin_cookies_list_map.end();
- ++it) {
- const net::CookieList* cookie_list = it->second;
- for (net::CookieList::const_iterator cookie = cookie_list->begin();
+ typedef CannedBrowsingDataCookieHelper::OriginCookieSetMap OriginCookieSetMap;
+ const OriginCookieSetMap& origin_cookies_set_map =
+ cookies()->origin_cookie_set_map();
+ for (OriginCookieSetMap::const_iterator it = origin_cookies_set_map.begin();
+ it != origin_cookies_set_map.end();
+ ++it) {
+ const canonical_cookie::CookieHashSet* cookie_list = it->second;
+ for (canonical_cookie::CookieHashSet::const_iterator cookie =
+ cookie_list->begin();
cookie != cookie_list->end();
++cookie) {
// Strip leading '.'s.
« no previous file with comments | « chrome/browser/browsing_data/canonical_cookie_hash.cc ('k') | chrome/chrome_browser.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698