Add Jinja2 macro for adding trailing strings.

third_party/WebKit/Source/build/scripts/templates/macros.tmpl

 {#
     FIXME: Do we need to put license blocks in generated files?
 #}
 {% macro license() %}
 // Copyright (c) 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.
 {%- endmacro %}
 
 
@@ skipping 37 matching lines @@
 
 
 {% macro trie_length_switch(length_tries, return_macro, lowercase_data) %}
 switch (length) {
 {% for length, trie in length_tries %}
 case {{ length }}:
     {{ trie_switch(trie, 0, return_macro, lowercase_data) | indent(4) }}
 {% endfor %}
 }
 {% endmacro %}
+
+
+{% macro trailing(loop, str) -%}

[meade_UTC10, 2017/02/13 06:19:58]

Would it make sense to pass in loop.last here instead of passing in loop? It
made me double take a little on why loop is being passed into trailing()
above...

[shend, 2017/02/13 06:40:37]

Hmm, my intention for this macro is to print a string based on the current loop
state. If I make it take in loop.last (i.e. a bool), then the macro kinda
becomes print_if_true(bool, str). It's also less typing for the caller :P. Would
it help if I added an explanatory comment here?

[meade_UTC10, 2017/02/13 06:51:15]

Hmm, it's really print_if_false right? I think it's actually clearer to name it
what it does - which is clearer about what's happening?


{% for field in fields %}
  {{field.name}}({{default_value(field)}}){{trailing(loop, ',')}}
{% endfor %}



{% for field in fields %}
  {{field.name}}({{default_value(field)}}){{print_if_false(loop.last, ',')}}
{% endfor %}

+{% if not loop.last %}{{str}}{% endif %}
+{%- endmacro %}