[Webkit-unassigned] [Bug 37215] Web Inspector: be more explicit about resource loading errors

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=37215





--- Comment #2 from Andrey Kosyakov <caseq at chromium.org>  2010-04-07 10:38:02 PST ---
(In reply to comment #1)
> I agree we should provide some additional diagnostic information regarding
> resources that don't load - I just experienced this myself regarding a resource
> not be loaded from an app-cache-enabled page, for instance.
> 
> However, I dislike spamming the console.  Error bubble in the resource list is
> fine.  Perhaps a line in the Timeline (category "Loading") with diagnostic
> info.

My point is that a failure to load resource is supposed to be rare, but rather
significant event, so in my view reporting it shouldn't be considered as
spamming by most. Besides, currently we're very likely to produce "content type
mismatch" warning in console when receiving error response from server
(typically text/html) in response to many resource types (images, scripts,
stylesheets) -- so my patch would just replace it with more comprehensible
message. Also, I think it makes sense to produce it even in case resource
tracking is disabled, to give users opening inspector a quick hint on why
something on a page might have failed. Do you have a use case in mind that
where a resource loading failure is "normal" and would cause lots of messages
to be logged?

> 
> Where would be put the diagnostic info in the Resources panel?  Create a new
> "tab" (peer of Headers/Content) with the information?

In the current version of my patch, the errors cause a red bubble to appear
next to a resource icon -- I think this is pretty sufficient to give a quick
overview of failed resources. If the consensus is that we don't want this
logged to console, I propose we still leave these bubbles (just implement it
slightly differently). I wouldn't like to introduce any additional tabs for
that, as it would be counter-intuitive -- one would need to know to navigate
there (we already display failed status on a resource's headers tab).

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.