|Summary:||Web Inspector: Add PrettyPrinter CSS tests|
|Product:||WebKit||Reporter:||Jonathan Wells <jonowells>|
|Component:||Web Inspector||Assignee:||Joseph Pecoraro <joepeck>|
|Severity:||Normal||CC:||burg, commit-queue, eric.carlson, glenn, graouts, jer.noble, joepeck, philipj, sergio, timothy, webkit-bug-importer|
|Version:||528+ (Nightly build)|
Description Jonathan Wells 2014-06-17 16:51:50 PDT
Comment 1 Joseph Pecoraro 2014-06-17 16:57:16 PDT
Comment 3 Joseph Pecoraro 2014-10-03 15:48:22 PDT
Created attachment 239246 [details] [PATCH] Proposed Fix
Comment 4 WebKit Commit Bot 2014-10-04 15:12:00 PDT
Comment on attachment 239246 [details] [PATCH] Proposed Fix Clearing flags on attachment: 239246 Committed r174323: <http://trac.webkit.org/changeset/174323>
Comment 5 WebKit Commit Bot 2014-10-04 15:12:04 PDT
All reviewed patches have been landed. Closing bug.
Comment 6 Brian Burg 2014-10-04 17:01:32 PDT
Why aren't these done as layout tests or tool tests? Will anyone remember to run the tests?
Comment 7 Joseph Pecoraro 2014-10-06 11:29:15 PDT
(In reply to comment #6) > Why aren't these done as layout tests or tool tests? Will anyone remember to run the tests? That is a good point. These could become layout tests. They only need to be run when we update CodeMirror, which is rare. I originally just added these as part of the tool because it was what I used for debugging and development, so it was easy and useful. We can move these.