[Webkit-unassigned] [Bug 37215] Web Inspector: be more explicit about resource loading errors
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Wed Apr 7 11:07:56 PDT 2010
https://bugs.webkit.org/show_bug.cgi?id=37215
Timothy Hatcher <timothy at hatcher.name> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #52752|review?, commit-queue? |review-
Flag| |
--- Comment #5 from Timothy Hatcher <timothy at hatcher.name> 2010-04-07 11:07:56 PST ---
(From update of attachment 52752)
> + String message = "Failed to load resource " + url;
> + if (status)
> + message += String::format(", status: %d", status);
> +
> + addMessageToConsole(OtherMessageSource, LogMessageType, ErrorMessageLevel, message, 0, url);
I think it might be best to do this in the front-end, then the message can be
localized. But that isn't as important.
The message could be clearer, something like:
Failed to load resource. The server responded with a status of %u.
Consider using format for both variations of the message instead of appending
strings. I don't think you need to include the URL in the message since the URL
is automatically included as meta data and shown as a link with the message.
> + void reportResourceError(const String& url, int status = 0);
Status should be unsigned.
> + // If status is an error, content is likely to be of an inconsistent type,
> + // as it's going to be an error message. We do not want to emit a warning
> + // for this, though, as this will already be reported as resource loading
> + // failure.
It is unfornanate to have a widow in comments like this. It is less annoying to
have longer lines than to have widows, IMHO.
> - resource.warnings = 0;
> - resource.errors = 0;
Why is this needed?
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list