Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(649)

Issue 1124033003: Omnibox - Fix Rare HQP Crash (Closed)

Created:
5 years, 7 months ago by Mark P
Modified:
5 years, 7 months ago
Reviewers:
Peter Kasting
CC:
chromium-reviews, James Su
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Omnibox - Fix Rare HQP Crash In the days of yore, there was an HQP crash: https://code.google.com/p/chromium/issues/detail?id=359270 I put a band-aid on the crash: https://codereview.chromium.org/222783006 Later, I fixed the root cause: https://codereview.chromium.org/255423002 Or at least I thought I did and close that days-of-yore bug. Apparently this didn't entirely fix the problem. (See the linked bug below.) I do not know what kind of situation can cause this crash any longer. (As I said, I thought I fixed the root cause.) But it's still appearing. I haven't been able to get anyone to help me look into the minidumps after more than a month of hunting. So I'll fix this crash the same as I did before. This change basically restores the band-aid fix from earlier. It'll prevent the bad information about what offsets to highlight from crashing the browser. It also expands on the DCHECK message in hopes that it makes it more likely that someone files a bug upon seeing it. BUG=464926

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+23 lines, -12 lines) Patch
M chrome/browser/autocomplete/history_quick_provider.cc View 1 chunk +23 lines, -12 lines 0 comments Download

Messages

Total messages: 4 (1 generated)
Mark P
Please take a look, thanks.
5 years, 7 months ago (2015-05-11 21:20:56 UTC) #2
Peter Kasting
If you need to figure out when something bad is happening, change the DCHECK to ...
5 years, 7 months ago (2015-05-11 22:09:00 UTC) #3
Mark P
5 years, 7 months ago (2015-05-13 20:07:46 UTC) #4
I'm abandoning this patch in favor of
https://codereview.chromium.org/1133533003/

--mark

Powered by Google App Engine
This is Rietveld 408576698