Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(586)

Unified Diff: mojo/android/javatests/src/org/chromium/mojo/bindings/ConnectorTest.java

Issue 371603003: Adding a router class to handle messages that expect responses. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Refactor header validation. Created 6 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: mojo/android/javatests/src/org/chromium/mojo/bindings/ConnectorTest.java
diff --git a/mojo/android/javatests/src/org/chromium/mojo/bindings/ConnectorTest.java b/mojo/android/javatests/src/org/chromium/mojo/bindings/ConnectorTest.java
index 57f2550a6941ca08ed5f768a632b5422acedde8a..b2e5d9af6c1e43ec617551db6a395f0012d94b7e 100644
--- a/mojo/android/javatests/src/org/chromium/mojo/bindings/ConnectorTest.java
+++ b/mojo/android/javatests/src/org/chromium/mojo/bindings/ConnectorTest.java
@@ -31,7 +31,7 @@ public class ConnectorTest extends MojoTestCase {
private MessagePipeHandle mHandle;
private Connector mConnector;
- private Message mTestMessage;
+ private MessageWithHeader mTestMessage;
private RecordingMessageReceiver mReceiver;
private CapturingErrorHandler mErrorHandler;
@@ -50,7 +50,8 @@ public class ConnectorTest extends MojoTestCase {
mErrorHandler = new CapturingErrorHandler();
mConnector.setErrorHandler(mErrorHandler);
mConnector.start();
- mTestMessage = new Message(TestUtils.newRandomBuffer(DATA_LENGTH), new ArrayList<Handle>());
+ mTestMessage = new MessageWithHeader(
+ new Message(TestUtils.newRandomBuffer(DATA_LENGTH), new ArrayList<Handle>()));
assertNull(mErrorHandler.exception);
assertEquals(0, mReceiver.messages.size());
}
@@ -77,7 +78,7 @@ public class ConnectorTest extends MojoTestCase {
MessagePipeHandle.ReadFlags.NONE);
assertEquals(MojoResult.OK, result.getMojoResult());
assertEquals(DATA_LENGTH, result.getMessageSize());
- assertEquals(mTestMessage.buffer, received);
+ assertEquals(mTestMessage.getMessage().buffer, received);
}
/**
@@ -85,14 +86,14 @@ public class ConnectorTest extends MojoTestCase {
*/
@SmallTest
public void testReceivingMessage() {
- mHandle.writeMessage(mTestMessage.buffer, new ArrayList<Handle>(),
+ mHandle.writeMessage(mTestMessage.getMessage().buffer, new ArrayList<Handle>(),
MessagePipeHandle.WriteFlags.NONE);
nativeRunLoop(RUN_LOOP_TIMEOUT_MS);
assertNull(mErrorHandler.exception);
assertEquals(1, mReceiver.messages.size());
- Message received = mReceiver.messages.get(0);
- assertEquals(0, received.handles.size());
- assertEquals(mTestMessage.buffer, received.buffer);
+ MessageWithHeader received = mReceiver.messages.get(0);
+ assertEquals(0, received.getMessage().handles.size());
+ assertEquals(mTestMessage.getMessage().buffer, received.getMessage().buffer);
}
/**

Powered by Google App Engine
This is Rietveld 408576698