Check default_container in FramePainter::UseSoloWindowHeader

Check default_container in FramePainter::UseSoloWindowHeader

I happened to catch this while debugging.

BUG=120786
TEST=Crash does not show up (hard to repro)


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=129591

[stevenjb, 2012-03-28 23:38:38]

+derat for OWNER

[Daniel Erat, 2012-03-28 23:58:30]

lgtm

http://codereview.chromium.org/9854045/diff/1/ash/wm/frame_painter.cc
File ash/wm/frame_painter.cc (right):

http://codereview.chromium.org/9854045/diff/1/ash/wm/frame_painter.cc#newcode550
ash/wm/frame_painter.cc:550: return false;  // Shutting down.
please add a link to the bug

[stevenjb, 2012-03-29 00:44:18]

http://codereview.chromium.org/9854045/diff/1/ash/wm/frame_painter.cc
File ash/wm/frame_painter.cc (right):

http://codereview.chromium.org/9854045/diff/1/ash/wm/frame_painter.cc#newcode550
ash/wm/frame_painter.cc:550: return false;  // Shutting down.
On 2012/03/28 23:58:30, Daniel Erat wrote:
> please add a link to the bug

Done.

[commit-bot: I haz the power, 2012-03-29 01:54:38]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/stevenjb@chromium.org/9854045/3001

[commit-bot: I haz the power, 2012-03-29 05:09:22]

Try job failure for 9854045-3001 (retry) on win_rel for step "browser_tests".
It's a second try, previously, step "browser_tests" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[commit-bot: I haz the power, 2012-03-29 05:10:41]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/stevenjb@chromium.org/9854045/3001

[commit-bot: I haz the power, 2012-03-29 08:32:15]

Change committed as 129591