Simplify whitespace handling in build/ Jinja templates

Simplify whitespace handling in build/ Jinja templates

No change to generated code, just cleanup.

This adds the flags:
lstrip_blocks=True
trim_blocks=True
...to the Jinja environment setup

This means we don't need {%- everywhere, and also allows us to indent control blocks.

(Other than that, a few consistency fixes of {{ foo() }} => {{foo()}} and " => ')

This CL does not change generated code, to simplify testing and review.
This turned up lots of minor whitespace errors or warts, which I've marked
with FIXME and will fix in a followup.

R=abarth

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=171422

[Nils Barth (inactive), 2014-04-14 03:51:09]

Hi Adam,
Here's a cleanup CL so we don't need to use {%- -%} everywhere in the Jinja
templates.
WDYT?

[abarth-chromium, 2014-04-14 04:11:35]

LGTM

These settings make the templates much prettier!

[Nils Barth (inactive), 2014-04-14 04:22:05]

The CQ bit was checked by nbarth@chromium.org

[commit-bot: I haz the power, 2014-04-14 04:22:15]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/nbarth@chromium.org/236673002/40001

[Nils Barth (inactive), 2014-04-14 04:33:43]

On 2014/04/14 04:11:35, abarth wrote:
> These settings make the templates much prettier!

Yeah, I've been staring at enough of these that I've made every effort to make
them legible.

BTW, there's also (nested) syntax highlighting available
(I have it working for vim; should be possible in emacs too):
http://www.chromium.org/developers/jinja#TOC-Syntax-highlighting

[commit-bot: I haz the power, 2014-04-14 05:27:06]

Change committed as 171422