For extremely large CSS files, it can take a very long time for us to pretty-print it using CodeMirror. Rather than pretty-print using the main thread (e.g. CodeMirror), we should do the same as JavaScript and pretty-print off the main thread (e.g. a Worker).
<rdar://problem/36891532>
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!