12860 - [DOGFOOD] [PP]nsIWindow::Scroll() - Scrolling on unix flashes too much

Status: VERIFIED FIXED

(Core :: XUL, defect, P3)

Description

[Ramiro Estrugo]

nsIWindow::Scroll() - Scrolling on unix flashes too much

The problem is the "guffaw" scrolling effect caused by the gtk layout widget.

You can see this on the top and bottom parts of the the scrolled region.

Comment 1

[ramiro]

*** Bug 6029 has been marked as a duplicate of this bug. ***

Comment 2

[leger]

Puttin on [PP] radar.

Comment 3

[ramiro]

*** Bug 9994 has been marked as a duplicate of this bug. ***

Comment 4

[Paul MacQuiddy]

giving me rest of phillips open qa contact bugs, sorry for spam

Comment 5

[Ramiro Estrugo]

Reassigning to blizzard.  Adding pavlov to cc.

This is a placeholder bug that i filed a long time ago, which blizzard is
working on now.

Comment 6

[Christopher Blizzard (:blizzard)]

I have a patch for this now.  Waiting for review and approval to check in.

Comment 7

[Christopher Blizzard (:blizzard)]

Attachment: Scrolling patch

Comment 8

[(not reading, please use [e-mail] instead)]

we got to get this in for m12.  this patch rocks.  it makes
5.0 scroll as fast as 4.x!

Comment 9

[Syd Logan]

Seems reasonably well thought out, although it would be nice to get rid of all
the list processing that goes on accumulating the translation queue (if in fact
that is possible). Regardless, looks like the gains are positive. r=syd

Comment 10

[Christopher Blizzard (:blizzard)]

The list operations in glib are pretty lightweight compared to the drawing
operations.  It's the server roundtrips and polling the queue that are painful.

Comment 11

[Christopher Blizzard (:blizzard)]

Fix checked in.

Comment 12

[Marko Macek]

On my machine dragging an xterm across 4.7 causing expose events uses about 50%
CPU (truecolor ATI). M11 uses 100% CPU and has terrible blinking. It seems that
the patch will fix the blinking (the no-background is for all windows, right?),
but I am not sure if it causes more than 2x slowdown.

Comment 13

[Paul MacQuiddy]

verified

Comment 14

[Christopher Blizzard (:blizzard)]

Please ignore the spam.  Changing address.

Comment 15

[Christopher Blizzard (:blizzard)]

bustage from my reassign

Comment 16

[Christopher Blizzard (:blizzard)]

bustage from my reassign