Respond to Connector::Connect when a native service binary fails to launch

Respond to Connector::Connect when a native service binary fails to launch

Otherwise we hit a DCHECK in the bindings, because the Connector
pipe is still open when we drop the callback.

BUG=675019
Committed: https://crrev.com/05cb8ae729c29d4f69626ac0c59b49e2259c5f9f
Cr-Commit-Position: refs/heads/master@{#439227}

[Ken Rockot(use gerrit already), 2016-12-16 21:17:03]

The CQ bit was checked by rockot@chromium.org to run a CQ dry run

[Ken Rockot(use gerrit already), 2016-12-16 21:17:22]

rockot@chromium.org changed reviewers:
+ ben@chromium.org

[Ben Goodger (Google), 2016-12-16 21:18:05]

lgtm

[commit-bot: I haz the power, 2016-12-16 21:18:10]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Ken Rockot(use gerrit already), 2016-12-16 21:19:16]

The CQ bit was unchecked by rockot@chromium.org

[Ken Rockot(use gerrit already), 2016-12-16 21:19:17]

The CQ bit was checked by rockot@chromium.org

[commit-bot: I haz the power, 2016-12-16 21:19:52]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-16 23:13:56]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1481923157551420, "parent_rev":
"e0c5e1c9e8146b1bbcbd711b5e1c236bd97ca9f1", "commit_rev":
"87fdb3700d8a2510d971527ef76bb005c1be81b7"}

[commit-bot: I haz the power, 2016-12-16 23:14:21]

Description was changed from

==========
Respond to Connector::Connect when a native service binary fails to launch

Otherwise we hit a DCHECK in the bindings, because the Connector
pipe is still open when we drop the callback.

BUG=675019
==========

to

==========
Respond to Connector::Connect when a native service binary fails to launch

Otherwise we hit a DCHECK in the bindings, because the Connector
pipe is still open when we drop the callback.

BUG=675019

Review-Url: https://codereview.chromium.org/2581873003
==========

[commit-bot: I haz the power, 2016-12-16 23:14:22]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-16 23:19:01]

Description was changed from

==========
Respond to Connector::Connect when a native service binary fails to launch

Otherwise we hit a DCHECK in the bindings, because the Connector
pipe is still open when we drop the callback.

BUG=675019

Review-Url: https://codereview.chromium.org/2581873003
==========

to

==========
Respond to Connector::Connect when a native service binary fails to launch

Otherwise we hit a DCHECK in the bindings, because the Connector
pipe is still open when we drop the callback.

BUG=675019

Committed: https://crrev.com/05cb8ae729c29d4f69626ac0c59b49e2259c5f9f
Cr-Commit-Position: refs/heads/master@{#439227}
==========

[commit-bot: I haz the power, 2016-12-16 23:19:03]

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