Always set the error handler closure of Binding<> after binding to a message pipe.

Always set the error handler closure of Binding<> after binding to a message pipe.

This change only modifies users of Binding<>, and not the bindings code
itself.  The intention is to bring the behaviour of Binding<> closer to
that of InterfacePtr<>, which already has this restriction.

BUG=569694
Committed: https://crrev.com/d8ee03d1531318ede1c06ce7b2d3bff129eb8863
Cr-Commit-Position: refs/heads/master@{#365710}

[Anand Mistry (off Chromium), 2015-12-16 00:40:36]

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

[Anand Mistry (off Chromium), 2015-12-16 00:42:12]

amistry@chromium.org changed reviewers:
+ jam@chromium.org

[Anand Mistry (off Chromium), 2015-12-16 00:42:13]

Change is trivial, to several different directories. Figured I'd just get one
stamp instead of asking for several.

[commit-bot: I haz the power, 2015-12-16 00:45:07]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1525033004/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1525033004/1

[commit-bot: I haz the power, 2015-12-16 02:53:01]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-12-16 02:53:01]

Dry run: Try jobs failed on following builders:
  android_arm64_dbg_recipe on tryserver.chromium.linux (JOB_TIMED_OUT, no build
URL)

[jam, 2015-12-16 18:38:56]

lgtm

[Anand Mistry (off Chromium), 2015-12-16 22:16:28]

The CQ bit was checked by amistry@chromium.org

[commit-bot: I haz the power, 2015-12-16 22:18:47]

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

[commit-bot: I haz the power, 2015-12-16 22:26:08]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-12-16 22:26:09]

Try jobs failed on following builders:
  android_arm64_dbg_recipe on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_arm6...)
  android_chromium_gn_compile_dbg on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_chro...)
  android_chromium_gn_compile_rel on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_chro...)

[Anand Mistry (off Chromium), 2015-12-16 22:44:16]

The CQ bit was checked by amistry@chromium.org

[Anand Mistry (off Chromium), 2015-12-16 22:44:16]

The patchset sent to the CQ was uploaded after l-g-t-m from jam@chromium.org
Link to the patchset: https://codereview.chromium.org/1525033004/#ps20001
(title: "Rebase")

[commit-bot: I haz the power, 2015-12-16 22:44:56]

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

[commit-bot: I haz the power, 2015-12-17 03:23:25]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-12-17 03:24:17]

Description was changed from

==========
Always set the error handler closure of Binding<> after binding to a message
pipe.

This change only modifies users of Binding<>, and not the bindings code
itself.  The intention is to bring the behaviour of Binding<> closer to
that of InterfacePtr<>, which already has this restriction.

BUG=569694
==========

to

==========
Always set the error handler closure of Binding<> after binding to a message
pipe.

This change only modifies users of Binding<>, and not the bindings code
itself.  The intention is to bring the behaviour of Binding<> closer to
that of InterfacePtr<>, which already has this restriction.

BUG=569694

Committed: https://crrev.com/d8ee03d1531318ede1c06ce7b2d3bff129eb8863
Cr-Commit-Position: refs/heads/master@{#365710}
==========

[commit-bot: I haz the power, 2015-12-17 03:24:18]

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