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

Unified Diff: mojo/bindings/java/src/org/chromium/mojo/bindings/Message.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/bindings/java/src/org/chromium/mojo/bindings/Message.java
diff --git a/mojo/bindings/java/src/org/chromium/mojo/bindings/Message.java b/mojo/bindings/java/src/org/chromium/mojo/bindings/Message.java
index f826a8e034c9215a8a8e6496e05c716619a32d0d..b7ec42dbcfad44d4d16fcc1f16fb1f0a05e2703d 100644
--- a/mojo/bindings/java/src/org/chromium/mojo/bindings/Message.java
+++ b/mojo/bindings/java/src/org/chromium/mojo/bindings/Message.java
@@ -17,7 +17,7 @@ import javax.annotation.Nullable;
/**
* A raw message to be sent/received from a {@link MessagePipeHandle}.
*/
-public final class Message {
+public class Message {
rmcilroy 2014/07/14 14:44:57 I think this can still be final since you are wrap
qsr 2014/07/15 12:03:16 Done.
/**
* The data of the message.
@@ -56,7 +56,7 @@ public final class Message {
result = handle.readMessage(buffer, result.getHandlesCount(),
MessagePipeHandle.ReadFlags.NONE);
if (receiver != null && result.getMojoResult() == MojoResult.OK) {
- receiver.accept(new Message(buffer, result.getHandles()));
+ receiver.accept(new MessageWithHeader(new Message(buffer, result.getHandles())));
}
return result.getMojoResult();
}

Powered by Google App Engine
This is Rietveld 408576698