Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(169)

Side by Side Diff: mojo/public/tools/bindings/generators/cpp_templates/interface_macros.tmpl

Issue 2136733002: Mojo C++ bindings: add a new mode to generator to use native STL/WTF types (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@67_new
Patch Set: . Created 4 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 {%- macro declare_params(prefix, parameters) %} 1 {%- macro declare_params(prefix, parameters) %}
2 {%- for param in parameters -%} 2 {%- for param in parameters -%}
3 {{param.kind|cpp_wrapper_param_type}} {{prefix}}{{param.name}} 3 {{param.kind|cpp_wrapper_param_type}} {{prefix}}{{param.name}}
4 {%- if not loop.last %}, {% endif %} 4 {%- if not loop.last %}, {% endif %}
5 {%- endfor %} 5 {%- endfor %}
6 {%- endmacro %} 6 {%- endmacro %}
7 7
8 {%- macro declare_responder_params(prefix, parameters, for_blink) %} 8 {%- macro declare_responder_params(prefix, parameters, for_blink, use_new_wrappe r_types) %}
9 {%- for param in parameters -%} 9 {%- for param in parameters -%}
10 {%- if (not param.kind|is_string_kind) or for_blink -%} 10 {%- if (not param.kind|is_string_kind) or for_blink or
11 use_new_wrapper_types -%}
11 {{param.kind|cpp_wrapper_param_type}} {{prefix}}{{param.name}} 12 {{param.kind|cpp_wrapper_param_type}} {{prefix}}{{param.name}}
12 {%- else %} 13 {%- else %}
13 mojo::String {{prefix}}{{param.name}} 14 mojo::String {{prefix}}{{param.name}}
14 {%- endif %} 15 {%- endif %}
15 {%- if not loop.last %}, {% endif %} 16 {%- if not loop.last %}, {% endif %}
16 {%- endfor %} 17 {%- endfor %}
17 {%- endmacro %} 18 {%- endmacro %}
18 19
19 {%- macro declare_callback(method, for_blink) -%} 20 {%- macro declare_callback(method, for_blink, use_new_wrapper_types) -%}
20 base::Callback<void( 21 base::Callback<void(
21 {%- for param in method.response_parameters -%} 22 {%- for param in method.response_parameters -%}
22 {#- TODO(yzshen): For historical reasons, we use mojo::String here (instead of 23 {#- TODO(yzshen): For historical reasons, we use mojo::String here (instead of
23 const mojo::String&) inconsistently. Preserve the behavior temporarily. #} 24 const mojo::String&) inconsistently. Preserve the behavior temporarily. #}
24 {%- if (not param.kind|is_string_kind) or for_blink -%} 25 {%- if (not param.kind|is_string_kind) or for_blink or
26 use_new_wrapper_types -%}
25 {{param.kind|cpp_wrapper_param_type}} 27 {{param.kind|cpp_wrapper_param_type}}
26 {%- else -%} 28 {%- else -%}
27 mojo::String 29 mojo::String
28 {%- endif %} 30 {%- endif %}
29 {%- if not loop.last %}, {% endif %} 31 {%- if not loop.last %}, {% endif %}
30 {%- endfor -%} 32 {%- endfor -%}
31 )> 33 )>
32 {%- endmacro -%} 34 {%- endmacro -%}
33 35
34 {%- macro declare_request_params(prefix, method) -%} 36 {%- macro declare_request_params(prefix, method) -%}
35 {{declare_params(prefix, method.parameters)}} 37 {{declare_params(prefix, method.parameters)}}
36 {%- if method.response_parameters != None -%} 38 {%- if method.response_parameters != None -%}
37 {%- if method.parameters %}, {% endif -%} 39 {%- if method.parameters %}, {% endif -%}
38 const {{method.name}}Callback& callback 40 const {{method.name}}Callback& callback
39 {%- endif -%} 41 {%- endif -%}
40 {%- endmacro -%} 42 {%- endmacro -%}
41 43
42 {%- macro declare_sync_method_params(prefix, method) -%} 44 {%- macro declare_sync_method_params(prefix, method) -%}
43 {{declare_params(prefix, method.parameters)}} 45 {{declare_params(prefix, method.parameters)}}
44 {%- if method.response_parameters %} 46 {%- if method.response_parameters %}
45 {%- if method.parameters %}, {% endif %} 47 {%- if method.parameters %}, {% endif %}
46 {%- for param in method.response_parameters -%} 48 {%- for param in method.response_parameters -%}
47 {{param.kind|cpp_wrapper_type}}* {{prefix}}{{param.name}} 49 {{param.kind|cpp_wrapper_type}}* {{prefix}}{{param.name}}
48 {%- if not loop.last %}, {% endif %} 50 {%- if not loop.last %}, {% endif %}
49 {%- endfor %} 51 {%- endfor %}
50 {%- endif -%} 52 {%- endif -%}
51 {%- endmacro -%} 53 {%- endmacro -%}
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698