323329 - toolbarbutton and type radio

Status: RESOLVED WORKSFORME

(Firefox :: General, defect)

Description

[Willy also]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9a1) Gecko/20051110 Firefox/1.6a1
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9a1) Gecko/20051110 Firefox/1.6a1

define the following

<toolbarbutton label="XXX" type="radio" group="thegroup" />
<toolbarbutton label="YYY" type="radio" group="thegroup" />

press one of the buttons.
Mozilla goes into an infinite recursion and stays there till the user refreshes the document. The browser slows down for 60%. You may test it with other items on the same document like text- or listboxes.

This bug is somehow related to the groupproblem which I already described in bug with ID 322377. Please have a look at this bug, cause we are running into problems to realize our platformindependent XUL GUI.

Thank you
Willy Also

Reproducible: Always

Comment 1

[Martijn Wargers (dead)]

Attachment: testcase

Nothing bad happens for me when clicking on one of the buttons, or is my testcase wrong?

Comment 2

[Willy also]

I would like to add an a Attachment, but it seems like I'm not allowed todo it.
For testing, you can also use the <label id="LABMESS", but the HTMLMESS works better. Just click on a toolbarbutton and then play around with the listboxes. This listboxes are just for testing, you can 'feel' how the system is going slower. It has todo with the radiogroup "thegroup" and the output in ittOnSelItem(). This radiogroup is just an example, the problem appears at all groups we created with XBL. A 1Ghz PC with only 3 XUL-documents and some resourceintensive items on it like trees, listitems or textboxes, works unacceptable slow cause of this problem.

greetings
also

<window title="Bug 323329 - toolbarbutton and type radio"
	xmlns:html="http://www.w3.org/1999/xhtml"
	xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul">

<script>
<![CDATA[
function ittOnSelItem(event)
{
<!--	document.getElementById("LABMESS").value="ID: "+event.target.id; -->
	document.getElementById("HTMLMESS").innerHTML="ID: "+event.target.id;
}
]]>
</script>

<vbox flex="1">

<vbox>
	<label id="LABMESS" value="Message" />
	<html:pre id="HTMLMESS">Message</html:pre>
</vbox>

<vbox>
<toolbarbutton label="XXX" type="radio" group="thegroup"/>
<toolbarbutton label="YYY" type="radio" group="thegroup"/>
<toolbarbutton label="ZZZ" type="radio" group="thegroup"/>
</vbox>


<vbox>
<groupbox>
<caption label="LBX_SGP01"/>
<listbox id="LBX_SGP01" rows="6" flex="1" onselect="ittOnSelItem(event);">
<listitem label="listitem0"/>
<listitem label="listitem1"/>
<listitem label="listitem2"/>
<listitem label="listitem3"/>
<listitem label="listitem4"/>
<listitem label="listitem5"/>
</listbox>
</groupbox>
</vbox>

<vbox>
<groupbox>
<caption label="LBX_SGP02"/>
<listbox id="LBX_SGP02" rows="6" flex="1" onselect="ittOnSelItem(event);">
<listitem label="listitem0"/>
<listitem label="listitem1"/>
<listitem label="listitem2"/>
<listitem label="listitem3"/>
<listitem label="listitem4"/>
<listitem label="listitem5"/>
</listbox>
</groupbox>
</vbox>
</vbox>
</window>

Comment 3

[Martijn Wargers (dead)]

Attachment: testcase2

Well, you can attach a testcase, using the "Create a New Attachment" link (don't forget to choose the mime-type).

This testcase also works for me, also when opening 3 instances of it and vigorously clicking inside it. I'm using a 800MHz Duron, on WinXP, using 20060113 trunk build.

Comment 4

[Willy also]

I will give you a more effective example over the weekend.

Comment 5

[Willy also]

Attachment: testcase 3

Press the toolbarbutton checkbox or one of the radioitems and test the performance of the listbox.
There must be:
1. Two <html:pre>Text</html:pre>
2. Set the content of the SECOND html:pre with innerHTML
3. If I set a '\n' newLine into the string, the problem does not occur.
4. The first <html:pre> must be, but it can be any text without changing it.
5. Only toolbarbuttons defined as radio or checkboxes bring the effect.
6. If this conditions are true, all my own and also some available widgets, if they have to process a lot of data, slow down and after a while they stuck.

Comment 6

[Martijn Wargers (dead)]

I haven't seen any slowness yet, I've had the testcase3 opened in 3 different tabs while clicking on the radioitems and scrolling in the listbox.

Comment 7

[Serge Gautherie (:sgautherie)]

Willy,

Can you reproduce with Firefox (v2.0.0.14 or) v3.0pre ?

Comment 8

[Willy also]

(In reply to comment #7)
> Willy,
> 
> Can you reproduce with Firefox (v2.0.0.14 or) v3.0pre ?
> 

v2.0.0.14
No

Comment 9

[Tyler Downer [He/Him]]

WFM based off of comment 8