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

Unified Diff: mojo/public/cpp/bindings/lib/control_message_handler.cc

Issue 2112093002: Mojo C++ bindings: Merge EncodePointers/DecodePointers into Serialize/Deserialize, respectively. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@61_array_fix
Patch Set: . Created 4 years, 6 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/cpp/bindings/lib/control_message_handler.cc
diff --git a/mojo/public/cpp/bindings/lib/control_message_handler.cc b/mojo/public/cpp/bindings/lib/control_message_handler.cc
index 1bcd87daf9ef56103b30255ec0a5ce83c2fd51ad..9f44e88f4f4c06d623d25b2f0ce78841b980e384 100644
--- a/mojo/public/cpp/bindings/lib/control_message_handler.cc
+++ b/mojo/public/cpp/bindings/lib/control_message_handler.cc
@@ -63,7 +63,6 @@ bool ControlMessageHandler::Run(Message* message,
RunResponseMessageParams_Data* response_params = nullptr;
Serialize<RunResponseMessageParamsPtr>(response_params_ptr, builder.buffer(),
&response_params, &context_);
- response_params->EncodePointers();
bool ok = responder->Accept(builder.message());
ALLOW_UNUSED_LOCAL(ok);
delete responder;
@@ -75,8 +74,6 @@ bool ControlMessageHandler::RunOrClosePipe(Message* message) {
RunOrClosePipeMessageParams_Data* params =
reinterpret_cast<RunOrClosePipeMessageParams_Data*>(
message->mutable_payload());
- params->DecodePointers();
-
RunOrClosePipeMessageParamsPtr params_ptr;
Deserialize<RunOrClosePipeMessageParamsPtr>(params, &params_ptr, &context_);
« no previous file with comments | « mojo/public/cpp/bindings/lib/bindings_internal.cc ('k') | mojo/public/cpp/bindings/lib/control_message_proxy.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698