Generate getters/setters for some fields on groups in ComputedStyle

third_party/WebKit/Source/build/scripts/templates/fields/external.tmpl

Index: third_party/WebKit/Source/build/scripts/templates/fields/external.tmpl
diff --git a/third_party/WebKit/Source/build/scripts/templates/fields/external.tmpl b/third_party/WebKit/Source/build/scripts/templates/fields/external.tmpl
index b5753c26a001eebabd14697eafbff861b33a901b..a969e7600d450addef4d2674a04ad9970f3df94b 100644
--- a/third_party/WebKit/Source/build/scripts/templates/fields/external.tmpl
+++ b/third_party/WebKit/Source/build/scripts/templates/fields/external.tmpl
@@ -1,8 +1,14 @@
 {% import 'fields/base.tmpl' as base %}
 {% from 'fields/field.tmpl' import decode %}
-{% macro decl_methods(field) %}
-{{base.decl_methods(field)}}
+{% macro decl_public_methods(field) %}
+{{base.decl_public_methods(field)}}
 void {{field.setter_method_name}}({{field.type_name}}&& v) {
   {{field.name}} = std::move({{decode(field, "v")}});
 }
 {% endmacro %}
+
+{% macro decl_protected_methods(field) -%}
+{{return_type}} {{field.internal_mutable_method_name}}() {

[shend, 2017/04/20 03:57:55]

I think this one can stay as {{field.type_name}}&. External fields can never be
a bit field. Hence, we would always be returning by reference. return_type is
only useful when field can be a bit field or a non bit field (e.g. storage_only
fields).

[nainar, 2017/04/20 05:15:58]

Done.

+  return {{decode(field, field.name)}};
+}
+{%- endmacro %}