Open Bug 1154178 Opened 9 years ago Updated 2 years ago

Improve xul_selectcontrol.js test

Categories

(Toolkit :: UI Widgets, defect)

defect

Tracking

()

Tracking Status
firefox40 --- affected

People

(Reporter: Ms2ger, Unassigned)

References

Details

::: toolkit/content/tests/chrome/xul_selectcontrol.js
@@ +132,5 @@
>    test_nsIDOMXULSelectControlElement_States(element, testid + "removeItemAt", 1,
>          selectionRequired ? seconditem : null, selectionRequired ? 0 : -1,
>          selectionRequired ? secondvalue : "");
>  
> +  is(removeditem.control, undefined, testid + "control not set");

From all the examination of instances of "control" (with quotes) in toolkit/content/widgets, I can't find a reason for this to ever be the case (let alone *always* be the case).  Please file a followup and investigate the circumstances under which this happens.

(Note that because removeditem is firstitem from line 131, and firstitem had a control a ways up in this file, this does *not* appear to just be a totally non-existent property detection.  What exactly is wrong here, I wouldn't know without some debugging beyond visual inspection.)
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.