557788 - Tree with both rows- and flex-attributes set leads to 100% CPU load

Status: RESOLVED DUPLICATE of bug 541804

(Core :: XUL, defect)

Description

[Edwin Martin]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 6.1; nl; rv:1.9.2.3) Gecko/20100401 Firefox/3.6.3 GTBDFff GTB7.0 (.NET CLR 3.5.30729)
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 6.1; nl; rv:1.9.2.3) Gecko/20100401 Firefox/3.6.3 GTBDFff GTB7.0 (.NET CLR 3.5.30729)

In a XUL-file with both rows- and flex-attributes set, the CPU-load will become 100%.

Example:

<?xml version="1.0"?>
<overlay xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul">
  <vbox>
    <description class="header">This is a test page reproducing 100% CPU. Click the "Name"-header and watch your CPU load.</description>
	  <tree rows="6" seltype="single" flex="1">
		<treecols>
		  <treecol label="Name" flex="2" minwidth="50"/>
		</treecols>
		<treechildren/>
	  </tree>
  </vbox>
</overlay>

Reproducible: Always

Steps to Reproduce:
1. Load the following XUL-file in Firefox 3.6 in Windows (7):

<?xml version="1.0"?>
<overlay xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul">
  <vbox>
    <description class="header">This is a test page reproducing 100% CPU. Click the "Name"-header and watch your CPU load.</description>
	  <tree rows="6" seltype="single" flex="1">
		<treecols>
		  <treecol label="Name" flex="2" minwidth="50"/>
		</treecols>
		<treechildren/>
	  </tree>
  </vbox>
</overlay>

2. Open the CPU-monitor (press CTRL-SHIFT-ESC).
3. Click the Name-header from the XUL-file.
4. Watch the CPU-load go to 100%.
Actual Results:  
CPU load became 100%.

Expected Results:  
Normal CPU-load.

The bug became apparent in View Cookies:
https://addons.mozilla.org/nl/firefox/addons/versions/315#version-1.9.2

Comment 1

[Boris Zbarsky [:bzbarsky]]

I think we have bugs on file on this already...

Comment 2

[Edwin Martin]

Attachment: XUL-file triggering bug

This is the example file.

Comment 3

[Edwin Martin]

Ah, I didn't search well enough. Seems duplicate of #541804.