Bug 171901

Summary: Web Inspector: Provide resource load error reason text in details sidebar
Product: WebKit Reporter: Joseph Pecoraro <joepeck>
Component: Web InspectorAssignee: Joseph Pecoraro <joepeck>
Status: RESOLVED FIXED    
Severity: Normal CC: bburg, commit-queue, inspector-bugzilla-changes, joepeck, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: All   
OS: All   
Attachments:
Description Flags
[PATCH] Proposed Fix
none
[IMAGE] Example 1
none
[IMAGE] Example 2
none
[IMAGE] Example 3 none

Joseph Pecoraro
Reported 2017-05-09 18:09:41 PDT
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
Attachments
[PATCH] Proposed Fix (15.01 KB, patch)
2017-05-09 18:12 PDT, Joseph Pecoraro 		no flags
DetailsFormatted DiffDiff
[IMAGE] Example 1 (153.84 KB, image/png)
2017-05-09 18:12 PDT, Joseph Pecoraro 		no flags
Details
[IMAGE] Example 2 (14.41 KB, image/png)
2017-05-09 18:13 PDT, Joseph Pecoraro 		no flags
Details
[IMAGE] Example 3 (7.92 KB, image/png)
2017-05-09 18:13 PDT, Joseph Pecoraro 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
Joseph Pecoraro
Comment 1 2017-05-09 18:09:52 PDT
<rdar://problem/29850995>
Joseph Pecoraro
Comment 2 2017-05-09 18:12:25 PDT
Created attachment 309566 [details] [PATCH] Proposed Fix
Joseph Pecoraro
Comment 3 2017-05-09 18:12:47 PDT
Created attachment 309567 [details] [IMAGE] Example 1
Joseph Pecoraro
Comment 4 2017-05-09 18:13:06 PDT
Created attachment 309568 [details] [IMAGE] Example 2
Joseph Pecoraro
Comment 5 2017-05-09 18:13:22 PDT
Created attachment 309569 [details] [IMAGE] Example 3
Blaze Burg
Comment 6 2017-05-09 22:00:03 PDT
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 7 2017-05-09 22:28:58 PDT
Comment on attachment 309566 [details] [PATCH] Proposed Fix Clearing flags on attachment: 309566 Committed r216564: <http://trac.webkit.org/changeset/216564>
WebKit Commit Bot
Comment 8 2017-05-09 22:28:59 PDT
All reviewed patches have been landed. Closing bug.
Joseph Pecoraro
Comment 9 2017-05-10 10:54:12 PDT
> 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.
Note You need to log in before you can comment on or make changes to this bug.