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

Issue 9317095: Fix crash on startup because function returned before creating required (Closed)

Created:
8 years, 10 months ago by Roger Tawa OOO till Jul 10th
Modified:
8 years, 10 months ago
Reviewers:
Finnur
CC:
chromium-reviews
Visibility:
Public.

Description

Fix crash on startup because function returned before creating required members. Finnur fixed a similar crash about a month ago (crrev.com/116316), so I added some DCHECK()s to help prevent this in the future. BUG=110234 TEST=This may happen when ResourceBundle::InitSharedInstanceWithLocale() returns an empty string. Not sure if there are other scenarios that would cause this crash. Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=120788

Patch Set 1 #

Patch Set 2 : Sync and merge, no new changes #

Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -4 lines) Patch
M chrome/browser/chrome_browser_main.cc View 1 3 chunks +7 lines, -4 lines 0 comments Download

Messages

Total messages: 10 (0 generated)
Roger Tawa OOO till Jul 10th
8 years, 10 months ago (2012-02-03 22:00:35 UTC) #1
Finnur
This change looks fine, per se, but what I don't understand is how we get ...
8 years, 10 months ago (2012-02-06 09:52:27 UTC) #2
Finnur
Change LGTM. Feel free to answer the question after checking in.
8 years, 10 months ago (2012-02-06 10:47:15 UTC) #3
Roger Tawa OOO till Jul 10th
Thanks Finnur. I think your original fix was fine, but a new return was introduced ...
8 years, 10 months ago (2012-02-06 14:32:25 UTC) #4
Finnur
No, I think it is fine as is -- the DCHECK will take care of ...
8 years, 10 months ago (2012-02-06 14:49:37 UTC) #5
Roger Tawa OOO till Jul 10th
Cool, thanks Finnur.
8 years, 10 months ago (2012-02-06 14:59:29 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/rogerta@chromium.org/9317095/1
8 years, 10 months ago (2012-02-06 15:00:21 UTC) #7
commit-bot: I haz the power
Try job failure for 9317095-1 (retry) (retry) (retry) on mac_rel for steps "test_shell_tests, ui_tests". It's ...
8 years, 10 months ago (2012-02-06 18:37:34 UTC) #8
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/rogerta@chromium.org/9317095/7004
8 years, 10 months ago (2012-02-06 20:39:52 UTC) #9
commit-bot: I haz the power
8 years, 10 months ago (2012-02-07 00:13:41 UTC) #10
Try job failure for 9317095-7004 (retry) (previous was lost) on mac_rel for step
"browser_tests".
It's a second try, previously, steps "base_unittests, browser_tests, ui_tests"
failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

Powered by Google App Engine
This is Rietveld 408576698