98327 - The <getter> and <setter> functions only work in the 1st binding

Status: VERIFIED INVALID

(Core :: XBL, defect)

Description

[Wolfgang Grinfeld]

From Bugzilla Helper:
User-Agent: Mozilla/4.0 (compatible; MSIE 5.5; Windows NT 4.0; Hotbar 2.0)
BuildID:    All versions

In case one has an html having elements bound to XBL id's, and 
if the used XBL element uses <getter> or <setter> functions against a 
<property> , then only the 1st html bound element can activate the relevant 
getter and setter functions.
See Testcase.


Reproducible: Always
Steps to Reproduce:
1.See Testcase
2.
3.

Actual Results:  See testcase

Expected Results:  the getter and setter functions should always get executed, 
not just for the first occurence of the bound element

This bug was originally reported as bug 98143, but as David Hyatt took 
exception to the provided testcase, it has been rewritten.

Comment 1

[Wolfgang Grinfeld]

Attachment: XBL demonstrating the bug (do not run this file - only view it)

Comment 2

[Wolfgang Grinfeld]

Attachment: CSS used in the testcase (do not run this file - only view it)

Comment 3

[Wolfgang Grinfeld]

Attachment: The Testcase: run this and push all buttons to see the bug

Comment 4

[David Hyatt]

You cannot mix a raw property value with a getter/setter.  A property is either
a field (defined using getter/setter) or it is a raw value.  It is not sensical
for it to be defined as both.  The getter has no way of referencing the raw
value that you're attempting to initialize the property with.  This is a
malformed XBL.

Remove the "null" raw value from the property definition and you'll see that
everything works fine.

This error illustrates that there should perhaps be a new tag introduced to
distinguish raw properties from getter/setter properties.

Comment 5

[David Hyatt]

Filed bug 98337 to add a new tag to XBL to distinguish raw props from
getter/setter props.

Comment 6

[Wolfgang Grinfeld]

The XBL example on http://www.xulplanet.com/tutorials/xultu/xblprops.html gave 
the impression that one could mix them, sorry.

Comment 7

[Ryan Pertusio]

marking verified