[mojo-edk] Fix bootstrap NodeChannel Start/ShutDown race

[mojo-edk] Fix bootstrap NodeChannel Start/ShutDown race

If the parent bootstrap signals an error very early in
NodeController startup, it can be ShutDown() before
Start() is invoked on the IO thread via
ConnectoToParentOnIOThread.

This changes Start() to gracefully handle a null
|channel_|.

BUG=610337
R=amistry@chromium.org

Committed: https://crrev.com/1febcb4dfce7928a6d4bee93888bdb88bfa18ce4
Cr-Commit-Position: refs/heads/master@{#394055}

[Ken Rockot(use gerrit already), 2016-05-17 03:34:48]

Patchset #1 (id:1) has been deleted

[Ken Rockot(use gerrit already), 2016-05-17 03:37:13]

Description was changed from

==========
[mojo-edk] Fix bootstrap NodeChannel Start/ShutDown race

If the parent bootstrap signals an error very early in
NodeController startup, it can be ShutDown() before
Start() is invoked on the IO thread.

This changes Start() to gracefully handle a null
|channel_|.

BUG=610337
R=amistry@chromium.org
==========

to

==========
[mojo-edk] Fix bootstrap NodeChannel Start/ShutDown race

If the parent bootstrap signals an error very early in
NodeController startup, it can be ShutDown() before
Start() is invoked on the IO thread via
ConnectoToParentOnIOThread.

This changes Start() to gracefully handle a null
|channel_|.

BUG=610337
R=amistry@chromium.org
==========

[Anand Mistry (off Chromium), 2016-05-17 04:00:15]

LGTM.

Trivial and safe. BTW, I can't see that bug.

[Ken Rockot(use gerrit already), 2016-05-17 04:05:37]

On 2016/05/17 at 04:00:15, amistry wrote:
> LGTM.
> 
> Trivial and safe. BTW, I can't see that bug.

Thanks - CCed on the bug so you can see it

[Ken Rockot(use gerrit already), 2016-05-17 04:05:43]

The CQ bit was checked by rockot@chromium.org

[commit-bot: I haz the power, 2016-05-17 04:06:12]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1985993002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1985993002/20001

[commit-bot: I haz the power, 2016-05-17 05:02:40]

Committed patchset #1 (id:20001)

[commit-bot: I haz the power, 2016-05-17 05:03:47]

Description was changed from

==========
[mojo-edk] Fix bootstrap NodeChannel Start/ShutDown race

If the parent bootstrap signals an error very early in
NodeController startup, it can be ShutDown() before
Start() is invoked on the IO thread via
ConnectoToParentOnIOThread.

This changes Start() to gracefully handle a null
|channel_|.

BUG=610337
R=amistry@chromium.org
==========

to

==========
[mojo-edk] Fix bootstrap NodeChannel Start/ShutDown race

If the parent bootstrap signals an error very early in
NodeController startup, it can be ShutDown() before
Start() is invoked on the IO thread via
ConnectoToParentOnIOThread.

This changes Start() to gracefully handle a null
|channel_|.

BUG=610337
R=amistry@chromium.org

Committed: https://crrev.com/1febcb4dfce7928a6d4bee93888bdb88bfa18ce4
Cr-Commit-Position: refs/heads/master@{#394055}
==========

[commit-bot: I haz the power, 2016-05-17 05:03:48]

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