diff options
| author | rdsmith@google.com <rdsmith@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-10-18 17:34:28 +0000 |
|---|---|---|
| committer | rdsmith@google.com <rdsmith@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-10-18 17:34:28 +0000 |
| commit | 9c2addffef686c093b78e50770efae3c199ea3ff (patch) | |
| tree | 5b7bbe82a1c1aed66ea72411dc4d78fd42bb1a1f /chrome/browser/instant | |
| parent | 70d2cf46df412888fd1f1fdb1535c2bc80cbf58c (diff) | |
| download | chromium_src-9c2addffef686c093b78e50770efae3c199ea3ff.zip chromium_src-9c2addffef686c093b78e50770efae3c199ea3ff.tar.gz chromium_src-9c2addffef686c093b78e50770efae3c199ea3ff.tar.bz2 | |
Don't rely on "access_time < Time::Now()" to indicate "all cookies".
The interface to CookieMonster::GarbageCollectDeleteList() was
signalling "delete all cookies in list" by specifying "delete all
cookies in list with last access time before Now()". This was failing
to work properly when the system clock wasn't updated frequently
enough (== flaky test) and is also vulnerable to changes in systems
clocks. That semantic is now explicitly signalled by a null time.
BUG=58197
TEST=net_unittests CookieMonsterTest.* on Win/Linux and CookieMonsterTest.TestHostGarbageCollection (flaky test) repeatedly on Windows.
Review URL: http://codereview.chromium.org/3780004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62943 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/instant')
0 files changed, 0 insertions, 0 deletions