Store in memory cookies in a hash set instead of a list.

chrome/browser/browsing_data/browsing_data_cookie_helper.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_BROWSING_DATA_BROWSING_DATA_COOKIE_HELPER_H_
 #define CHROME_BROWSER_BROWSING_DATA_BROWSING_DATA_COOKIE_HELPER_H_
 
 #include <map>
 #include <string>
 
 #include "base/basictypes.h"
 #include "base/callback.h"
 #include "base/memory/ref_counted.h"
+#include "chrome/browser/browsing_data/canonical_cookie_hash.h"
 #include "net/cookies/cookie_monster.h"
 
 class GURL;
 
 namespace net {
 class CanonicalCookie;
 class URLRequestContextGetter;
 }
 
 // This class fetches cookie information on behalf of a caller
@@ skipping 59 matching lines @@
 // that are received from a server or set via JavaScript using the method
 // AddChangedCookie.
 // Cookies are distinguished by the tuple cookie name (called cookie-name in
 // RFC 6265), cookie domain (called cookie-domain in RFC 6265), cookie path
 // (called cookie-path in RFC 6265) and host-only-flag (see RFC 6265 section
 // 5.3). Cookies with same tuple (cookie-name, cookie-domain, cookie-path,
 // host-only-flag) as cookie that are already stored, will replace the stored
 // cookies.
 class CannedBrowsingDataCookieHelper : public BrowsingDataCookieHelper {
  public:
-  typedef std::map<GURL, net::CookieList*> OriginCookieListMap;
+  typedef std::map<GURL, canonical_cookie::CookieHashSet*> OriginCookieSetMap;
 
   explicit CannedBrowsingDataCookieHelper(
       net::URLRequestContextGetter* request_context);
 
   // Return a copy of the cookie helper. Only one consumer can use the
   // StartFetching method at a time, so we need to create a copy of the helper
   // everytime we instantiate a cookies tree model for it.
   CannedBrowsingDataCookieHelper* Clone();
 
   // Adds the cookies from |cookie_list|. Current cookies that have the same
@@ skipping 27 matching lines @@
 
   // BrowsingDataCookieHelper methods.
   virtual void StartFetching(
       const net::CookieMonster::GetCookieListCallback& callback) OVERRIDE;
   virtual void DeleteCookie(const net::CanonicalCookie& cookie) OVERRIDE;
 
   // Returns the number of stored cookies.
   size_t GetCookieCount() const;
 
   // Returns the map that contains the cookie lists for all frame urls.
-  const OriginCookieListMap& origin_cookie_list_map() {
-    return origin_cookie_list_map_;
+  const OriginCookieSetMap& origin_cookie_set_map() {
+    return origin_cookie_set_map_;
   }
 
  private:
-  // Check if the cookie list contains a cookie with the same name,
+  // Check if the cookie set contains a cookie with the same name,
   // domain, and path as the newly created cookie. Delete the old cookie
   // if does.
   bool DeleteMatchingCookie(const net::CanonicalCookie& add_cookie,
-                            net::CookieList* cookie_list);
+                            canonical_cookie::CookieHashSet* cookie_set);
 
   virtual ~CannedBrowsingDataCookieHelper();
 
-  // Returns the |CookieList| for the given |origin|.
-  net::CookieList* GetCookiesFor(const GURL& origin);
+  // Returns the |CookieSet| for the given |origin|.
+  canonical_cookie::CookieHashSet* GetCookiesFor(const GURL& origin);
 
-  // Adds the |cookie| to the cookie list for the given |frame_url|.
+  // Adds the |cookie| to the cookie set for the given |frame_url|.
   void AddCookie(const GURL& frame_url,
                  const net::CanonicalCookie& cookie);
 
-  // Map that contains the cookie lists for all frame origins.
-  OriginCookieListMap origin_cookie_list_map_;
+  // Map that contains the cookie sets for all frame origins.
+  OriginCookieSetMap origin_cookie_set_map_;
 
   DISALLOW_COPY_AND_ASSIGN(CannedBrowsingDataCookieHelper);
 };
 
 #endif  // CHROME_BROWSER_BROWSING_DATA_BROWSING_DATA_COOKIE_HELPER_H_