Add bindProxy implementation without callback.

Add bindProxy implementation without callback.

The original implementation of bindProxy in Mojo connection requires a callback
in order to receive the proxy for the remote interface. This pattern is
counterintuitive and unnecessary. A new function that doesn't require a
callback has been added here.

Committed: https://crrev.com/3648b35f8ed44f505c4c62c7a63be90c66760006
Cr-Commit-Position: refs/heads/master@{#422992}

[mbrunson, 2016-10-04 21:35:59]

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

[mbrunson, 2016-10-04 21:35:59]

This is that function that was added when we were trying to implement interface
requests in JavaScript. Let me know if there are any changes that should be made
or if I should just drop this completely now.

[Ken Rockot(use gerrit already), 2016-10-04 22:36:51]

lgtm

https://codereview.chromium.org/2389303002/diff/20001/mojo/public/js/connecti...
File mojo/public/js/connection.js (right):

https://codereview.chromium.org/2389303002/diff/20001/mojo/public/js/connecti...
mojo/public/js/connection.js:104: function bindProxy2(remoteInterface) {
Let's call this getProxy. We'll probably change it eventually anyway since
Connection is a weird name for the module, but getProxy matches the C++ GetProxy
which does exactly the same thing.

https://codereview.chromium.org/2389303002/diff/20001/mojo/public/js/connecti...
mojo/public/js/connection.js:115: handle: messagePipe.handle1,
nit: Let's call this requestHandle

[mbrunson, 2016-10-04 22:55:09]

https://codereview.chromium.org/2389303002/diff/20001/mojo/public/js/connecti...
File mojo/public/js/connection.js (right):

https://codereview.chromium.org/2389303002/diff/20001/mojo/public/js/connecti...
mojo/public/js/connection.js:104: function bindProxy2(remoteInterface) {
On 2016/10/04 22:36:51, Ken Rockot wrote:
> Let's call this getProxy. We'll probably change it eventually anyway since
> Connection is a weird name for the module, but getProxy matches the C++
GetProxy
> which does exactly the same thing.

Done.

https://codereview.chromium.org/2389303002/diff/20001/mojo/public/js/connecti...
mojo/public/js/connection.js:115: handle: messagePipe.handle1,
On 2016/10/04 22:36:51, Ken Rockot wrote:
> nit: Let's call this requestHandle

Done.

[mbrunson, 2016-10-04 22:56:19]

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

[commit-bot: I haz the power, 2016-10-04 22:56:53]

Dry run: 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-10-04 23:06:46]

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

[commit-bot: I haz the power, 2016-10-04 23:06:47]

Dry run: This issue passed the CQ dry run.

[mbrunson, 2016-10-04 23:12:50]

The CQ bit was checked by mbrunson@chromium.org

[mbrunson, 2016-10-04 23:12:50]

The patchset sent to the CQ was uploaded after l-g-t-m from rockot@chromium.org
Link to the patchset: https://codereview.chromium.org/2389303002/#ps40001
(title: "Renaming")

[commit-bot: I haz the power, 2016-10-04 23:13:37]

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-10-04 23:20:10]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-10-04 23:23:11]

Description was changed from

==========
Add bindProxy implementation without callback.

The original implementation of bindProxy in Mojo connection requires a callback
in order to receive the proxy for the remote interface. This pattern is
counterintuitive and unnecessary. A new function that doesn't require a
callback has been added here.
==========

to

==========
Add bindProxy implementation without callback.

The original implementation of bindProxy in Mojo connection requires a callback
in order to receive the proxy for the remote interface. This pattern is
counterintuitive and unnecessary. A new function that doesn't require a
callback has been added here.

Committed: https://crrev.com/3648b35f8ed44f505c4c62c7a63be90c66760006
Cr-Commit-Position: refs/heads/master@{#422992}
==========

[commit-bot: I haz the power, 2016-10-04 23:23:15]

Patchset 3 (id:??) landed as
https://crrev.com/3648b35f8ed44f505c4c62c7a63be90c66760006
Cr-Commit-Position: refs/heads/master@{#422992}