346274 - Slider horizontal scrollbar missing while resizing window

Status: RESOLVED WORKSFORME

(Core :: XUL, defect)

Description

[Ria Klaassen (not reading all bugmail)]

Steps to reproduce:
* Go to a site like http://www.rnw.nl/nieuws/nederlands.html#3292436
* Grab the right edge of the window and drag it smaller
* As soon as the horizontal scrollbar appears, you'll see that it has no slider while dragging
* When you release the slider the slider will show up

This is another bug with another cause than Bug 328040.

Regression between 1.9a1_2006051010 and 1.9a1_2006051022:
http://bonsai.mozilla.org/cvsquery.cgi?module=PhoenixTinderbox&date=explicit&mindate=2006-05-10+09%3A00&maxdate=2006-05-10+23%3A00

Bug 326273 might have caused this.

Comment 1

[Ria Klaassen (not reading all bugmail)]

(In reply to comment #0)
>
> * When you release the slider the slider will show up
> 
I meant: * When you release the mouse the slider will show up.

Comment 2

[Patrick]

The entire horizontal scrollbar fails to appear when reducing window size.
The bar appears on
http://forums.mozillazine.org/viewtopic.php?t=451065
but does not on http://forums.mozillazine.org/viewtopic.php?t=451065&postdays=0&postorder=asc&postsperpage=15&start=15
which is page two of the same topic.

Comment 4

[Hideo Saito]

A timer event request for resize reflow called from PresShell::ResizeReflow is ignored by nsBaseAppShell::NativeEventCallback because mProcessingNextNativeEvent is true. I think following code should be changed in order to start to dispatch the event.

nsBaseAppShell::NativeEventCallback()
{
...
  if (mProcessingNextNativeEvent) {
#if 0
    // XXX(darin): This causes a hefty Ts and Txul regression.  We need some
    // sort of solution to handle native event loops spun from native events,
    // but we should try to find something better.
    mProcessingNextNativeEvent = PR_FALSE;
    if (NS_HasPendingEvents(NS_GetCurrentThread()))
      OnDispatchedEvent(nsnull);
#endif
    return;
  }

Comment 5

[Hideo Saito]

Attachment: patch

Comment 6

[Ria Klaassen (not reading all bugmail)]

This was fixed somewhere in the first quarter of 2007.