diff options
| author | erikchen <erikchen@chromium.org> | 2015-03-09 11:37:39 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-03-09 18:38:12 +0000 |
| commit | 24ab88b28d3fcdb67a919527e67c49ba5d55ed93 (patch) | |
| tree | c36e816e90b74f1e7323d61064eff8058de7fd46 /content/browser/net/sqlite_persistent_cookie_store.cc | |
| parent | 07bd36abf92fd84bffd6f2d8feb4532dfb77901a (diff) | |
| download | chromium_src-24ab88b28d3fcdb67a919527e67c49ba5d55ed93.zip chromium_src-24ab88b28d3fcdb67a919527e67c49ba5d55ed93.tar.gz chromium_src-24ab88b28d3fcdb67a919527e67c49ba5d55ed93.tar.bz2 | |
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
Review URL: https://codereview.chromium.org/992723002
Cr-Commit-Position: refs/heads/master@{#319696}
Diffstat (limited to 'content/browser/net/sqlite_persistent_cookie_store.cc')
| -rw-r--r-- | content/browser/net/sqlite_persistent_cookie_store.cc | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/content/browser/net/sqlite_persistent_cookie_store.cc b/content/browser/net/sqlite_persistent_cookie_store.cc index 6332b2b..75c89fd 100644 --- a/content/browser/net/sqlite_persistent_cookie_store.cc +++ b/content/browser/net/sqlite_persistent_cookie_store.cc @@ -50,7 +50,10 @@ namespace { // The persistent cookie store is loaded into memory on eTLD at a time. This // variable controls the delay between loading eTLDs, so as to not overload the // CPU or I/O with these low priority requests immediately after start up. -const int kLoadDelayMilliseconds = 200; +// TODO(erikchen): Investigate why setting this value to 200 causes hangs on +// shut down. +// http://crbug.com/448910 +const int kLoadDelayMilliseconds = 0; } // namespace |