Bug 148096 - Web Inspector: Show page source does not work for web archive
Summary: Web Inspector: Show page source does not work for web archive
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (show other bugs)
Version: WebKit Nightly Build
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2015-08-17 14:31 PDT by Daniel Bates
Modified: 2016-12-13 15:40 PST (History)
3 users (show)

See Also:


Attachments
nightly.webkit.org Web Archive (14.62 KB, application/x-webarchive)
2015-08-17 14:31 PDT, Daniel Bates
no flags Details
Screen recording (9.05 MB, video/quicktime)
2015-08-17 14:35 PDT, Daniel Bates
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel Bates 2015-08-17 14:31:16 PDT
Created attachment 259185 [details]
nightly.webkit.org Web Archive

Cannot view the source of the page from a web archive. Web Inspector shows an animated loading indicator and never renders the source of the page.

Ensure the Develop menu is enabled. Perform the following:

1. Open the attached archive in Safari.
2. Choose Develop > Show Page Source.

Then the page source is never shown in the opened Web Inspector window. Instead an animated loading indicator is shown.

I am using Safari Version 8.0.8 (10600.8.9, r188521) on OS 10.10.5 (14F27).

Once you become familiar with this issue then you will find that you can reproduce it with an arbitrary web archive.
Comment 1 Radar WebKit Bug Importer 2015-08-17 14:32:08 PDT
<rdar://problem/22314078>
Comment 2 Daniel Bates 2015-08-17 14:35:47 PDT
Created attachment 259187 [details]
Screen recording

Screen recording that demonstrates the issue.
Comment 3 BJ Burg 2015-08-17 15:32:57 PDT
It looks like we are not properly detecting the main frame resource, or it's not being sent, or something. Will investigate a bit.
Comment 4 BJ Burg 2015-08-17 15:50:14 PDT
The spinning is gone after landing an unrelated fix, and replaced with an error message . :| Next step is to dump protocol in order to find out what's being sent by the backend.
Comment 5 Timothy Hatcher 2015-08-18 17:16:13 PDT
This has been broken forever. There is an old Radar for this I will relate and dupe when I find it. The old behavior was that we showed the binary plist data as the source. Not good all around. I attempted a fix one and it might be attached to that old Radar.
Comment 6 Timothy Hatcher 2015-08-25 20:01:33 PDT
<rdar://problem/6420783>