Reland "Re-enable the new SQLitePersistentCookieStore load strategy."
commitecd42cf5ae19e2807067ad1add9430af0d34510d
authorerikchen <erikchen@chromium.org>
Fri, 13 Mar 2015 23:49:41 +0000 (13 16:49 -0700)
committerCommit bot <commit-bot@chromium.org>
Fri, 13 Mar 2015 23:50:08 +0000 (13 23:50 +0000)
treee48be2f923aaf6b7621b6aa62224ad4d934e8b75
parenta3e011e4ffd0d629a1af75d2b1aff6b2909650cd
Reland "Re-enable the new SQLitePersistentCookieStore load strategy."

Reason for revert:
The source of the crash was another bug in SequencedWorkerPool. I just landed a fix for that:
https://codereview.chromium.org/994483005/

So I'm going to reland this CL.

Original issue's description:
> Revert of Re-enable the new SQLitePersistentCookieStore load strategy. (patchset #1 id:1 of https://codereview.chromium.org/953993002/)
>
> Reason for revert:
> This CL causes hangs during shutdown.
>
> https://code.google.com/p/chromium/issues/detail?id=465271
>
> Original issue's description:
> > Re-enable the new SQLitePersistentCookieStore load strategy.
> >
> > When I first turned on the new load strategy, it caused deadlocks during
> > shutdown. I found and fixed the bug in SequencedWorkerPool.
> >
> > BUG=448910
> >
> > Committed: https://crrev.com/043d6b497dda7a39138eca1fb9df89ee340e543b
> > Cr-Commit-Position: refs/heads/master@{#318305}
>
> TBR=erikwright@chromium.org
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=448910
>
> Committed: https://crrev.com/24ab88b28d3fcdb67a919527e67c49ba5d55ed93
> Cr-Commit-Position: refs/heads/master@{#319696}

TBR=erikwright@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=448910

Review URL: https://codereview.chromium.org/1005953003

Cr-Commit-Position: refs/heads/master@{#320605}
content/browser/net/sqlite_persistent_cookie_store.cc