Chromium Code Reviews| Index: mojo/public/interfaces/bindings/interface_control_messages.mojom |
| diff --git a/mojo/public/interfaces/bindings/interface_control_messages.mojom b/mojo/public/interfaces/bindings/interface_control_messages.mojom |
| index 10bd269696783dac1cab6380b6face78ca6c5e1c..775bd22f22904ea0ea864f783205e021eeeceda2 100644 |
| --- a/mojo/public/interfaces/bindings/interface_control_messages.mojom |
| +++ b/mojo/public/interfaces/bindings/interface_control_messages.mojom |
| @@ -2,7 +2,8 @@ |
| // Use of this source code is governed by a BSD-style license that can be |
| // found in the LICENSE file. |
| -[DartPackage="mojo", JavaPackage="org.chromium.mojo.bindings"] |
| +[DartPackage="mojo", |
| +JavaPackage="org.chromium.mojo.bindings"] |
| module mojo; |
| // For each message pipe representing a user-defined interface, some control |
| @@ -38,22 +39,21 @@ const uint32 kRunMessageId = 0xFFFFFFFF; |
| struct RunMessageParams { |
| // The reserved fields make the layout compatible with the RunInput union |
| // described above. |
| - uint32 reserved0; // Must be set to 16. |
| - uint32 reserved1; // Must be set to 0; |
| + uint32 reserved0; // Must be set to 16. |
|
jamesr
2016/02/29 23:40:49
two spaces before the start of a comment is fairly
vtl
2016/02/29 23:41:59
I think in general, we prefer two spaces before th
azani
2016/03/04 01:08:48
Done.
|
| + uint32 reserved1; // Must be set to 0; |
| QueryVersion query_version; |
| }; |
| struct RunResponseMessageParams { |
| // The reserved fields make the layout compatible with the RunOutput union |
| // described above. |
| - uint32 reserved0; // Must be set to 16. |
| - uint32 reserved1; // Must be set to 0. |
| + uint32 reserved0; // Must be set to 16. |
| + uint32 reserved1; // Must be set to 0. |
| QueryVersionResult query_version_result; |
| }; |
| // Queries the max supported version of the user-defined interface. |
| -struct QueryVersion { |
| -}; |
| +struct QueryVersion{}; |
|
vtl
2016/02/29 23:41:59
This seems to be broken. At the very least, there
azani
2016/03/04 01:08:48
Done.
|
| struct QueryVersionResult { |
| uint32 version; |
| }; |
| @@ -78,8 +78,8 @@ const uint32 kRunOrClosePipeMessageId = 0xFFFFFFFE; |
| struct RunOrClosePipeMessageParams { |
| // The reserved fields make the layout compatible with the RunOrClosePipeInput |
| // union described above. |
| - uint32 reserved0; // Must be set to 16. |
| - uint32 reserved1; // Must be set to 0. |
| + uint32 reserved0; // Must be set to 16. |
| + uint32 reserved1; // Must be set to 0. |
| RequireVersion require_version; |
| }; |