bindings: Place bool flags in a batch in IDL dictionary

third_party/WebKit/Source/bindings/templates/dictionary_impl.h.tmpl

Index: third_party/WebKit/Source/bindings/templates/dictionary_impl.h.tmpl
diff --git a/third_party/WebKit/Source/bindings/templates/dictionary_impl.h.tmpl b/third_party/WebKit/Source/bindings/templates/dictionary_impl.h.tmpl
index 7c904e5ee5aa9c3dc082d67b0eee68e536e19eed..72a584577a832c5dc33e0b4f014dfc8800583a2d 100644
--- a/third_party/WebKit/Source/bindings/templates/dictionary_impl.h.tmpl
+++ b/third_party/WebKit/Source/bindings/templates/dictionary_impl.h.tmpl
@@ -36,10 +36,10 @@ class {{exported}}{{cpp_class}}{% if parent_cpp_class %} : public {{parent_cpp_c
   DECLARE_VIRTUAL_TRACE();
 
  private:
-  {% for member in members %}
-  {% if member.nullable_indicator_name %}
+  {% for member in members if member.nullable_indicator_name %}
   bool {{member.nullable_indicator_name}} = false;
-  {% endif %}
+  {% endfor %}
+  {% for member in members %}

[peria, 2017/04/14 03:20:04]

nit: would you put an empty line between two {% for %} loops?

[bashi, 2017/04/14 03:22:47]

Hmm, actually I did that but reverted because it produced an empty line when
there is no bool flag.

[peria, 2017/04/14 03:44:49]

IMO, such empty lines should be removed in the filter
format_blink_cpp_source_code, and having an empty line
just below "private:" is acceptable.

[bashi, 2017/04/14 03:55:28]

I'd prefer not having an empty line but either is fine with me. Done.

   {{member.member_cpp_type}} m_{{member.cpp_name}};
   {% endfor %}