Generate getters/setters for some fields on groups in ComputedStyle

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

Index: third_party/WebKit/Source/build/scripts/templates/ComputedStyleBase.h.tmpl
diff --git a/third_party/WebKit/Source/build/scripts/templates/ComputedStyleBase.h.tmpl b/third_party/WebKit/Source/build/scripts/templates/ComputedStyleBase.h.tmpl
index 03a48012e77dde6c9d76d4582bd0b1692826e4ba..8c14ccd7a64dd72b4802ce2aeae83d4de0bd2006 100644
--- a/third_party/WebKit/Source/build/scripts/templates/ComputedStyleBase.h.tmpl
+++ b/third_party/WebKit/Source/build/scripts/templates/ComputedStyleBase.h.tmpl
@@ -81,7 +81,7 @@ class CORE_EXPORT ComputedStyleBase {
 
   {% 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:
@@ -93,6 +93,12 @@ class CORE_EXPORT ComputedStyleBase {
   {% 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.