diff options
| author | rdsmith@google.com <rdsmith@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-07-07 19:58:36 +0000 |
|---|---|---|
| committer | rdsmith@google.com <rdsmith@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-07-07 19:58:36 +0000 |
| commit | c8c7d8a930c1cb7af9a13c60cab10cfece2d0645 (patch) | |
| tree | 3c9f421c9c4f5f56f871d904028892afe536fa31 /net/base/cookie_monster.h | |
| parent | f8abf723d5fe8a85fade164dd030d4a13e0c7c29 (diff) | |
| download | chromium_src-c8c7d8a930c1cb7af9a13c60cab10cfece2d0645.zip chromium_src-c8c7d8a930c1cb7af9a13c60cab10cfece2d0645.tar.gz chromium_src-c8c7d8a930c1cb7af9a13c60cab10cfece2d0645.tar.bz2 | |
Fix bug in DeleteAllForURL; deletes entire store instead of just
cookies related to URL (found by inspection.) Also changed name
and semantics to more closely reflect usage of primary caller (extension
data deleter), and added test for that set of semantics.
BUG=none
TEST=Linux CookieMonsterTest.*:ParsedCookieTest.* (especially new CookieMonsterTest.DeleteAllHost)
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=51544
Review URL: http://codereview.chromium.org/2857029
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@51766 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net/base/cookie_monster.h')
| -rw-r--r-- | net/base/cookie_monster.h | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/net/base/cookie_monster.h b/net/base/cookie_monster.h index 43943a0..eadc59a 100644 --- a/net/base/cookie_monster.h +++ b/net/base/cookie_monster.h @@ -138,8 +138,11 @@ class CookieMonster : public CookieStore { // one passed into the function via |delete_after|. int DeleteAllCreatedAfter(const base::Time& delete_begin, bool sync_to_store); - // Delete all cookies that match the given URL. - int DeleteAllForURL(const GURL& url, bool sync_to_store); + // Delete all cookies that match the host of the given URL + // regardless of path. This includes all http_only and secure cookies, + // but does not include any domain cookies that may apply to this host. + // Returns the number of cookies deleted. + int DeleteAllForHost(const GURL& url); // Delete one specific cookie. bool DeleteCookie(const std::string& domain, |