Fix handling of null pointers in JS bindings.

mojo/apps/js/test/js_to_cpp_unittest.js

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 define('mojo/apps/js/test/js_to_cpp_unittest', [
   'console',
   'mojo/apps/js/test/js_to_cpp.mojom',
   'mojo/public/js/bindings/connection',
   'mojo/public/js/bindings/connector',
   'mojo/public/js/bindings/core',
@@ skipping 11 matching lines @@
     this.cppSide_ = cppSide;
     cppSide.startTest();
   }
 
   JsSideConnection.prototype = Object.create(jsToCpp.JsSideStub.prototype);
 
   JsSideConnection.prototype.ping = function (arg) {
     this.cppSide_.pingResponse();
   };
 
-  JsSideConnection.prototype.echo = function (numIterations, arg) {
-    var arg2;
+  JsSideConnection.prototype.echo = function (numIterations, list) {
+    var arg = list.item;
     var dataPipe1;
     var dataPipe2;
     var i;
     var messagePipe1;
     var messagePipe2;
+    var resultList;
+    var specialArg;
 
     // Ensure expected negative values are negative.
     if (arg.si64 > 0)
       arg.si64 = BAD_VALUE;
 
     if (arg.si32 > 0)
       arg.si32 = BAD_VALUE;
 
     if (arg.si16 > 0)
       arg.si16 = BAD_VALUE;
 
     if (arg.si8 > 0)
       arg.si8 = BAD_VALUE;
 
     for (i = 0; i < numIterations; ++i) {
       dataPipe1 = core.createDataPipe(DATA_PIPE_PARAMS);
       dataPipe2 = core.createDataPipe(DATA_PIPE_PARAMS);
       messagePipe1 = core.createMessagePipe();
       messagePipe2 = core.createMessagePipe();
 
       arg.data_handle = dataPipe1.consumerHandle;
       arg.message_handle = messagePipe1.handle1;
 
-      arg2 = new jsToCpp.EchoArgs();
-      arg2.si64 = -1;
-      arg2.si32 = -1;
-      arg2.si16 = -1;
-      arg2.si8 = -1;
-      arg2.name = 'going';
-      arg2.data_handle = dataPipe2.consumerHandle;
-      arg2.message_handle = messagePipe2.handle1;
+      specialArg = new jsToCpp.EchoArgs();
+      specialArg.si64 = -1;
+      specialArg.si32 = -1;
+      specialArg.si16 = -1;
+      specialArg.si8 = -1;
+      specialArg.name = 'going';
+      specialArg.data_handle = dataPipe2.consumerHandle;
+      specialArg.message_handle = messagePipe2.handle1;
 
       writeDataPipe(dataPipe1, senderData);
       writeDataPipe(dataPipe2, senderData);
-      this.cppSide_.echoResponse(arg, arg2);
+
+      resultList = new jsToCpp.EchoArgsList();
+      resultList.next = new jsToCpp.EchoArgsList();
+      resultList.item = specialArg;
+      resultList.next.item = arg;
+
+      this.cppSide_.echoResponse(resultList);
 
       core.close(dataPipe1.producerHandle);
       core.close(dataPipe2.producerHandle);
       core.close(messagePipe1.handle0);
       core.close(messagePipe2.handle0);
     }
     this.cppSide_.testFinished();
   };
 
   JsSideConnection.prototype.bitFlip = function (arg) {
@@ skipping 53 matching lines @@
   return function(handle) {
     var i;
     senderData = new Uint8Array(DATA_PIPE_PARAMS.capacityNumBytes);
     for (i = 0; i < senderData.length; ++i)
       senderData[i] = i;
 
     retainedConnection = new connection.Connection(handle, JsSideConnection,
                                                    jsToCpp.CppSideProxy);
   };
 });