bindings: Place bool flags in a batch in IDL dictionary

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

 {% filter format_blink_cpp_source_code %}
 
 {% include 'copyright_block.txt' %}
 #ifndef {{cpp_class}}_h
 #define {{cpp_class}}_h
 
 {% for filename in header_includes %}
 #include "{{filename}}"
 {% endfor %}
 
@@ skipping 18 matching lines @@
   void {{member.setter_name}}({{member.rvalue_cpp_type}});
   {% if member.null_setter_name %}
   void {{member.null_setter_name}}();
   {% endif %}
 
   {% endfor %}
   v8::Local<v8::Value> ToV8Impl(v8::Local<v8::Object>, v8::Isolate*) const override;
   DECLARE_VIRTUAL_TRACE();
 
  private:
+  {% for member in members if member.nullable_indicator_name %}
+  bool {{member.nullable_indicator_name}} = false;
+  {% 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.

-  {% if member.nullable_indicator_name %}
-  bool {{member.nullable_indicator_name}} = false;
-  {% endif %}
   {{member.member_cpp_type}} m_{{member.cpp_name}};
   {% endfor %}
 
   friend class V8{{cpp_class}};
 };
 
 }  // namespace blink
 
 #endif  // {{cpp_class}}_h
 
 {% endfilter %}{# format_blink_cpp_source_code #}