[MediaRouter] Update MR-2-Extension's PostMessage to return boolean.

extensions/renderer/resources/media_router_bindings.js

Index: extensions/renderer/resources/media_router_bindings.js
diff --git a/extensions/renderer/resources/media_router_bindings.js b/extensions/renderer/resources/media_router_bindings.js
index f28c354d2df7241a7ed0571a243be1b7e2a13884..431b3ae43068bd1470d7fbe98055eaf4c5cd15b9 100644
--- a/extensions/renderer/resources/media_router_bindings.js
+++ b/extensions/renderer/resources/media_router_bindings.js
@@ -304,9 +304,9 @@ define('media_router_bindings', [
     this.stopObservingMediaSinks = null;
 
     /**
-     * @type {function(string, string, string)}
+     * @type {function(string, string)}
      */
-    this.postMessage = null;
+    this.sendMessage = null;
 
     /**
      * @type {function()}
@@ -353,7 +353,7 @@ define('media_router_bindings', [
     var requiredHandlers = [
       'stopObservingMediaRoutes',
       'startObservingMediaRoutes',
-      'postMessage',
+      'sendMessage',
       'closeRoute',
       'joinRoute',
       'createRoute',
@@ -448,12 +448,17 @@ define('media_router_bindings', [
    * Posts a message to the route designated by |routeId|.
    * @param {!string} routeId
    * @param {!string} message
-   * @param {string} extraInfoJson
+   * @return {!Promise.<boolean>} Resolved with true if the message was sent,
+   *  or false on failure.

[Ken Rockot(use gerrit already), 2015/06/11 21:22:04]

super nitty style nit: should indent 4 spaces from the @

  * @return ...
  *     or false ...

[haibinlu, 2015/06/11 22:46:23]

Done.

    */
-  MediaRouter.prototype.postMessage = function(
-      routeId, message, extraInfoJson) {
-    // TODO(mfoltz): Remove extraInfoJson if no longer needed.
-    this.handlers_.postMessage(routeId, message, JSON.parse(extraInfoJson));
+  MediaRouter.prototype.sendMessage = function(

[mark a. foltz, 2015/06/11 18:48:33]

It seems odd that the Promise should be resolved here.  Perhaps this function
should return the Promise to the caller so it can chain off of it.

[Ken Rockot(use gerrit already), 2015/06/11 21:22:04]

I agree. A Promise resolving/rejecting is effectively already a boolean result,
so doing this only serves to hide potential failure details in case of
rejection.

[haibinlu, 2015/06/11 22:46:23]

For mojo method that returns a value,  mojo generates Promise as return value at
Javascript side, and callback at c++ side. I did not find a way to tell mojo how
to convert a rejected promise into proper return value for the callback.

+      routeId, message) {
+    this.handlers_.sendMessage(routeId, message)
+        .then(function() {
+          return true;
+        }, function() {
+          return false;
+        });
   };
 
   /**