RESOLVED FIXED Bug 238533
Web Inspector: Sources: the mapped file URL of a Response Local Override should be clickable 
https://bugs.webkit.org/show_bug.cgi?id=238533
Summary Web Inspector: Sources: the mapped file URL of a Response Local Override shou...
Devin Rousso
Reported 2022-03-29 18:31:32 PDT
clicking it should either open the file in the default editor or reveal it in the system file explorer
Attachments
[fast-cq] Patch (23.50 KB, patch)
2022-03-29 18:34 PDT, Devin Rousso 		pangle: review+
ews-feeder: commit-queue-
DetailsFormatted DiffDiff
[fast-cq] Patch (23.76 KB, patch)
2022-03-30 09:53 PDT, Devin Rousso 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Devin Rousso
Comment 1 2022-03-29 18:34:24 PDT
Created attachment 456084 [details] [fast-cq] Patch
Patrick Angle
Comment 2 2022-03-29 20:24:06 PDT
Comment on attachment 456084 [details] [fast-cq] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=456084&action=review rs=me with one suggestion – also the build failures on GTK and others look real. > Source/WebCore/ChangeLog:11 > + (WebCore::InspectorFrontendHost::selectFileExternally): Added. Just one overarching question throughout: can we consider naming this `revealFileExternally` (and changing all the other method names to match)? Perhaps that is too much Mac influence here, but in general I see this sort of action behind a "Reveal in [...]" menu item in apps. `select`, in my opinion, is somewhat ambiguous with showing a file chooser to let someone choose (or "select") a file.
Devin Rousso
Comment 3 2022-03-30 09:53:41 PDT
Created attachment 456133 [details] [fast-cq] Patch
EWS
Comment 4 2022-03-30 13:57:12 PDT
Committed r292120 (249033@main): <https://commits.webkit.org/249033@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 456133 [details].
Radar WebKit Bug Importer
Comment 5 2022-03-30 13:58:17 PDT
<rdar://problem/91065118>
Note You need to log in before you can comment on or make changes to this bug.