Summary: | Web Inspector: Formatter: pretty-print CSS using a Worker | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Devin Rousso <hi> | ||||||||
Component: | Web Inspector | Assignee: | Devin Rousso <hi> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | commit-queue, ews-watchlist, hi, inspector-bugzilla-changes, joepeck, timothy, webkit-bug-importer | ||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||
Version: | WebKit Nightly Build | ||||||||||
Hardware: | All | ||||||||||
OS: | All | ||||||||||
Bug Depends on: | |||||||||||
Bug Blocks: | 198806, 198807 | ||||||||||
Attachments: |
|
Description
Devin Rousso
2019-05-12 22:46:44 PDT
Created attachment 369695 [details]
Patch
This is probably missing some edge cases, and should have some tests, but it's a start :)
Attachment 369695 [details] did not pass style-queue:
ERROR: Source/WebInspectorUI/UserInterface/Workers/Formatter/CSSFormatter.js:64: Line contains single-quote character. [js/syntax] [5]
Total errors found: 1 in 6 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 369695 [details] Patch Attachment 369695 [details] did not pass win-ews (win): Output: https://webkit-queues.webkit.org/results/12176877 New failing tests: js/dom/custom-constructors.html Created attachment 369736 [details]
Archive of layout-test-results from ews213 for win-future
The attached test failures were seen while running run-webkit-tests on the win-ews.
Bot: ews213 Port: win-future Platform: CYGWIN_NT-10.0-17763-3.0.5-338.x86_64-x86_64-64bit
Created attachment 369944 [details]
Patch
Attachment 369944 [details] did not pass style-queue:
ERROR: Source/WebInspectorUI/UserInterface/Workers/Formatter/CSSFormatter.js:64: Line contains single-quote character. [js/syntax] [5]
Total errors found: 1 in 23 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 369944 [details] Patch Clearing flags on attachment: 369944 Committed r246178: <https://trac.webkit.org/changeset/246178> All reviewed patches have been landed. Closing bug. Comment on attachment 369944 [details]
Patch
Nice!
|