Web Inspector: Audits (Image Dimensions): full image URLs are prefixed with the hosting page name

[Alexander Pavlov (apavlov)]

If there is an "Image Dimensions" audit violation and the image URL is a complete http://foo-type URL, it will be prefixed with the page name (like, home/http://foo) in the audit results.

[Alexander Pavlov (apavlov)]

Created attachment 54055 [details]
[PATCH] Suggested fix

[Joseph Pecoraro]

Comment on attachment 54055 [details]
[PATCH] Suggested fix

> +            var srcMatch = src.match(WebInspector.URLRegExp);
> +            if (!srcMatch) {

You don't use the match object, you're just testing. This can be:

  if (!WebInspector.URLRegExp.test(src)) {

[Alexander Pavlov (apavlov)]

Created attachment 54068 [details]
[PATCH] Comments addressed

[WebKit Commit Bot]

Comment on attachment 54068 [details]
[PATCH] Comments addressed

Clearing flags on attachment: 54068

Committed r58163: <http://trac.webkit.org/changeset/58163>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.