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

Unified Diff: Source/build/scripts/templates/MediaFeatures.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/MediaFeatures.h.tmpl
diff --git a/Source/build/scripts/templates/MediaFeatures.h.tmpl b/Source/build/scripts/templates/MediaFeatures.h.tmpl
index 224e7d07efaf279a7e1420f3621d7669cf9d50b8..18a0819a15213e2d9a4714e74823d440b83fcbec 100644
--- a/Source/build/scripts/templates/MediaFeatures.h.tmpl
+++ b/Source/build/scripts/templates/MediaFeatures.h.tmpl
@@ -1,12 +1,16 @@
-{% from "macros.tmpl" import license -%}
+{% from "macros.tmpl" import license %}
{{ license() }}
#ifndef MediaFeatures_h
#define MediaFeatures_h
#define CSS_MEDIAQUERY_NAMES_FOR_EACH_MEDIAFEATURE(macro) \
-{%- for entry in entries %}
- macro({{entry|symbol}}) {%- if not loop.last %}\{%-endif %}
-{%- endfor %}
+ {% for entry in entries %}
+ {# FIXME: add space before \ #}
+ macro({{entry|symbol}}){% if not loop.last %}\
+ {% endif %}
+ {% endfor %}
+{# FIXME: Remove extra blank line #}
+
#endif

Powered by Google App Engine
This is Rietveld 408576698