RESOLVED FIXED 74181
Web Inspector: add "install source map" to JS source frame context menu. 
https://bugs.webkit.org/show_bug.cgi?id=74181
Summary Web Inspector: add "install source map" to JS source frame context menu.
Pavel Podivilov
Reported 2011-12-09 07:00:38 PST
Web Inspector: add "install source map" to JS source frame context menu.
Attachments
Patch (11.71 KB, patch)
2011-12-09 07:02 PST, Pavel Podivilov 		no flags
DetailsFormatted DiffDiff
Patch (12.23 KB, patch)
2011-12-12 03:42 PST, Pavel Podivilov 		pfeldman: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Pavel Podivilov
Comment 1 2011-12-09 07:02:18 PST
Created attachment 118572 [details] Patch
Pavel Feldman
Comment 2 2011-12-09 07:07:07 PST
Comment on attachment 118572 [details] Patch Is there a screenshot for this UI change?
Pavel Podivilov
Comment 3 2011-12-09 07:31:25 PST
(In reply to comment #2) > (From update of attachment 118572 [details]) > Is there a screenshot for this UI change? Currently, it's just a context menu item that appears when source map is auto detected. Do you think screenshot could be useful?
Pavel Feldman
Comment 4 2011-12-09 08:16:28 PST
Comment on attachment 118572 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=118572&action=review > Source/WebCore/inspector/front-end/JavaScriptSourceFrame.js:160 > + var installSourceMapLabel = WebInspector.UIString(WebInspector.useLowerCaseMenuTitles() ? "Install source map" : "Install Source Map"); Please add it to the localized strings list.
Pavel Feldman
Comment 5 2011-12-09 08:16:52 PST
> Currently, it's just a context menu item that appears when source map is auto detected. Do you think screenshot could be useful? It would be nice to see this kind of information in the ChangeLog.
Pavel Podivilov
Comment 6 2011-12-12 03:42:16 PST
Created attachment 118769 [details] Patch
Pavel Podivilov
Comment 7 2011-12-22 08:46:26 PST
Committed r103541: <http://trac.webkit.org/changeset/103541>
Note You need to log in before you can comment on or make changes to this bug.