Bug 148096

Summary:

Web Inspector: Show page source does not work for web archive

Product:

WebKit

Reporter:

Daniel Bates <dbates>

Component:

Web Inspector

Assignee:

Nobody <webkit-unassigned>

Status:

NEW

Severity:

Normal

CC:

graouts, inspector-bugzilla-changes, webkit-bug-importer

Priority:

Keywords:

InRadar

Version:

WebKit Nightly Build

Hardware:

All

OS:

All

Attachments:

Description	Flags
nightly.webkit.org Web Archive	none
Screen recording	none

Daniel Bates

Reported 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.

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
View All Add attachment proposed patch, testcase, etc.

Radar WebKit Bug Importer

Comment 1 2015-08-17 14:32:08 PDT

<rdar://problem/22314078>

Daniel Bates

Comment 2 2015-08-17 14:35:47 PDT

Created attachment 259187 [details] Screen recording Screen recording that demonstrates the issue.

Blaze Burg

Comment 3 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.

Blaze Burg

Comment 4 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.

Timothy Hatcher

Comment 5 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.

Timothy Hatcher

Comment 6 2015-08-25 20:01:33 PDT

<rdar://problem/6420783>

Note You need to log in before you can comment on or make changes to this bug.