Bug 65784 – Web Inspector: Network panel always displays file:// resources as "(pending)"

• Status: RESOLVED
• Resolution: FIXED
• Component: Web Inspector (Deprecated)

[Andrey Kosyakov]

1. Open inspector
2. Switch to the Network panel
3. Navigate to a file:// url
3. Observe that all file URLs have status of "(pending)"

[Andrey Kosyakov]

Created attachment 103095 [details]
patch

- report status for non-http(s?) resources that are finished and not failed as "Success"
- "(Pending)" -> "(pending)" for consistency with other bogus status values

[Pavel Feldman]

(you should not regress data url behavior).
btw: a test for file: and data: would be great.

[Andrey Kosyakov]

Created attachment 103239 [details]
patch

- added a test

note that data URL behavior is not actually regressed: we used to have "(data url)" as a status for data URLs, now it's just "Success".

[Pavel Feldman]

Comment on attachment 103239 [details]
patch

View in context: https://bugs.webkit.org/attachment.cgi?id=103239&action=review

> LayoutTests/inspector/network-status-non-http.html:5
> +test = function() {

function test() {

> LayoutTests/inspector/network-status-non-http.html:14
> +        var dataGrid = logView.element.querySelector("table.data");

what is dataGrid.textContent ?

> LayoutTests/inspector/network-status-non-http.html:44
> +function addElement(src)

addScriptElement

> LayoutTests/inspector/network-status-non-http.html:48
> +    document.body.appendChild(element);

document.head.appendChild

[WebKit Review Bot]

Comment on attachment 103239 [details]
patch

Attachment 103239 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/9330533

New failing tests:
inspector/network-status-non-http.html

[Andrey Kosyakov]

Manually committed r92755: http://trac.webkit.org/changeset/92755