New issue
Advanced search Search tips

Issue 720378 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: May 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug



Sign in to add a comment

Add new layout-test-tidy tool to LayoutTests/webaudio

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

Issue description

Implement a Node.js utility that cleans up layout tests automatically. Apply such tool to entire test files as the last step of WPT upstreaming.
 
Project Member

Comment 1 by bugdroid1@chromium.org, May 15 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/1dd94e01da63c05f3c6855ac25a911f2d43868ae

commit 1dd94e01da63c05f3c6855ac25a911f2d43868ae
Author: hongchan <hongchan@chromium.org>
Date: Mon May 15 19:12:24 2017

Add layout-test-tidy to LayoutTests/webaudio/tools

This tool, a Node.js CLI utility, performs a set of clean-up tasks for layout
test files in LayoutTests/webaudio. (Potentially it can be applied to any kind
of layout test files in HTML or JS format.)

The clean-up tasks includes:
  - Sanitize missing or incorrect HTML elements: reordering |script| elements or
    adding missing |title| element.
  - Apply [html-tidy](http://www.html-tidy.org/).
  - Apply [clang-format](https://clang.llvm.org/docs/ClangFormat.html).
  - Perform RegExp substitution based on predefined dictionary. (e.g. |var| to
    |let|, or redundant empty lines in markup.)

For every step of clean-up processing, the tool collects warning, diagnostics or
notes and print out the logs at the end of the processing.

For more info, refer README.md file included.

BUG= 720378 

Review-Url: https://codereview.chromium.org/2872393002
Cr-Commit-Position: refs/heads/master@{#471857}

[add] https://crrev.com/1dd94e01da63c05f3c6855ac25a911f2d43868ae/third_party/WebKit/LayoutTests/webaudio/tools/.gitignore
[add] https://crrev.com/1dd94e01da63c05f3c6855ac25a911f2d43868ae/third_party/WebKit/LayoutTests/webaudio/tools/README.md
[add] https://crrev.com/1dd94e01da63c05f3c6855ac25a911f2d43868ae/third_party/WebKit/LayoutTests/webaudio/tools/layout-test-tidy.js
[add] https://crrev.com/1dd94e01da63c05f3c6855ac25a911f2d43868ae/third_party/WebKit/LayoutTests/webaudio/tools/package.json

Status: Verified (was: Started)

Sign in to add a comment