Bug 62757

Summary:

Have Document keep track of whether scroll listeners are registered

Product:

WebKit

Reporter:

Simon Fraser (smfr) <simon.fraser>

Component:

New Bugs

Assignee:

Simon Fraser (smfr) <simon.fraser>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

ap, mihaip, sam, simon.fraser

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch	mitz: review+

Simon Fraser (smfr)

Reported 2011-06-15 14:40:06 PDT

Have Document keep track of whether scroll listeners are registered

Attachments
Patch (5.01 KB, patch) 2011-06-15 14:44 PDT, Simon Fraser (smfr)	mitz: review+	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Simon Fraser (smfr)

Comment 1 2011-06-15 14:44:51 PDT

Alexey Proskuryakov

Comment 2 2011-06-15 14:52:51 PDT

Comment on attachment 97365 [details] Patch Did you check that this works with both document.onscroll and window.onscroll?

Simon Fraser (smfr)

Comment 3 2011-06-15 14:56:50 PDT

Will check.

Simon Fraser (smfr)

Comment 4 2011-06-15 15:56:36 PDT

window.onscroll and document.onscroll both work fine. DOMWindow::addEventListener() calls document->addListenerTypeIfNeeded().

Simon Fraser (smfr)

Comment 5 2011-06-15 16:03:25 PDT

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