[Ash] Always show the root window

Root window wasn't shown in browser_test.

BUG=115967
TEST=none

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

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=134821

[sky, 2012-05-01 15:43:27]

http://codereview.chromium.org/10273023/diff/1/chrome/browser/chrome_browser_...
File chrome/browser/chrome_browser_main.cc (right):

http://codereview.chromium.org/10273023/diff/1/chrome/browser/chrome_browser_...
chrome/browser/chrome_browser_main.cc:1898: #if defined(USE_ASH)
If you need to move Show here, can it now be moved to
ChromeBrowserMainExtraPartsAsh::PreMainMessageLoopRun, or maybe
MainMessageLoopRun?

[oshima, 2012-05-01 16:25:33]

http://codereview.chromium.org/10273023/diff/1/chrome/browser/chrome_browser_...
File chrome/browser/chrome_browser_main.cc (right):

http://codereview.chromium.org/10273023/diff/1/chrome/browser/chrome_browser_...
chrome/browser/chrome_browser_main.cc:1898: #if defined(USE_ASH)
On 2012/05/01 15:43:27, sky wrote:
> If you need to move Show here, can it now be moved to
> ChromeBrowserMainExtraPartsAsh::PreMainMessageLoopRun, or maybe
> MainMessageLoopRun?

Done.

[sky, 2012-05-01 17:04:20]

Nice, LGTM

[commit-bot: I haz the power, 2012-05-01 17:14:32]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/oshima@chromium.org/10273023/1003

[commit-bot: I haz the power, 2012-05-01 18:59:10]

Change committed as 134740

[oshima, 2012-05-01 20:55:30]

On 2012/05/01 18:59:10, I haz the power (commit-bot) wrote:
> Change committed as 134740

Looks like PreMessageLoopStart happens before PreProfileInit, therefore this
broke the tree :(.

I move it to PreProfileInit. Can you take another look?

[commit-bot: I haz the power, 2012-05-01 21:28:18]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/oshima@chromium.org/10273023/7006

[commit-bot: I haz the power, 2012-05-01 22:37:08]

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

[commit-bot: I haz the power, 2012-05-01 22:41:29]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/oshima@chromium.org/10273023/7006

[commit-bot: I haz the power, 2012-05-01 22:53:48]

Try job failure for 10273023-7006 on win_rel for step "update".
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

Step "update" is always a major failure.
Look at the try server FAQ for more details.