Chromecast: moves all service startup code to PreMainMessageLoopRun.

Chromecast: moves all service startup code to PreMainMessageLoopRun.

On Android, CastService::Start is never called (!). This is because
Android uses a Java-based UI message loop after
BrowserMainRunner::Initialize is completed, and BrowserMainRunner:Run is
never called (nor should it be).

See ChromeBrowserMainParts for similar handling of the Android case.

R=lcwu@chromium.org,byungchul@chromium.org
BUG=internal b/18934906

Committed: https://crrev.com/ec6d20ecb9586de310cf524ec1cfa1c0111a9427
Cr-Commit-Position: refs/heads/master@{#318136}

[lcwu1, 2015-02-25 21:48:27]

https://codereview.chromium.org/953283002/diff/1/chromecast/browser/cast_brow...
File chromecast/browser/cast_browser_main_parts.cc (right):

https://codereview.chromium.org/953283002/diff/1/chromecast/browser/cast_brow...
chromecast/browser/cast_browser_main_parts.cc:285:
cast_browser_process_.reset();
Where does the tear-down of the cast objects take place for Android?

[gunsch, 2015-02-25 21:58:57]

https://codereview.chromium.org/953283002/diff/1/chromecast/browser/cast_brow...
File chromecast/browser/cast_browser_main_parts.cc (right):

https://codereview.chromium.org/953283002/diff/1/chromecast/browser/cast_brow...
chromecast/browser/cast_browser_main_parts.cc:285:
cast_browser_process_.reset();
On 2015/02/25 21:48:27, lcwu1 wrote:
> Where does the tear-down of the cast objects take place for Android?

It doesn't. For a similar comment, see:
https://code.google.com/p/chromium/codesearch#chromium/src/chrome/browser/chr...

[lcwu1, 2015-02-25 22:21:20]

lgtm

[gunsch, 2015-02-25 22:25:58]

The CQ bit was checked by gunsch@chromium.org

[commit-bot: I haz the power, 2015-02-25 22:26:58]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/953283002/1

[commit-bot: I haz the power, 2015-02-25 23:10:27]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-02-25 23:12:01]

Patchset 1 (id:??) landed as
https://crrev.com/ec6d20ecb9586de310cf524ec1cfa1c0111a9427
Cr-Commit-Position: refs/heads/master@{#318136}