56164 - Initial position of platform scrollbar incorrect

Status: VERIFIED FIXED

(Core :: Web Painting, defect, P3)

Description

[Jeffrey S. Kobal]

This shows up as a problem on OS/2, but it appears to be a general bug that
doesn't affect behavior on other OSes.

On the initial drop-down of a view with a platform-specific scrollbar, the
scrollbar position is incorrect.  After the drop-down is closed and re-opened,
the scrollbar position is correct.  I have seen this on the "Editions" pop-up
window from the CNN web site, when dropping down the "Select your location"
option box.

The problem is that nsScrollingView::SetDimensions sets the new bounds of the
clipView, but does not update the scrollbars accordingly.  The current code in
view/src/nsScrollingView.cpp is:

  // Size and position the clip view
  if (nsnull != mClipView)
    mClipView->SetBounds(clipRect, aPaint);

The code should be changed to be:

  // Size and position the clip view
  if (nsnull != mClipView) {
    mClipView->SetBounds(clipRect, aPaint);
    UpdateScrollControls(aPaint);
  }

This will recalculate the size and position of the scrollbars relative to the
new size of the view.

Comment 1

[Mike Kaply [:mkaply]]

we're looking for an r= on this...

Comment 2

[Kevin McCluskey (gone)]

r=[e-mail]

Comment 3

[Robert O'Callahan (:roc) (email my personal email if necessary)]

sr=[e-mail]

Comment 4

[Mike Kaply [:mkaply]]

Fix checked in.

Comment 5

[Mike Kaply [:mkaply]]

Verified