Bug 21750

Summary:

Eliminate the notion of "temporary events"

Product:

WebKit

Reporter:

Alexey Proskuryakov <ap>

Component:

WebCore Misc.

Assignee:

Alexey Proskuryakov <ap>

Status:

RESOLVED FIXED

Severity:

Normal

Priority:

Version:

528+ (Nightly build)

Hardware:

Mac

OS:

OS X 10.5

Attachments:

Description	Flags
proposed patch	darin: review+

Description Alexey Proskuryakov 2008-10-20 06:52:53 PDT

We mark some events as temporary when dispatching, for the sole reason of manually calling forgetDOMObject when done dispatching.

There doesn't seem to be any reason to call it manually, as JSEvent destructor will do this automatically right away.

Patch forthcoming. Please let me know why I'm wrong :-)

Comment 1 Alexey Proskuryakov 2008-10-20 06:56:56 PDT

Created attachment 24521 [details]
proposed patch

Comment 2 Darin Adler 2008-10-20 08:02:31 PDT

Comment on attachment 24521 [details]
proposed patch

r=me

Comment 3 Alexey Proskuryakov 2008-10-20 08:15:01 PDT

Committed revision 37718 and 37719.