mojo: java bindigns: add factory for InterfaceRequest that takes a client

mojo: java bindigns: add factory for InterfaceRequest that takes a client

When cretaing an InterfaceRequest for an interface that have a client,
there must be a way to provide the client implementation.

R=ppi@chromium.org

Committed: https://crrev.com/14755294fdb15b90dfc0b3653277a0a65b8250de
Cr-Commit-Position: refs/heads/master@{#294593}

[ppi, 2014-09-12 13:07:56]

lgtm % nits, thanks

https://codereview.chromium.org/562133002/diff/1/mojo/public/java/bindings/sr...
File
mojo/public/java/bindings/src/org/chromium/mojo/bindings/InterfaceWithClient.java
(right):

https://codereview.chromium.org/562133002/diff/1/mojo/public/java/bindings/sr...
mojo/public/java/bindings/src/org/chromium/mojo/bindings/InterfaceWithClient.java:93:
* Construct a new |InterfaceRequest| for the interface. This method returns a
Pair where
nit: s/Construct/Constructs/ (also in the corresponding place of Interface.java,
if you could fix it while at it).

https://codereview.chromium.org/562133002/diff/1/mojo/public/java/bindings/sr...
mojo/public/java/bindings/src/org/chromium/mojo/bindings/InterfaceWithClient.java:95:
* used immediately. |client| is the implementation of the client interface.
nit: make the last sentence a @param statement

[qsr, 2014-09-12 14:16:07]

https://codereview.chromium.org/562133002/diff/1/mojo/public/java/bindings/sr...
File
mojo/public/java/bindings/src/org/chromium/mojo/bindings/InterfaceWithClient.java
(right):

https://codereview.chromium.org/562133002/diff/1/mojo/public/java/bindings/sr...
mojo/public/java/bindings/src/org/chromium/mojo/bindings/InterfaceWithClient.java:93:
* Construct a new |InterfaceRequest| for the interface. This method returns a
Pair where
On 2014/09/12 13:07:56, ppi wrote:
> nit: s/Construct/Constructs/ (also in the corresponding place of
Interface.java,
> if you could fix it while at it).

Done.

https://codereview.chromium.org/562133002/diff/1/mojo/public/java/bindings/sr...
mojo/public/java/bindings/src/org/chromium/mojo/bindings/InterfaceWithClient.java:95:
* used immediately. |client| is the implementation of the client interface.
On 2014/09/12 13:07:56, ppi wrote:
> nit: make the last sentence a @param statement

Done.

[qsr, 2014-09-12 14:16:12]

The CQ bit was checked by qsr@chromium.org

[commit-bot: I haz the power, 2014-09-12 14:21:41]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patchset/562133002/20001

[commit-bot: I haz the power, 2014-09-12 15:30:51]

Committed patchset #2 (id:20001) as 478f23a3af137f217bbf6638f86b2492dfd41efc

[commit-bot: I haz the power, 2014-09-12 15:40:16]

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