395099 - src attribute doesn't work in statusbarpanel object, regression in FFx3

Status: RESOLVED FIXED

(Toolkit :: UI Widgets, defect)

Description

[Ben Hsieh]

This is a regression in FFx3, doesn't happen in 2.0.0.6

The image displays correctly if you use

> style="list-style-image: url('chrome://qa/content/qmo-16px.png')"

but not if you use
> src="chrome://qa/content/qmo-16px.png"

Both the style and the src work in 2.0.0.6. Testcase attached, unzip into your extension directory.

Comment 1

[Ben Hsieh]

Attachment: testcase

Er, actually attached zip this time. Comment out one of the two <statusbarpanel> lines in browserOverlays.xul to see src vs. style functionality.

Comment 2

[:Gavin Sharp [email: [e-mail]]]

Neither 
<statusbarpanel class="statusbarpanel-iconic" src="chrome://browser/skin/Toolbar.png"/> nor 
<statusbarpanel class="statusbarpanel-iconic" image="chrome://browser/skin/Toolbar.png"/> seem to work in a current trunk build. Changing the inherits at http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/toolkit/content/widgets/general.xml&rev=1.20&mark=283#283 back to just "src" fixes it, which points to bug 377696.

Comment 3

[:Gavin Sharp [email: [e-mail]]]

Attachment: simplified testcase

Comment 4

[Neil Deakin]

Attachment: fix

Didn't seem to have much luck with bug 377696 :(

Comment 5

[Asaf Romano (gone)]

Comment on attachment 279895 [details] [diff] [review]
fix

should have caught that, ugh.

Comment 6

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 279895 [details] [diff] [review]
fix

a=bzbarsky