|Summary:||ContentSecurityPolicy::logToConsole should include line/column number and source location|
|Product:||WebKit||Reporter:||Joseph Pecoraro <joepeck>|
|Component:||Web Inspector||Assignee:||Devin Rousso <drousso>|
|Severity:||Normal||CC:||bfulgham, commit-queue, dbates, drousso, ews-watchlist, inspector-bugzilla-changes, joepeck, mkwst, timothy, webkit-bug-importer, wilander|
|Version:||528+ (Nightly build)|
Description Joseph Pecoraro 2013-04-09 16:14:13 PDT
ContentSecurityPolicy's security ConsoleMessages should include a column number alongside the line number. If the Web Inspector wants to jump to a specific place in source code it will need both to jump to an accurate place.
Comment 3 Brent Fulgham 2016-03-18 13:24:13 PDT
Dan: Did you do anything to address this in your various CSP changes? Otherwise, John should try to attack this soon.
Comment 4 Daniel Bates 2016-03-19 16:17:15 PDT
(In reply to comment #3) > Dan: Did you do anything to address this in your various CSP changes? No.
Comment 5 John Wilander 2016-03-21 12:11:22 PDT
Re-assigning to Daniel Bates since he is refactoring this code any way.
Comment 6 Joseph Pecoraro 2019-02-14 11:18:50 PST
TextPostition is used above this which has an m_line and m_column. Might be easy.
Comment 8 Joseph Pecoraro 2019-02-14 11:58:35 PST
Comment on attachment 362044 [details] Patch I think this needs a test, otherwise I'd have done the same thing myself! There is a test case attached to the bugzilla bug.
Comment 9 Daniel Bates 2019-02-14 12:12:30 PST
Comment on attachment 362044 [details] Patch I think this patch will improve some violations, but I don’t think this will improve much. The problem is that we may not have source information at all. Last I recall source information is only available during script execution and at parse time. Subresource loads tend to be scheduled outside these Windows. Prove me wrong, please!
Comment 10 Daniel Bates 2019-02-14 12:18:08 PST
I think I have a partial, likely outdated, patch for this with a boat load of tests. That’s why this bug is assigned to me :) unassigning and please take take it! Why do I have a boat load tests: I seem to recall under testing of log messsages. I will post once I get to my computer and assuming my memory is correct :D. Worst case, I have a boat load of tests for some other bug. Definitely have a boat load of something.
Comment 12 WebKit Commit Bot 2019-04-23 13:41:03 PDT
Comment on attachment 364694 [details] Patch Clearing flags on attachment: 364694 Committed r244563: <https://trac.webkit.org/changeset/244563>
Comment 13 WebKit Commit Bot 2019-04-23 13:41:05 PDT
All reviewed patches have been landed. Closing bug.