Migrate cookies for the time epoch change.

Migrate cookies for the time epoch change.

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=25040

[brettw, 2009-09-01 00:00:53]

I did not see something that would cause your cookies to all get deleted more
than once (which was reported at some point). Under the old code, existing
cookies would all get deleted, but new cookies would be OK.

[Evan Martin, 2009-09-01 00:04:14]

LGTM

http://codereview.chromium.org/183021/diff/4/3001
File chrome/common/net/cookie_monster_sqlite.cc (right):

http://codereview.chromium.org/183021/diff/4/3001#newcode279
Line 279: static const int kCompatibleVersionNumber = 3;
Can you doc this too?  (I checked the code and it seems correct to leave it at
3, but it'd be nice if the comments made it so I didn't need to check.)

[Dean McNamee, 2009-09-02 16:01:09]

I didn't mind the plan of just dropping cookies.  If the expire jumps
back in time, they will just be garbage collected eventually.

On Tue, Sep 1, 2009 at 2:04 AM, <evan@chromium.org> wrote:
> LGTM
>
>
> http://codereview.chromium.org/183021/diff/4/3001
> File chrome/common/net/cookie_monster_sqlite.cc (right):
>
> http://codereview.chromium.org/183021/diff/4/3001#newcode279
> Line 279: static const int kCompatibleVersionNumber =3D 3;
> Can you doc this too? =A0(I checked the code and it seems correct to leav=
e
> it at 3, but it'd be nice if the comments made it so I didn't need to
> check.)
>
> http://codereview.chromium.org/183021
>