Index: mojo/android/javatests/src/org/chromium/mojo/bindings/InterfacesTest.java |
diff --git a/mojo/android/javatests/src/org/chromium/mojo/bindings/InterfacesTest.java b/mojo/android/javatests/src/org/chromium/mojo/bindings/InterfacesTest.java |
index 58e2351073f6f550c6bea340b96dffd195b502b6..cb4b8a2dbf7ca5fe0b303f1373c6671625948702 100644 |
--- a/mojo/android/javatests/src/org/chromium/mojo/bindings/InterfacesTest.java |
+++ b/mojo/android/javatests/src/org/chromium/mojo/bindings/InterfacesTest.java |
@@ -175,16 +175,6 @@ public class InterfacesTest extends MojoTestCase { |
super.tearDown(); |
} |
- private <I extends Interface, P extends Interface.Proxy> P newProxyOverPipe( |
- Interface.Manager<I, P> manager, I impl) { |
- Pair<MessagePipeHandle, MessagePipeHandle> handles = |
- CoreImpl.getInstance().createMessagePipe(null); |
- P proxy = manager.attachProxy(handles.first); |
- mCloseablesToClose.add(proxy); |
- manager.bind(impl, handles.second); |
- return proxy; |
- } |
- |
/** |
* Check that the given proxy receives the calls. If |impl| is not null, also check that the |
* calls are forwared to |impl|. |
@@ -241,14 +231,16 @@ public class InterfacesTest extends MojoTestCase { |
@SmallTest |
public void testProxyAndStubOverPipe() { |
MockNamedObjectImpl impl = new MockNamedObjectImpl(); |
- NamedObject.Proxy proxy = newProxyOverPipe(NamedObject.MANAGER, impl); |
+ NamedObject.Proxy proxy = |
+ BindingsTestUtils.newProxyOverPipe(NamedObject.MANAGER, impl, mCloseablesToClose); |
checkProxy(proxy, impl); |
} |
@SmallTest |
public void testFactoryOverPipe() { |
- Factory.Proxy proxy = newProxyOverPipe(Factory.MANAGER, new MockFactoryImpl()); |
+ Factory.Proxy proxy = BindingsTestUtils.newProxyOverPipe( |
+ Factory.MANAGER, new MockFactoryImpl(), mCloseablesToClose); |
Pair<NamedObject.Proxy, InterfaceRequest<NamedObject>> request = |
NamedObject.MANAGER.getInterfaceRequest(CoreImpl.getInstance()); |
mCloseablesToClose.add(request.first); |
@@ -260,7 +252,8 @@ public class InterfacesTest extends MojoTestCase { |
@SmallTest |
public void testInterfaceClosing() { |
MockFactoryImpl impl = new MockFactoryImpl(); |
- Factory.Proxy proxy = newProxyOverPipe(Factory.MANAGER, impl); |
+ Factory.Proxy proxy = |
+ BindingsTestUtils.newProxyOverPipe(Factory.MANAGER, impl, mCloseablesToClose); |
assertFalse(impl.isClosed()); |
@@ -273,7 +266,8 @@ public class InterfacesTest extends MojoTestCase { |
@SmallTest |
public void testResponse() { |
MockFactoryImpl impl = new MockFactoryImpl(); |
- Factory.Proxy proxy = newProxyOverPipe(Factory.MANAGER, impl); |
+ Factory.Proxy proxy = |
+ BindingsTestUtils.newProxyOverPipe(Factory.MANAGER, impl, mCloseablesToClose); |
Request request = new Request(); |
request.x = 42; |
Pair<MessagePipeHandle, MessagePipeHandle> handles = |