Chromium Code Reviews
DescriptionMerge 22814 - Fix for an "update in progress" DCHECK.
A DCHECK in SafeBrowsingService::UpdateStarted can potentially be
triggered if there is an error opening the database for reading,
or if there is no response from the server for a update request.
Both of these cases can cause our 'update_in_progress_' flag to
not be reset properly before the next update.
This CL adds:
a timeout for update responses
better handling for database errors
that properly resets the update state.
BUG=12835
(http://crbug.com/12835)
TEST=None.
Review URL: http://codereview.chromium.org/165008
TBR=paul@chromium.org
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=23281
Patch Set 1 #
Messages
Total messages: 2 (0 generated)
|
||||||||||||||||||||||||||||