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

Unified Diff: Source/build/scripts/templates/InternalSettingsGenerated.h.tmpl

Issue 236673002: Simplify whitespace handling in build/ Jinja templates (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: One more FIXME Created 6 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: Source/build/scripts/templates/InternalSettingsGenerated.h.tmpl
diff --git a/Source/build/scripts/templates/InternalSettingsGenerated.h.tmpl b/Source/build/scripts/templates/InternalSettingsGenerated.h.tmpl
index 17300b76eef3cd34fa63e82fe6bc7365583c97e3..5d92ea6b036169e2a294e1d072b5924130fa9c0b 100644
--- a/Source/build/scripts/templates/InternalSettingsGenerated.h.tmpl
+++ b/Source/build/scripts/templates/InternalSettingsGenerated.h.tmpl
@@ -1,4 +1,4 @@
-{% from "macros.tmpl" import license -%}
+{% from "macros.tmpl" import license %}
{{ license() }}
#ifndef InternalSettingsGenerated_h
@@ -19,18 +19,18 @@ public:
explicit InternalSettingsGenerated(Page*);
virtual ~InternalSettingsGenerated();
void resetToConsistentState();
-{%- for setting in settings if setting.type|to_idl_type %}
+ {% for setting in settings if setting.type|to_idl_type %}
void set{{setting.name|upper_first}}({{setting.type|to_passing_type}} {{setting.name}});
-{%- endfor %}
+ {% endfor %}
virtual void trace(Visitor*) { }
private:
Page* m_page;
-
- {%- for setting in settings if setting.type|to_idl_type %}
+ {# FIXME: add blank line #}
+ {% for setting in settings if setting.type|to_idl_type %}
{{setting.type}} m_{{setting.name}};
- {%- endfor %}
+ {% endfor %}
};
} // namespace WebCore

Powered by Google App Engine
This is Rietveld 408576698