463958 - Remove nsEventStateManager::ForceViewUpdate

Status: RESOLVED FIXED

(Core :: Widget, defect)

Description

[Olli Pettay [:smaug][[e-mail]]]

Attachment: proposed patch

With Steven's patch for bug 428070 scrolling the testcase is
pretty good. However now Shark shows 
nsEventStateManager::ForceViewUpdate in the profile when scrolling
the page using mousewheel.
I think we could just remove the method.
I tested normal pages and also about:config and other xul:trees in the UI.
Seems like bryner added that method year 2000 to fix some xul:tree problems.
Unfortunately the commit message doesn't contain link to the bug.
I guess the relevant bug is bug 43710 (no patches there :( ).

With this change and bug 428070
http://news.opensuse.org/2007/10/04/announcing-opensuse-103-gm/
becomes pretty fast to scroll even with mousewheel on OSX.

Comment 1

[Mike Beltzner [:beltzner, not reading bugmail]]

Comment on attachment 347215 [details] [diff] [review]
proposed patch

a191=beltzner

I think we'll want to really watch hard for unexpected functional regressions here, so when you land this, please make sure to mention it in the weekly development notes so we can all be aware. Just nervous about removing a forced view update that we know little about :(

Comment 2

[Olli Pettay [:smaug][[e-mail]]]

Fixed on trunk and 1.9.1

Comment 3

[Olli Pettay [:smaug][[e-mail]]]

Will add some comment to status meeting wiki.