New issue
Advanced search Search tips

Issue 725262 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Wrap long strings over 80 columns in some layout tests

Project Member Reported by hongchan@chromium.org, May 22 2017

Issue description

Some tests are still having lines over 80 columns due to long strings literals.

See the attached log from layout-test-tidy.
 
result.txt
131 KB View Download

Comment 1 by rtoy@chromium.org, Jul 26 2017

Blocking: 745778

Comment 2 by foolip@chromium.org, Jul 31 2017

Note that if it's hard to avoid this, then exceptions can be added to https://github.com/w3c/web-platform-tests/blob/master/lint.whitelist if that's what's blocking upstreaming.

Comment 3 by rtoy@chromium.org, Jul 31 2017

Not strictly required, I think.  We just want to beautify our own tests before upstreaming them.

Comment 4 by rtoy@chromium.org, Jun 28 2018

Blocking: -745778
Doesn't block upstreaming tests to WPT.

Sign in to add a comment