RESOLVED FIXED 156278
Web Inspector: Do not create CodeMirror color/gradient markers in JavaScript resources 
https://bugs.webkit.org/show_bug.cgi?id=156278
Summary Web Inspector: Do not create CodeMirror color/gradient markers in JavaScript ...
Joseph Pecoraro
Reported 2016-04-05 21:47:30 PDT
* SUMMARY Do not create CodeMirror color/gradient markers in JavaScript resources. * TEST var red = "foo"; * STEPS TO REPRODUCE 1. Inspect test page 2. Show JavaScript resource with a variable name that is a CSS color name 3. Hover variable name => Get color marker * NOTES Performance is poor. Creating markers for JavaScript resources doesn't make sense either. Logging time it took to generate markers on the JavaScript resources on apple.com: MARKERS 43ms MARKERS 807ms MARKERS 58ms MARKERS 285ms MARKERS 824ms MARKERS 134ms This can be completely eliminated.
Attachments
[PATCH] Proposed Fix (2.06 KB, patch)
2016-04-05 21:48 PDT, Joseph Pecoraro 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2016-04-05 21:47:41 PDT
<rdar://problem/25570404>
Joseph Pecoraro
Comment 2 2016-04-05 21:48:40 PDT
Created attachment 275751 [details] [PATCH] Proposed Fix
WebKit Commit Bot
Comment 3 2016-04-05 22:45:16 PDT
Comment on attachment 275751 [details] [PATCH] Proposed Fix Clearing flags on attachment: 275751 Committed r199095: <http://trac.webkit.org/changeset/199095>
WebKit Commit Bot
Comment 4 2016-04-05 22:45:21 PDT
All reviewed patches have been landed. Closing bug.
Joseph Pecoraro
Comment 5 2016-06-04 23:51:40 PDT
*** Bug 129629 has been marked as a duplicate of this bug. ***
Note You need to log in before you can comment on or make changes to this bug.