Generate getters/setters for some fields on groups in ComputedStyle

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

 {% from 'fields/field.tmpl' import encode, decode, return_type %}
-{% macro decl_methods(field) -%}
-inline static {{return_type(field)}} {{field.initial_method_name}}() {
+{% macro decl_public_methods(field) -%}
+inline static {{field.type_name}} {{field.initial_method_name}}() {

[suzyh_UTC10 (ex-contributor), 2017/04/19 08:03:33]

Why the change from return_type(field) to field.type_name?

[nainar, 2017/04/19 08:49:35]

Mistake. Fixed. 

   return {{field.default_value}};
 }
 {{return_type(field)}} {{field.getter_method_name}}() const {
   return {{decode(field, field.name)}};
 }
-void {{field.setter_method_name}}({{field.type_name}} v) {
+void {{field.setter_method_name}}(const {{field.type_name}}& v) {

[suzyh_UTC10 (ex-contributor), 2017/04/19 08:03:33]

This is resulting in a rather odd-looking "const bool&" in the generated diff.
Are you sure this is what you want?

[nainar, 2017/04/19 08:49:35]

Fixed by using new argument_type macro

   {{field.name}} = {{encode(field, "v")}};
 }
 inline void {{field.resetter_method_name}}() {
   {{field.name}} = {{encode(field, field.default_value)}};
 }
 {%- endmacro %}
 
+{% macro decl_protected_methods(field) -%}
+{{return_type(field)}} {{field.internal_getter_method_name}}() const {

[suzyh_UTC10 (ex-contributor), 2017/04/19 08:03:33]

On first glance it's not clear why there's an internal and external version of
these functions, since they appear to be the same. I think it may only be the
same here as a base implementation, but I could be totally confused.

[nainar, 2017/04/19 08:49:35]

They are the same except the ones I am adding are protected. I could as shend@
suggested in a follow up CL refactor this out to have less repetition. 

+  return {{decode(field, field.name)}};
+}
+void {{field.internal_setter_method_name}}({{field.type_name}} v) {
+  {{field.name}} = {{encode(field, "v")}};
+}
+{%- endmacro %}