342072 - User cannot grab splitter to resize columns in tree, until columns have been moved

Status: RESOLVED FIXED

(Core :: XUL, defect)

Description

[Alex Vincent [:WeirdAl]]

Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a1) Gecko/20060619 SeaMonkey/1.5a

Steps to reproduce:
(1) Launch DOM Inspector, and have it inspect itself.
(2) Right-hand viewer, switch to JavaScript Object panel.
(3) Attempt to resize the Property column in the JS Object panel.

Expected results: Panel resizes.
Actual results:  Mouse cannot pick up the column splitter.

This is working in Firefox 1.5.0.4.  This works in SeaMonkey if you first select the column as if you are going to move the column around.

Comment 1

[Alex Vincent [:WeirdAl]]

Also WFM Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.0.4) Gecko/20060516 SeaMonkey/1.0.2

Comment 2

[Brian Polidoro]

In firefox I found this isn't in the latest 1.8.1 branch build either.  As for trunk:
works  2006051807
broken 2006051905

http://bonsai.mozilla.org/cvsquery.cgi?treeid=default&module=all&branch=HEAD&branchtype=match&dir=&file=&filetype=match&who=&whotype=match&sortby=Date&hours=2&date=explicit&mindate=2006-05-18+07%3A00&maxdate=2006-05-19+04%3A00&cvsroot=%2Fcvsroot

Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a1) Gecko/20060519 Minefield/3.0a1 ID:2006051905 [cairo]

Comment 3

[Brian Polidoro]

Oops my query was a little off and I think it makes all the difference here:
http://bonsai.mozilla.org/cvsquery.cgi?treeid=default&module=all&branch=HEAD&branchtype=match&dir=&file=&filetype=match&who=&whotype=match&sortby=Date&hours=2&date=explicit&mindate=1147960800&maxdate=1148040000&cvsroot=%2Fcvsroot

Bug 330236?

Comment 4

[Adam Guthrie]

Yeah, that looks like it. Thanks for the regression range, Brian.

Comment 5

[[e-mail]]

Attachment: Proposed patch

I discovered that _ensureColumnOrder was getting a tree-less columns object.

Comment 6

[[e-mail]]

Doh, my backup mx server ate both the r+ and sr+ mails :-(

Fix checked in, eventually.