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

Side by Side Diff: net/base/cookie_monster.h

Issue 2857029: Fix bug in DeleteAllForURL; deletes entire store instead of just (Closed)
Patch Set: Incorporated comments from eroman and mnissler. Created 10 years, 5 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 unified diff | Download patch
« no previous file with comments | « chrome/browser/extensions/extension_data_deleter.cc ('k') | net/base/cookie_monster.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2010 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 // Brought to you by the letter D and the number 2. 5 // Brought to you by the letter D and the number 2.
6 6
7 #ifndef NET_BASE_COOKIE_MONSTER_H_ 7 #ifndef NET_BASE_COOKIE_MONSTER_H_
8 #define NET_BASE_COOKIE_MONSTER_H_ 8 #define NET_BASE_COOKIE_MONSTER_H_
9 9
10 #include <map> 10 #include <map>
(...skipping 120 matching lines...) Expand 10 before | Expand all | Expand 10 after
131 int DeleteAll(bool sync_to_store); 131 int DeleteAll(bool sync_to_store);
132 // Delete all of the cookies that have a creation_date greater than or equal 132 // Delete all of the cookies that have a creation_date greater than or equal
133 // to |delete_begin| and less than |delete_end| 133 // to |delete_begin| and less than |delete_end|
134 int DeleteAllCreatedBetween(const base::Time& delete_begin, 134 int DeleteAllCreatedBetween(const base::Time& delete_begin,
135 const base::Time& delete_end, 135 const base::Time& delete_end,
136 bool sync_to_store); 136 bool sync_to_store);
137 // Delete all of the cookies that have a creation_date more recent than the 137 // Delete all of the cookies that have a creation_date more recent than the
138 // one passed into the function via |delete_after|. 138 // one passed into the function via |delete_after|.
139 int DeleteAllCreatedAfter(const base::Time& delete_begin, bool sync_to_store); 139 int DeleteAllCreatedAfter(const base::Time& delete_begin, bool sync_to_store);
140 140
141 // Delete all cookies that match the given URL. 141 // Delete all cookies that match the host of the given URL
142 int DeleteAllForURL(const GURL& url, bool sync_to_store); 142 // regardless of path. This includes all http_only and secure cookies,
143 // but does not include any domain cookies that may apply to this host.
144 // Returns the number of cookies deleted.
145 int DeleteAllForHost(const GURL& url);
143 146
144 // Delete one specific cookie. 147 // Delete one specific cookie.
145 bool DeleteCookie(const std::string& domain, 148 bool DeleteCookie(const std::string& domain,
146 const CanonicalCookie& cookie, 149 const CanonicalCookie& cookie,
147 bool sync_to_store); 150 bool sync_to_store);
148 151
149 // Override the default list of schemes that are allowed to be set in 152 // Override the default list of schemes that are allowed to be set in
150 // this cookie store. Calling his overrides the value of 153 // this cookie store. Calling his overrides the value of
151 // "enable_file_scheme_". 154 // "enable_file_scheme_".
152 // If this this method is called, it must be called before first use of 155 // If this this method is called, it must be called before first use of
(...skipping 360 matching lines...) Expand 10 before | Expand all | Expand 10 after
513 protected: 516 protected:
514 PersistentCookieStore() { } 517 PersistentCookieStore() { }
515 518
516 private: 519 private:
517 DISALLOW_COPY_AND_ASSIGN(PersistentCookieStore); 520 DISALLOW_COPY_AND_ASSIGN(PersistentCookieStore);
518 }; 521 };
519 522
520 } // namespace net 523 } // namespace net
521 524
522 #endif // NET_BASE_COOKIE_MONSTER_H_ 525 #endif // NET_BASE_COOKIE_MONSTER_H_
OLDNEW
« no previous file with comments | « chrome/browser/extensions/extension_data_deleter.cc ('k') | net/base/cookie_monster.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698