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

Unified Diff: mojo/public/java/bindings/src/org/chromium/mojo/bindings/Interface.java

Issue 2277853003: Use unions for interface control messages. (Closed)
Patch Set: Created 4 years, 4 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/public/java/bindings/src/org/chromium/mojo/bindings/Interface.java
diff --git a/mojo/public/java/bindings/src/org/chromium/mojo/bindings/Interface.java b/mojo/public/java/bindings/src/org/chromium/mojo/bindings/Interface.java
index c2bbc8ebddb4127bef1020b75395dd673b8ad691..50c4de5997f46e07d6697a7b01e5ad6e229160b9 100644
--- a/mojo/public/java/bindings/src/org/chromium/mojo/bindings/Interface.java
+++ b/mojo/public/java/bindings/src/org/chromium/mojo/bindings/Interface.java
@@ -186,15 +186,16 @@ public interface Interface extends ConnectionErrorHandler, Closeable {
@Override
public void queryVersion(final Callback1<Integer> callback) {
RunMessageParams message = new RunMessageParams();
- message.reserved0 = 16;
- message.reserved1 = 0;
- message.queryVersion = new QueryVersion();
+ message.input = new RunInput();
+ message.input.setQueryVersion(new QueryVersion());
InterfaceControlMessagesHelper.sendRunMessage(getCore(), mMessageReceiver, message,
new Callback1<RunResponseMessageParams>() {
@Override
public void call(RunResponseMessageParams response) {
- mVersion = response.queryVersionResult.version;
+ if (response.output.which() == RunOutput.Tag.QueryVersionResult) {
+ mVersion = response.output.getQueryVersionResult().version;
+ }
callback.call(mVersion);
}
});
@@ -210,10 +211,9 @@ public interface Interface extends ConnectionErrorHandler, Closeable {
}
mVersion = version;
RunOrClosePipeMessageParams message = new RunOrClosePipeMessageParams();
- message.reserved0 = 16;
- message.reserved1 = 0;
- message.requireVersion = new RequireVersion();
- message.requireVersion.version = version;
+ message.input = new RunOrClosePipeInput();
+ message.input.setRequireVersion(new RequireVersion());
+ message.input.getRequireVersion().version = version;
InterfaceControlMessagesHelper.sendRunOrClosePipeMessage(
getCore(), mMessageReceiver, message);
}

Powered by Google App Engine
This is Rietveld 408576698