Bug 193148 – Web Inspector: Add a Setting Toggle for Source Maps

• Component: Web Inspector
• Status: RESOLVED FIXED

[Joseph Pecoraro]

Add a Setting Toggle for Source Maps

[Joseph Pecoraro]

Created attachment 358338 [details]
[PATCH] Proposed Fix

[Devin Rousso]

Comment on attachment 358338 [details]
[PATCH] Proposed Fix

r=me, should we add a `console.assert(WI.settings.sourceMapsEnabled.value)` inside `WI.SourceMap` somewhere, for sanity's sake?  Do we want to do anything for the case where the user disables source maps after having already downloaded and shown them (e.g. in Resources)?

[Simon Fraser (smfr)]

If I'm a naive user debugging JS stuff on a web site, how do I know that I need to enable or disable source maps?

[Joseph Pecoraro]

(In reply to Devin Rousso from comment #2)
> Comment on attachment 358338 [details]
> [PATCH] Proposed Fix
> 
> r=me, should we add a `console.assert(WI.settings.sourceMapsEnabled.value)`
> inside `WI.SourceMap` somewhere, for sanity's sake?  Do we want to do
> anything for the case where the user disables source maps after having
> already downloaded and shown them (e.g. in Resources)?

We should eventually do something where when the user toggles source maps it goes and hides / shows source maps stuff, but for now a reload is sufficient.


(In reply to Simon Fraser (smfr) from comment #3)
> If I'm a naive user debugging JS stuff on a web site, how do I know that I
> need to enable or disable source maps?

JavaScript developers should be familiar with the concept of source maps. If they are debugging their own content they should know, if they are debugging random content then its just something unfamiliar they would have to come across. The Web Inspector UI indicates SourceMaps uniquely in a few ways, which may help them recognize it is different.

[Joseph Pecoraro]

> (In reply to Simon Fraser (smfr) from comment #3)
> > If I'm a naive user debugging JS stuff on a web site, how do I know that I
> > need to enable or disable source maps?
> 
> JavaScript developers should be familiar with the concept of source maps. If
> they are debugging their own content they should know, if they are debugging
> random content then its just something unfamiliar they would have to come
> across. The Web Inspector UI indicates SourceMaps uniquely in a few ways,
> which may help them recognize it is different.

In other words, learning about Source Maps is not much different than learning about some JS / CSS features you aren't familiar with.

[WebKit Commit Bot]

Comment on attachment 358338 [details]
[PATCH] Proposed Fix

Clearing flags on attachment: 358338

Committed r239635: <https://trac.webkit.org/changeset/239635>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Radar WebKit Bug Importer]

<rdar://problem/47055858>