Closed Bug 463958 Opened 14 years ago Closed 14 years ago
Event State Manager::Force View Update
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.
Attachment #347215 - Flags: approval1.9.1?
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 :(
Attachment #347215 - Flags: approval1.9.1? → approval1.9.1+
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Assignee: nobody → Olli.Pettay
Version: unspecified → Trunk
Fixed on trunk and 1.9.1
Will add some comment to status meeting wiki.
Target Milestone: --- → mozilla1.9.1b3
You need to log in before you can comment on or make changes to this bug.