Mojo JS bindings: make sure that console.log() could be used in the bindings.

mojo/public/js/router.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/public/js/router", [
+  "console",
   "mojo/public/js/codec",
   "mojo/public/js/core",
   "mojo/public/js/connector",
   "mojo/public/js/validator",
-], function(codec, core, connector, validator) {
+], function(console, codec, core, connector, validator) {
 
   var Connector = connector.Connector;
   var MessageReader = codec.MessageReader;
   var Validator = validator.Validator;
 
   function Router(handle, connectorFactory) {
     if (!core.isHandle(handle))
       throw new Error("Router constructor: Not a handle");
     if (connectorFactory === undefined)
       connectorFactory = Connector;
@@ skipping 74 matching lines @@
     for (var i = 0; err === noError && i < this.payloadValidators_.length; ++i)
       err = this.payloadValidators_[i](messageValidator);
 
     if (err == noError)
       this.handleValidIncomingMessage_(message);
     else
       this.handleInvalidIncomingMessage_(message, err);
   };
 
   Router.prototype.handleValidIncomingMessage_ = function(message) {
+    console.log("heeeeeeeeeeeeeeeeeeeeeeeeeeeeeere");
     if (this.testingController_)
       return;
 
     if (message.expectsResponse()) {
       if (this.incomingReceiver_) {
         this.incomingReceiver_.acceptWithResponder(message, this);
       } else {
         // If we receive a request expecting a response when the client is not
         // listening, then we have no choice but to tear down the pipe.
         this.close();
       }
     } else if (message.isResponse()) {
       var reader = new MessageReader(message);
       var requestID = reader.requestID;
       var completer = this.completers_.get(requestID);
-      this.completers_.delete(requestID);
-      completer.resolve(message);
+      if (completer) {
+        this.completers_.delete(requestID);
+        completer.resolve(message);
+      } else {
+        console.log("Unexpected response with request ID: " + requestID);
+      }
     } else {
       if (this.incomingReceiver_)
         this.incomingReceiver_.accept(message);
     }
   };
 
   Router.prototype.handleInvalidIncomingMessage_ = function(message, error) {
     if (!this.testingController_) {
-      // TODO(yzshen): Consider logging and notifying the embedder.
+      // TODO(yzshen): Consider notifying the embedder.
       // TODO(yzshen): This should also trigger connection error handler.
       // Consider making accept() return a boolean and let the connector deal
       // with this, as the C++ code does.
+      console.log("Invalid message: " + validator.validationError[error]);
+
       this.close();
       return;
     }
 
     this.testingController_.onInvalidIncomingMessage(error);
   };
 
   Router.prototype.handleConnectionError_ = function(result) {
     this.completers_.forEach(function(value) {
       value.reject(result);
@@ skipping 23 matching lines @@
   RouterTestingController.prototype.onInvalidIncomingMessage =
       function(error) {
     if (this.invalidMessageHandler_)
       this.invalidMessageHandler_(error);
   };
 
   var exports = {};
   exports.Router = Router;
   return exports;
 });