Bug 22278 - scrollToOffset should not relayout, repaint, or emit DOM event if not scrolled
Summary: scrollToOffset should not relayout, repaint, or emit DOM event if not scrolled
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac OS X 10.5
: P2 Normal
Assignee: George Staikos
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-11-15 04:29 PST by George Staikos
Modified: 2008-11-16 04:46 PST (History)
1 user (show)

See Also:

Attachments
patch to implement early exit (1.22 KB, patch)
2008-11-15 04:30 PST, George Staikos 		timothy: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description George Staikos 2008-11-15 04:29:34 PST 
scrollToOffset does a lot of work it doesn't seem to need to do if already scrolled to the location requested.  It should not relayout, repaint, or emit DOM event if not scrolled.
Comment 1 George Staikos 2008-11-15 04:30:39 PST 
Created attachment 25184 [details]
patch to implement early exit
Comment 2 George Staikos 2008-11-16 04:46:01 PST 
Landed r38445