Bug 22278

Summary:

scrollToOffset should not relayout, repaint, or emit DOM event if not scrolled

Product:

WebKit

Reporter:

George Staikos <staikos>

Component:

WebCore Misc.

Assignee:

George Staikos <staikos>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

yong.li.webkit

Priority:

Version:

528+ (Nightly build)

Hardware:

Mac

OS:

OS X 10.5

Attachments:

Description	Flags
patch to implement early exit	timothy: review+

George Staikos

Reported 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.

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 attachment proposed patch, testcase, etc.

George Staikos

Comment 1 2008-11-15 04:30:39 PST

Created attachment 25184 [details] patch to implement early exit

George Staikos

Comment 2 2008-11-16 04:46:01 PST

Landed r38445

Note You need to log in before you can comment on or make changes to this bug.