135436 - (osx classic) when resizing <FRAME>, scrollbars don't move

Status: VERIFIED FIXED

(Core :: XUL, defect)

Description

[Mike Pinkerton (not reading bugmail)]

go to warp (or any page with frames and scrollbars in each frame). Grab the
resizer in the middle and resize the frames. Notice the scrollbars remain
stationary as the frames resize.

gecko is not telling the nsNativeScrollbarFrame to reflow for some reason (my
EndLayout() routine is never called). I'm not familiar enough with box reflow to
know why. clicking the up/down arrows to cause a scroll seems to force a reflow
that fixes the scrollbar position. resizing the entire window also fixes it.

Comment 1

[Mike Pinkerton (not reading bugmail)]

Attachment: use a view tied to the widget to make things move correctly

instead of moving things around ourselves, let gecko do it by tying our widget
to a view. simplifies the code _a lot_.

needing r/sr.

Comment 2

[David Hyatt]

Comment on attachment 77735 [details] [diff] [review]
use a view tied to the widget to make things move correctly

Excellent, young Skywalker. :)

sr=hyatt

Comment 3

[Ben Goodger (use ben at mozilla dot org for email)]

Comment on attachment 77735 [details] [diff] [review]
use a view tied to the widget to make things move correctly

r=[e-mail]

Comment 4

[Asa Dotzler [:asa]]

Comment on attachment 77735 [details] [diff] [review]
use a view tied to the widget to make things move correctly

a=asa (on behalf of drivers) for checkin to the 1.0 trunk

Comment 5

[Frank Tang]

please tell us is this only happen WHEN RESIZING and it refresh after you
release the button, or the problem still there after you finish the refresh?
can you make a screenshot ?

Comment 6

[David Hyatt]

The problem is there after you finish a refresh because the widget never 
got moved.  The code in this patch is actually a simplification, since it now 
uses Gecko's built-in code for automatically keeping widgets in sync with 
their corresponding layout frames.

Comment 7

[Mike Pinkerton (not reading bugmail)]

the problem is still there after you resize the frame. the scrollbar is gone.
you need this fix.

Comment 8

[Mike Pinkerton (not reading bugmail)]

landed

Comment 9

[Paul Wyskoczka]

v