WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED WONTFIX
141266
Web Inspector: Support data URL SourceMaps for iOS 6 and iOS 7
https://bugs.webkit.org/show_bug.cgi?id=141266
Summary
Web Inspector: Support data URL SourceMaps for iOS 6 and iOS 7
Jonathan Wells
Reported
2015-02-04 14:58:52 PST
Right now iOS 6 and iOS 7 using the Remote Inspector won't show source maps with data URLs even though they could.
Attachments
Add attachment
proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1
2015-02-04 14:59:02 PST
<
rdar://problem/19721661
>
Jonathan Wells
Comment 2
2015-02-04 15:05:32 PST
SourceMapResource#sourceMapResourceLoadError will need to not throw an error, or be otherwise avoided when using the remote inspector on iOS 6 or 7, or else execution will stop.
Joseph Pecoraro
Comment 3
2015-02-04 17:26:29 PST
When possible, we should title radars based on their user impact, instead of the code change. The _loadAndParseSourceMap part is overly specific, in case this were to change.
Joseph Pecoraro
Comment 4
2015-02-04 17:27:01 PST
s/radars/bugs/ =)
Jonathan Wells
Comment 5
2015-02-04 20:26:14 PST
(In reply to
comment #2
)
> SourceMapResource#sourceMapResourceLoadError will need to not throw an > error, or be otherwise avoided when using the remote inspector on iOS 6 or > 7, or else execution will stop.
This is no longer a concern. An error is no longer thrown.
Timothy Hatcher
Comment 6
2015-02-18 14:51:14 PST
What needs done here?
Joseph Pecoraro
Comment 7
2015-02-18 15:23:39 PST
(In reply to
comment #6
)
> What needs done here?
SourceMapManager only loads SourceMap resources if NetworkAgent.loadResource exists (so it can load the resource through the inspected page). If that doesn't exist (e.g. iOS 6 and 7) we don't even try to load source maps. I'm not sure if it is worth the time investment at this point to implement something for iOS 6 and 7 here. I believe we used to do an XHR / synchronous load from the frontend, which won't work in all cases anyway.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug