From c8c7d8a930c1cb7af9a13c60cab10cfece2d0645 Mon Sep 17 00:00:00 2001 From: "rdsmith@google.com" Date: Wed, 7 Jul 2010 19:58:36 +0000 Subject: 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 --- chrome/browser/extensions/extension_data_deleter.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'chrome/browser/extensions') diff --git a/chrome/browser/extensions/extension_data_deleter.cc b/chrome/browser/extensions/extension_data_deleter.cc index 1a9f84e..dec78eb 100644 --- a/chrome/browser/extensions/extension_data_deleter.cc +++ b/chrome/browser/extensions/extension_data_deleter.cc @@ -44,7 +44,7 @@ void ExtensionDataDeleter::DeleteCookiesOnIOThread() { net::CookieMonster* cookie_monster = extension_request_context_->GetCookieStore()->GetCookieMonster(); if (cookie_monster) - cookie_monster->DeleteAllForURL(extension_url_, true); + cookie_monster->DeleteAllForHost(extension_url_); } void ExtensionDataDeleter::DeleteDatabaseOnFileThread() { -- cgit v1.1