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+

Description Simon Fraser (smfr) 2011-06-15 14:40:06 PDT

Have Document keep track of whether scroll listeners are registered

Comment 1 Simon Fraser (smfr) 2011-06-15 14:44:51 PDT

Created attachment 97365 [details]
Patch

Comment 2 Alexey Proskuryakov 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?

Comment 3 Simon Fraser (smfr) 2011-06-15 14:56:50 PDT

Will check.

Comment 4 Simon Fraser (smfr) 2011-06-15 15:56:36 PDT

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

Comment 5 Simon Fraser (smfr) 2011-06-15 16:03:25 PDT

http://trac.webkit.org/changeset/88980