Bug 171901 – Web Inspector: Provide resource load error reason text in details sidebar

• Status: RESOLVED
• Resolution: FIXED
• Component: Web Inspector

[Joseph Pecoraro]

Summary:
Provide resource load error reason text in details sidebar

Sometimes it is not clear why a resource shows up in red in the Network Panel. One reason is that although there was a 200 response, the underlying load was cancelled and not all the data was loaded. Show the error reason in the sidebar alongside the status code, when there was an error.

Steps to Reproduce:
1. Open https://bug-171765-attachments.webkit.org/attachment.cgi?id=309257
2. Open inspector
3. Reload
  => Main resource is red, but status code is 200

[Joseph Pecoraro]

<rdar://problem/29850995>

[Joseph Pecoraro]

Created attachment 309566 [details]
[PATCH] Proposed Fix

[Joseph Pecoraro]

Created attachment 309567 [details]
[IMAGE] Example 1

[Joseph Pecoraro]

Created attachment 309568 [details]
[IMAGE] Example 2

[Joseph Pecoraro]

Created attachment 309569 [details]
[IMAGE] Example 3

[Blaze Burg]

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

r=me

Is any of the response text behavior of the backend covered by a test?

[WebKit Commit Bot]

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

Clearing flags on attachment: 309566

Committed r216564: <http://trac.webkit.org/changeset/216564>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Joseph Pecoraro]

> Is any of the response text behavior of the backend covered by a test?

No =(. I was going to try to write a test for this and totally forgot in the heat of looking into another issue. It might be possible to write a test for cancellations, but the text will likely differ across ports for issues happening at the networking level.