921686 - Firefox css inspector doesn't find the correct url for some images.

Status: RESOLVED FIXED

(DevTools :: Inspector, defect)

Description

[Luke]

User Agent: Mozilla/5.0 (Windows NT 6.1; rv:24.0) Gecko/20100101 Firefox/24.0 (Beta/Release)
Build ID: 20130910160258

Steps to reproduce:

Go to http://subtlepatterns.com/thumbnail-view/ for example, try to inspect and see the url for one of the texture images.


Actual results:

You can see the background-image: url("/patterns/wild_flowers.png"); in the element, but clicking the link sends you to chrome://browser/patterns/wild_flowers.png


Expected results:

Chrome sends to the correct url, and if you use Firefox with my devtools-tweaks addon (https://addons.mozilla.org/en-US/firefox/addon/devtools-tweaks/) you can at least get to it by right clicking url( and selecting open in new tab. Firefox should go to the correct url by default though.

Comment 1

[Loic]

WFM, I see the complete URL: http://i.imgur.com/MYUbkBj.jpg

Can you test in safe mode, please:
https://support.mozilla.org/en-US/kb/troubleshoot-firefox-issues-using-safe-mode

Comment 2

[Luke]

@Loic
What version of Firefox shows the whole url? That is incorrect, since that image was declared on the page with

url('/patterns/geometry.png')

as you can see in view-source. but that would be a different bug.

Comment 3

[Luke]

I'm seeing the same behavior in safe mode, and in nightly. The link in the right panel doesn't take you to the image.

Comment 4

[Brian Grinstead [:bgrins]]

I am seeing the problem in nightly as well.

Comment 5

[Brian Grinstead [:bgrins]]

Attachment: style-url.patch

This was an issue with a path needing to be resolved when stored in an inline style attribute.  Using rule.domRule.href instead of rule.sheet.href, which fixes the problem (as the server fakes the href property on attributes.

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/integration/fx-team/rev/335e5804cbf1

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/335e5804cbf1

Comment 8

[Luke]

Thank you! - but does that mean this will only be fixed in Firefox 27?

Comment 9

[Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe]]

You can check here:
https://wiki.mozilla.org/RapidRelease/Calendar

The code will be merged on 28 Oct 2013 into Firefox 28.

Comment 10

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

Huh?  Why Firefox 28?  It landed in m-c, so it's part of 27, which becomes the stable release on 2014-01-21.