Format dart:html and related generated files on output

Format dart:html and related generated files on output

BUG=
R=jacobr@google.com, terry@google.com

Committed: https://github.com/dart-lang/sdk/commit/cedd8a6854206f32ecd3a458ba1716ef9760171c

[Alan Knight, 2017-05-05 01:07:48]

alanknight@google.com changed reviewers:
+ jacobr@google.com, terry@google.com

[terry, 2017-05-05 06:54:31]

lgtm just need to handle one more file.

https://codereview.chromium.org/2858323003/diff/1/tools/dom/scripts/dartdomge...
File tools/dom/scripts/dartdomgenerator.py (right):

https://codereview.chromium.org/2858323003/diff/1/tools/dom/scripts/dartdomge...
tools/dom/scripts/dartdomgenerator.py:234: 
Looks good to me, although I think we'll want to dartfmt the cached_patches.dart
file too.  Maybe this should be a function that is called after line 363 below.

[terry, 2017-05-05 06:59:01]

lgtm

https://codereview.chromium.org/2858323003/diff/1/tools/dom/scripts/dartdomge...
File tools/dom/scripts/dartdomgenerator.py (right):

https://codereview.chromium.org/2858323003/diff/1/tools/dom/scripts/dartdomge...
tools/dom/scripts/dartdomgenerator.py:234: 
On 2017/05/05 06:54:31, terry wrote:
> Looks good to me, although I think we'll want to dartfmt the
cached_patches.dart
> file too.  Maybe this should be a function that is called after line 363
below.

You what this is fine just as it is.  This is a small template that is squirted
out shouldn't be dartfmt'd until after generate_patches.py is run which is much
later after Dartium is built.  You could add this code to generate_patches.py to
dartfmt the cached_patches.dart file after its really generated.

[Jacob, 2017-05-05 16:04:00]

lgtm

[Alan Knight, 2017-05-05 17:09:06]

Description was changed from

==========
Format dart:html and related generated files on output

BUG=
==========

to

==========
Format dart:html and related generated files on output

BUG=
R=jacobr@google.com, terry@google.com

Committed:
https://github.com/dart-lang/sdk/commit/cedd8a6854206f32ecd3a458ba1716ef9760171c
==========

[Alan Knight, 2017-05-05 17:09:08]

Committed patchset #1 (id:1) manually as
cedd8a6854206f32ecd3a458ba1716ef9760171c (presubmit successful).

[Alan Knight, 2017-05-05 17:09:14]

Formatting cached_patches looks simple except it's not clear where to get
dartfmt from, or if we can even rely on it being there. I'll leave that for
another CL.