Comment #6 on issue 24163 by bugdro...@chromium.org: VisitedLinkMaster
reads from disk on UI thread
http://code.google.com/p/chromium/issues/detail?id=24163
The following revision refers to this bug:
http://src.chromium.org/viewvc/chrome?view=rev&revision=35991
------------------------------------------------------------------------
r35991 | t...@chromium.org | 2010-01-11 19:39:53 -0800 (Mon, 11 Jan 2010) |
12 lines
Changed paths:
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/browser_main.cc?r1=35991&r2=35990
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/profile.cc?r1=35991&r2=35990
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/profile.h?r1=35991&r2=35990
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/visitedlink_master.cc?r1=35991&r2=35990
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/visitedlink_master.h?r1=35991&r2=35990
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/test/testing_profile.h?r1=35991&r2=35990
Take 2: Preload the visited link db on the file thread if
the file exists. Otherwise, just load like normal on the UI thread.
This failed before because the browser may have shutdown before the
posted task ran. When the posted task finally runs, it tried to
use the profile, but the profile was already deleted. Make a small
change to VisitedLinkMaster so GetDatabaseFileName no longer depends
on the profile.
BUG=24163
Review URL: http://codereview.chromium.org/507047
------------------------------------------------------------------------
--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings
--
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
