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

Unified Diff: third_party/mojo/src/mojo/public/cpp/bindings/lib/message_builder.cc

Issue 1127293003: Update mojo sdk to rev f84766d3b6420b7cf6a113d9d65d73cb5fe18d90 (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: formatting Created 5 years, 7 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: third_party/mojo/src/mojo/public/cpp/bindings/lib/message_builder.cc
diff --git a/third_party/mojo/src/mojo/public/cpp/bindings/lib/message_builder.cc b/third_party/mojo/src/mojo/public/cpp/bindings/lib/message_builder.cc
index 837ac81c8513db38c72741bc3b415aa85419da1d..003aba5a00a08bb621a7f89de7f07b544bcafc26 100644
--- a/third_party/mojo/src/mojo/public/cpp/bindings/lib/message_builder.cc
+++ b/third_party/mojo/src/mojo/public/cpp/bindings/lib/message_builder.cc
@@ -19,9 +19,7 @@ MessageBuilder::MessageBuilder(uint32_t name, size_t payload_size)
: buf_(sizeof(MessageHeader) + payload_size) {
MessageHeader* header;
Allocate(&buf_, &header);
- // TODO(yzshen): In order to work with other bindings which still interprets
- // the |version| field as |num_fields|, set it to version 2 for now.
- header->version = 2;
+ header->version = 0;
header->name = name;
}
@@ -43,9 +41,7 @@ MessageWithRequestIDBuilder::MessageWithRequestIDBuilder(uint32_t name,
: MessageBuilder(sizeof(MessageHeaderWithRequestID) + payload_size) {
MessageHeaderWithRequestID* header;
Allocate(&buf_, &header);
- // TODO(yzshen): In order to work with other bindings which still interprets
- // the |version| field as |num_fields|, set it to version 3 for now.
- header->version = 3;
+ header->version = 1;
header->name = name;
header->flags = flags;
header->request_id = request_id;

Powered by Google App Engine
This is Rietveld 408576698