Add a Binding class to manager the binding of a message pipe.

Add a Binding class to manager the binding of a message pipe.

R=ppi@chromium.org

Committed: https://chromium.googlesource.com/external/mojo/+/091c678187cd1cd2a3480ede0d4b9b5ce92ace99

[ppi, 2015-11-17 12:38:38]

lgtm % nits

https://codereview.chromium.org/1450373002/diff/1/mojo/public/java/bindings/s...
File mojo/public/java/bindings/src/org/chromium/mojo/bindings/Interface.java
(right):

https://codereview.chromium.org/1450373002/diff/1/mojo/public/java/bindings/s...
mojo/public/java/bindings/src/org/chromium/mojo/bindings/Interface.java:311: *
The |Binding| objects enables managing the bindings from an implementation to a
message pipe.
either "objects enable" or "object enables", I'd drop first three words and
start with "Enables".

https://codereview.chromium.org/1450373002/diff/1/mojo/public/java/bindings/s...
mojo/public/java/bindings/src/org/chromium/mojo/bindings/Interface.java:315: *
Unbind the implementation from the message pipe. Returns the handle that will be
owned by
Unbinds

https://codereview.chromium.org/1450373002/diff/1/mojo/public/java/bindings/s...
mojo/public/java/bindings/src/org/chromium/mojo/bindings/Interface.java:321: *
Register an error handler on the binding. It will be called if an error happen
on the
Registers

https://codereview.chromium.org/1450373002/diff/1/mojo/public/java/bindings/s...
mojo/public/java/bindings/src/org/chromium/mojo/bindings/Interface.java:321: *
Register an error handler on the binding. It will be called if an error happen
on the
happens

https://codereview.chromium.org/1450373002/diff/1/mojo/public/java/bindings/s...
mojo/public/java/bindings/src/org/chromium/mojo/bindings/Interface.java:400:
final DelegatingConnectionErrorHandler handlers =
s/handlers/delegatingHandler/ ? Plural makes it look as if it was a container.

[qsr, 2015-11-17 12:49:54]

https://codereview.chromium.org/1450373002/diff/1/mojo/public/java/bindings/s...
File mojo/public/java/bindings/src/org/chromium/mojo/bindings/Interface.java
(right):

https://codereview.chromium.org/1450373002/diff/1/mojo/public/java/bindings/s...
mojo/public/java/bindings/src/org/chromium/mojo/bindings/Interface.java:311: *
The |Binding| objects enables managing the bindings from an implementation to a
message pipe.
On 2015/11/17 12:38:38, ppi wrote:
> either "objects enable" or "object enables", I'd drop first three words and
> start with "Enables".

Done.

https://codereview.chromium.org/1450373002/diff/1/mojo/public/java/bindings/s...
mojo/public/java/bindings/src/org/chromium/mojo/bindings/Interface.java:315: *
Unbind the implementation from the message pipe. Returns the handle that will be
owned by
On 2015/11/17 12:38:38, ppi wrote:
> Unbinds

Done.

https://codereview.chromium.org/1450373002/diff/1/mojo/public/java/bindings/s...
mojo/public/java/bindings/src/org/chromium/mojo/bindings/Interface.java:321: *
Register an error handler on the binding. It will be called if an error happen
on the
On 2015/11/17 12:38:38, ppi wrote:
> happens

Done.

https://codereview.chromium.org/1450373002/diff/1/mojo/public/java/bindings/s...
mojo/public/java/bindings/src/org/chromium/mojo/bindings/Interface.java:321: *
Register an error handler on the binding. It will be called if an error happen
on the
On 2015/11/17 12:38:38, ppi wrote:
> Registers

Done.

https://codereview.chromium.org/1450373002/diff/1/mojo/public/java/bindings/s...
mojo/public/java/bindings/src/org/chromium/mojo/bindings/Interface.java:400:
final DelegatingConnectionErrorHandler handlers =
On 2015/11/17 12:38:38, ppi wrote:
> s/handlers/delegatingHandler/ ? Plural makes it look as if it was a container.

Done.

[qsr, 2015-11-17 13:27:58]

Committed patchset #2 (id:20001) manually as
091c678187cd1cd2a3480ede0d4b9b5ce92ace99 (presubmit successful).