Chromium Code Reviews| OLD | NEW |
|---|---|
| (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
| |
| OLD | NEW |