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

Unified Diff: Source/build/scripts/templates/InternalRuntimeFlags.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/InternalRuntimeFlags.h.tmpl
diff --git a/Source/build/scripts/templates/InternalRuntimeFlags.h.tmpl b/Source/build/scripts/templates/InternalRuntimeFlags.h.tmpl
index 8cd44d1b5d57ceff021002e26de03163dfa59a31..4e6b909074576c4a87bf50fe12556e98932af699 100644
--- a/Source/build/scripts/templates/InternalRuntimeFlags.h.tmpl
+++ b/Source/build/scripts/templates/InternalRuntimeFlags.h.tmpl
@@ -1,4 +1,4 @@
-{% from "macros.tmpl" import wrap_with_condition, license -%}
+{% from "macros.tmpl" import wrap_with_condition, license %}
{{ license() }}
#ifndef InternalRuntimeFlags_h
@@ -19,17 +19,19 @@ public:
return adoptRefWillBeNoop(new InternalRuntimeFlags);
}
-{#-
+{#
Setting after startup does not work for most runtime flags, but we
could add an option to print setters for ones which do:
void set{{feature.name}}Enabled(bool isEnabled) { RuntimeEnabledFeatures::set{{feature.name}}Enabled(isEnabled); }
If we do that, we also need to respect Internals::resetToConsistentState.
#}
-{% for feature in features if not feature.custom %}
-{%- call wrap_with_condition(feature.condition) %}
+ {% for feature in features if not feature.custom %}
+ {% call wrap_with_condition(feature.condition) %}
bool {{feature.first_lowered_name}}Enabled() { return RuntimeEnabledFeatures::{{feature.first_lowered_name}}Enabled(); }
-{%- endcall %}
-{% endfor %}
+ {% endcall %}
+ {# FIXME: remove extra blank line #}
+
+ {% endfor %}
void trace(Visitor*) { }
« no previous file with comments | « Source/build/scripts/templates/EventFactory.cpp.tmpl ('k') | Source/build/scripts/templates/InternalRuntimeFlags.idl.tmpl » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698