Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(55)

Side by Side Diff: third_party/WebKit/LayoutTests/webaudio/tools/README.md

Issue 2872393002: Add layout-test-tidy to LayoutTests/webaudio/tools (Closed)
Patch Set: Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
(Empty)
1 # layout-test-tidy
2
3 This tool, a Node.js CLI utility, performs a set of clean-up tasks for layout te st files in LayoutTests/webaudio. (Potentially it can be applied to any kind of layout test files in HTML or JS format.)
Raymond Toy 2017/05/10 21:28:43 These lines can be wrapped to 80 columns for markd
hongchan 2017/05/11 21:13:01 "When you do want to insert a break tag using Mark
4
5 The clean-up tasks includes:
6 - Sanitize missing or incorrect HTML elements: reordering |script| elements or adding missing |title| element.
7 - Apply [html-tidy](http://www.html-tidy.org/).
8 - Apply [clang-format](https://clang.llvm.org/docs/ClangFormat.html).
9 - Perform RegExp substitution based on predefined dictionary. (e.g. |var| to | let|, or redundant empty lines in markup.)
10
11 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.
12
13
14 # Installation
15
16 ```
17 cd ${WHERE_PACKAGE_JSON_IS} npm install
18 ```
19
20 The warning from html-tidy package can be safely ignored.
21
22
23 # Usage
24
25 ```
26 # Run the tool at a target path or a file.
27 node layout-test-tidy ${TARGET_PATH}
28
29 # Dry run and piping.
Raymond Toy 2017/05/10 21:28:43 Add a note that you need to inspect result.txt bec
hongchan 2017/05/11 21:13:01 Done.
30 node layout-test-tidy ${TARGET_PATH} --dryrun > result.txt
Raymond Toy 2017/05/10 21:28:43 I would really like it if layout-test-tidy would r
hongchan 2017/05/11 21:13:01 Make sense, but it sounds like a big change for th
31
32 # Sample command for LayoutTests/webaudio.
33 node layout-test-tidy ${CHROMIUM_SRC}/third_party/WebKit/LayoutTests/webaudio
34 ```
35
36 - If there is a file to be skipped, simply add its absolute file path to |skip -tidy| file.
Raymond Toy 2017/05/10 21:28:43 Do we really want to support a skip-tidy file? I'm
hongchan 2017/05/11 21:13:01 If you want to leave out certain files for some re
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698