Mojo C++ bindings: reject messages version 2 with null payload pointer.

mojo/edk/js/tests/js_to_cpp.mojom

Index: mojo/edk/js/tests/js_to_cpp.mojom
diff --git a/mojo/edk/js/tests/js_to_cpp.mojom b/mojo/edk/js/tests/js_to_cpp.mojom
index 688b22b3dea8321a9d402535f5983bebae8b2cb5..60ca57c275f3b94df910cbafd36ffd5ddef54057 100644
--- a/mojo/edk/js/tests/js_to_cpp.mojom
+++ b/mojo/edk/js/tests/js_to_cpp.mojom
@@ -28,6 +28,8 @@ struct EchoArgsList {
   EchoArgs? item;
 };
 
+interface NotUsed {};

[Tom Sepez, 2017/04/27 16:10:36]

nit: maybe call it AssociatedForTesting, as it is used, just only for testing
...

[yzshen1, 2017/04/27 17:15:33]

Good point, done. 

+
 // Note: For messages which control test flow, pick numbers that are unlikely
 // to be hit as a result of our deliberate corruption of response messages.
 interface CppSide {
@@ -40,7 +42,11 @@ interface CppSide {
   // Responses from specific tests.
   PingResponse();
   EchoResponse(EchoArgsList list);
-  BitFlipResponse(EchoArgsList arg);
+
+  // Having an associated interface pointer in the message makes sure the
+  // message header version 2 is tested.
+  BitFlipResponse(EchoArgsList arg, associated NotUsed? not_used);
+
   BackPointerResponse(EchoArgsList arg);
 };