Mojo: add javascript bindings for request/response

mojo/public/bindings/generators/js_templates/interface_definition.tmpl

 {%- for method in interface.methods %}
   var k{{interface.name}}_{{method.name}}_Name = {{method.ordinal}};
 {%- endfor %}
 
   function {{interface.name}}Proxy(receiver) {
     this.receiver_ = receiver;
   }
 
 {%- for method in interface.methods %}
   {{interface.name}}Proxy.prototype.{{method.name|stylize_method}} = function(
 {%- for parameter in method.parameters -%}
 {{parameter.name}}{% if not loop.last %}, {% endif %}
-{%- endfor %}) {
+{%- endfor -%}
+{%- if method.response_parameters != None -%}
+, closure

[abarth-chromium, 2014/03/26 18:26:32]

Why not use promises instead of callbacks?  This JS environment should have
Promises enabled now.

+{%- endif -%}
+) {
     var params = new {{interface.name}}_{{method.name}}_Params();
 {%- for parameter in method.parameters %}
     params.{{parameter.name}} = {{parameter.name}};
 {%- endfor %}
 
+{%- if method.response_parameters == None %}
     var builder = new codec.MessageBuilder(
         k{{interface.name}}_{{method.name}}_Name,
         codec.align({{interface.name}}_{{method.name}}_Params.encodedSize));
     builder.encodeStruct({{interface.name}}_{{method.name}}_Params, params);
     var message = builder.finish();
     this.receiver_.accept(message);
+{%- else %}
+    var builder = new codec.MessageWithRequestIDBuilder(
+        k{{interface.name}}_{{method.name}}_Name,
+        codec.align({{interface.name}}_{{method.name}}_Params.encodedSize),
+        codec.kMessageExpectsResponse, 0);
+    builder.encodeStruct({{interface.name}}_{{method.name}}_Params, params);
+    var message = builder.finish();
+    this.receiver_.acceptWithResponder(message, { accept: function(message) {
+      var reader = new codec.MessageReader(message);
+      var response_params =

[abarth-chromium, 2014/03/26 18:26:32]

responseParams

+          reader.decodeStruct({{interface.name}}_{{method.name}}_ResponseParams);
+      closure(
+{%- for parameter in method.response_parameters -%}
+response_params.{{parameter.name}}{% if not loop.last %}, {% endif %}
+{%- endfor -%});
+    }});
+{%- endif %}
   };
 {%- endfor %}
 
   function {{interface.name}}Stub() {
   }
 
   {{interface.name}}Stub.prototype.accept = function(message) {
     var reader = new codec.MessageReader(message);
     switch (reader.messageName) {
 {%- for method in interface.methods %}
+{%- if method.response_parameters == None %}

[Matt Perry, 2014/03/26 18:19:43]

is the idea that if the method has response_parameters, then acceptWithResponder
would be called instead? if so, maybe we should handle the case and raise/print
an error?

     case k{{interface.name}}_{{method.name}}_Name:
       var params = reader.decodeStruct({{interface.name}}_{{method.name}}_Params);
       this.{{method.name|stylize_method}}(
 {%- for parameter in method.parameters -%}
 params.{{parameter.name}}{% if not loop.last %}, {% endif %}
 {%- endfor %});
       return true;
+{%- endif %}
 {%- endfor %}
     default:
       return false;
+    }
+  };
+
+  {{interface.name}}Stub.prototype.acceptWithResponder =
+      function(message, responder) {
+    var reader = new codec.MessageReader(message);
+    switch (reader.messageName) {
+{%- for method in interface.methods %}
+{%- if method.response_parameters != None %}
+    case k{{interface.name}}_{{method.name}}_Name:
+      var params = reader.decodeStruct({{interface.name}}_{{method.name}}_Params);
+      this.{{method.name|stylize_method}}(
+{%- for parameter in method.parameters -%}
+params.{{parameter.name}},
+{%- endfor %} function (
+{%- for parameter in method.response_parameters -%}
+{{parameter.name}}{% if not loop.last %}, {% endif -%}
+{%- endfor -%}) {
+        var response_params =

[abarth-chromium, 2014/03/26 18:26:32]

responseParams

+            new {{interface.name}}_{{method.name}}_ResponseParams();
+{%-     for parameter in method.response_parameters %}
+        response_params.{{parameter.name}} = {{parameter.name}};
+{%-     endfor %}
+        var builder = new codec.MessageWithRequestIDBuilder(
+            k{{interface.name}}_{{method.name}}_Name,
+            codec.align({{interface.name}}_{{method.name}}_ResponseParams.encodedSize),
+            codec.kMessageIsResponse, reader.requestID);
+        builder.encodeStruct({{interface.name}}_{{method.name}}_ResponseParams,
+                             response_params);
+        var message = builder.finish();
+        responder.accept(message);
+      });
+      return true;
+{%- endif %}
+{%- endfor %}
+    default:
+      return false;
     }
   };
 
 {#--- Enums #}
 {%  from "enum_definition.tmpl" import enum_def -%}
 {%  for enum in interface.enums %}
   {{enum_def("%sProxy.%s"|format(interface.name, enum.name), enum, module)}}
   {{interface.name}}Stub.{{enum.name}} = {{interface.name}}Proxy.{{enum.name}};
 {%-  endfor %}