[polymer] skip linter in tests

[polymer] skip linter in tests

Arguably we should only be linting things reachable from deployment in deploy mode, but that's a bigger change.

Combined with the previous CL for inline scripts (https://codereview.chromium.org/180933002/) this is enough to import polymer's many HTML elements without causing tests to timeout/fail/run out of memory due to unnecessary scanning ~300 HTML files.

R=sigmund@google.com

Committed: https://code.google.com/p/dart/source/detail?r=33146

[Siggi Cherem (dart-lang), 2014-02-26 22:39:38]

lgtm, consider adding a small TODO to write down what we discussed (that we
probably want to run the linter on the transitively imported elements)

https://codereview.chromium.org/179163007/diff/20001/pkg/polymer/lib/src/buil...
File pkg/polymer/lib/src/build/common.dart (right):

https://codereview.chromium.org/179163007/diff/20001/pkg/polymer/lib/src/buil...
pkg/polymer/lib/src/build/common.dart:67: /// True to run liner on all html
files before starting other phasese.
phasese => phases

[Jennifer Messerly, 2014-02-28 03:00:22]

thanks!

https://codereview.chromium.org/179163007/diff/20001/pkg/polymer/lib/src/buil...
File pkg/polymer/lib/src/build/common.dart (right):

https://codereview.chromium.org/179163007/diff/20001/pkg/polymer/lib/src/buil...
pkg/polymer/lib/src/build/common.dart:67: /// True to run liner on all html
files before starting other phasese.
On 2014/02/26 22:39:38, Siggi Cherem (dart-lang) wrote:
> phasese => phases

Done. Also added bug reference.

[Jennifer Messerly, 2014-02-28 03:04:41]

Committed patchset #3 manually as r33146 (presubmit successful).