overflow (and thus scrolling) broken on comboboxes and listboxes (selects)




Layout: Form Controls
14 years ago
12 years ago


(Reporter: dbaron, Assigned: roc)


({regression, smoketest})

regression, smoketest

Firefox Tracking Flags

(Not tracked)



(1 attachment)



14 years ago
In my build from this evening, overflow on comboboxes and list boxes is broken.

Steps to reproduce:
 1. load this page
 2. examine "Target Milestone" dropdown
 3. load http://bugzilla.mozilla.org/enter_bug.cgi?product=Browser
 4. examine "Component" box

Actual results:
 2. Target milestone has no scrollbar and ends with mozilla0.8
 4. Component box ends with "Chatzilla" and scrollbar shows no room to scroll

Expected results:
 2. Target milestone dropdown is scrollable and shows recent milestones.
 4. Component box is scrollable and shows lots of components, many of which
start with "X"

I blame the aaronl + roc checkin for outlines that I reviewed.

Comment 1

14 years ago
(It's bug 151375 that I'm blaming.)

Comment 2

14 years ago
(although without proof)
I agree that it's probably us...

Comment 4

14 years ago
*** Bug 251558 has been marked as a duplicate of this bug. ***
Okay, the problem is right here:


This is being called on the nsBoxToBlockAdaptor for the nsAreaFrame holding the
options, and that's wiping out the overflow area of the nsAreaFrame.

My solution is to remove the FinishAndStoreOverflow from there and pass that
burden to the callers. Most callers of SetBounds will eventually do a Layout or
SyncLayout on the same box that will compute the overflow area, so they're taken
care of. Some other callers are just moving the box, so the overflow area
doesn't need to change. The remaining places are just a couple of places that
want to remove any overflow area because they're trying to collapse the box or
have somehow taken the overflow area into account already. I'll attach a patch.

Comment 6

14 years ago
i just did this:
cvs update -j1.15 -j1.14

and it cured the regression. (a checkin yesterday for bug 250553)
Are you SURE? that seems incredibly surprising to me

Comment 8

14 years ago
backed out in one tree and listboxes worked again - i thought - but i was not
able to repeat that in a second tree. False alarm - sorry.


14 years ago
Summary: overflow broken on comboboxes and listboxes → overflow (and thus scrolling) broken on comboboxes and listboxes (selects)

Comment 9

14 years ago
Is Bug 251586 a dupe of this one? This also happends in latest Firefox branch
build and suite 1.7.1 with classic.

Comment 10

14 years ago
Created attachment 153320 [details] [diff] [review]

This should fix it. The patch does what I mentioned before to fix SetBounds. It
also catches a situation where we try to recompute the overflow area of a
boxToBlockAdaptor which doesn't work because it has no box children.
Assignee: nobody → roc

Comment 12

14 years ago
*** Bug 251684 has been marked as a duplicate of this bug. ***
This was fixed by backing out the outline patch.
Last Resolved: 14 years ago
Resolution: --- → FIXED

Comment 14

14 years ago
*** Bug 251817 has been marked as a duplicate of this bug. ***

Comment 15

14 years ago
*** Bug 252701 has been marked as a duplicate of this bug. ***
You need to log in before you can comment on or make changes to this bug.