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

Unified Diff: mojo/edk/js/tests/js_to_cpp_tests.cc

Issue 2844143002: Mojo C++ bindings: reject messages version 2 with null payload pointer. (Closed)
Patch Set: Created 3 years, 8 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/edk/js/tests/js_to_cpp_tests.cc
diff --git a/mojo/edk/js/tests/js_to_cpp_tests.cc b/mojo/edk/js/tests/js_to_cpp_tests.cc
index b6b74e31fbfdc692fc2b8ec00faef1d82c215471..4f0313e06cc2a22f0237a2ef60f95d71a4f83d1d 100644
--- a/mojo/edk/js/tests/js_to_cpp_tests.cc
+++ b/mojo/edk/js/tests/js_to_cpp_tests.cc
@@ -239,7 +239,8 @@ class CppSideConnection : public js_to_cpp::CppSide {
mishandled_messages_ += 1;
}
- void BitFlipResponse(js_to_cpp::EchoArgsListPtr list) override {
+ void BitFlipResponse(js_to_cpp::EchoArgsListPtr list,
+ js_to_cpp::NotUsedAssociatedPtrInfo not_used) override {
mishandled_messages_ += 1;
}
@@ -332,7 +333,8 @@ class BitFlipCppSideConnection : public CppSideConnection {
// js_to_cpp::CppSide:
void StartTest() override { js_side_->BitFlip(BuildSampleEchoArgs()); }
- void BitFlipResponse(js_to_cpp::EchoArgsListPtr list) override {
+ void BitFlipResponse(js_to_cpp::EchoArgsListPtr list,
+ js_to_cpp::NotUsedAssociatedPtrInfo not_used) override {
CheckCorruptedEchoArgsList(list);
}

Powered by Google App Engine
This is Rietveld 408576698