Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1587)

Unified Diff: third_party/WebKit/Source/build/scripts/templates/ComputedStyleBase.h.tmpl

Issue 2826653002: Generate getters/setters for some fields on groups in ComputedStyle (Closed)
Patch Set: Append Internal to all protected methods Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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' %}
shend 2017/04/19 03:46:20 nit: I would do either: {% if field.field_template
nainar 2017/04/19 04:02:46 Went with option 1
+ {{field_templates[field.field_template].decl_protected_methods(field)|indent(2)}}
+ {% endif %}
+ {% endfor %}
+
~ComputedStyleBase() = default;
// Storage.

Powered by Google App Engine
This is Rietveld 408576698