Commit 66759041 authored by dhollowa@chromium.org's avatar dhollowa@chromium.org

DCHECK(result) keeps firing in AutocompleteHistoryManager::OnWebDataServiceRequestDone()

A NULL |result| should DCHECK, but not crash.  This is a short term solution to the top-crasher, will still DCHECK for devs while a longer term fix is determined.

BUG=68783, 69833
TEST=none

Review URL: http://codereview.chromium.org/6275011

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72094 0039d316-1c4b-4281-b951-d872f2087c98
parent 3e08fbd8
......@@ -175,6 +175,14 @@ void AutocompleteHistoryManager::OnWebDataServiceRequestDone(
}
DCHECK(result);
// Returning early here if |result| is NULL. We've seen this happen on
// Linux due to NFS dismounting and causing sql failures.
// See http://crbug.com/68783.
if (!result) {
SendSuggestions(NULL);
return;
}
DCHECK(result->GetType() == AUTOFILL_VALUE_RESULT);
const WDResult<std::vector<string16> >* autofill_result =
static_cast<const WDResult<std::vector<string16> >*>(result);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment