Generate getters/setters for some fields on groups in ComputedStyle

third_party/WebKit/Source/build/scripts/templates/ComputedStyleBase.h.tmpl

 {% from 'macros.tmpl' import license, print_if %}
 {{license()}}
 
 #ifndef ComputedStyleBase_h
 #define ComputedStyleBase_h
 
 #include "core/style/ComputedStyleConstants.h"
 #include "core/CoreExport.h"
 {% for path in include_paths %}
 #include "{{path}}"
@@ skipping 63 matching lines @@
   // style that are marked as inherited by this style.
   void PropagateIndependentInheritedProperties(
       const ComputedStyleBase& parentStyle);
 
   // Fields.
   // TODO(sashab): Remove initialFoo() static methods and update callers to
   // use resetFoo(), which can be more efficient.
 
   {% for field in fields %}
   // {{field.property_name}}
-  {{field_templates[field.field_template].decl_methods(field)|indent(2)}}
+  {{field_templates[field.field_template].decl_public_methods(field)|indent(2)}}
 
   {% endfor %}
  protected:
   // Constructor and destructor are protected so that only the parent class ComputedStyle
   // can instantiate this class.
   ALWAYS_INLINE ComputedStyleBase() :
   {% for field in fields %}
       {{field.name}}({{encode(field, field.default_value)}}){{print_if(not loop.last, ',')}}
   {% endfor %}
   {}
 
+  {% for field in fields %}
+  {% if field.field_template == 'storage_only' or field.field_template == 'monotonic_flag' or field.field_template == 'external' %}
+  {{field_templates[field.field_template].decl_protected_methods(field)|indent(2)}}
+  {% endif %}

[shend, 2017/04/19 03:41:29]

The public methods have a comment with the property name, and an extra blank
line after each set of methods. Do you think we need them here as well?

[nainar, 2017/04/19 04:02:46]

Makes sense. Done

+  {% endfor %}
+
   ~ComputedStyleBase() = default;
 
   // Storage.
   {% for field in fields %}
   {% if field.is_bit_field %}
   unsigned {{field.name}} : {{field.size}}; // {{field.type_name}}
   {% else %}
   {{field.type_name}} {{field.name}};
   {% endif %}
   {% endfor %}
 };
 
 } // namespace blink
 
 #endif // ComputedStyleBase_h