Bug 29298

Summary:

[V8] Added operator==() to event listeners.

Product:

WebKit

Reporter:

Vitaly Repeshko <vitalyr>

Component:

WebCore Misc.

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

dglazkov

Priority:

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Attachments:

Description	Flags
patch	dglazkov: review+

Vitaly Repeshko

Reported 2009-09-16 06:16:48 PDT

[V8] Added operator==() to event listeners to fix build after http://trac.webkit.org/changeset/48402.

Attachments
patch (1.94 KB, patch) 2009-09-16 06:22 PDT, Vitaly Repeshko	dglazkov: review+	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Vitaly Repeshko

Comment 1 2009-09-16 06:22:30 PDT

Created attachment 39645 [details] patch

Dimitri Glazkov (Google)

Comment 2 2009-09-16 07:14:09 PDT

Comment on attachment 39645 [details] patch r=me.

Dimitri Glazkov (Google)

Comment 3 2009-09-16 07:30:52 PDT

Landed as http://trac.webkit.org/changeset/48418.

Darin Fisher (:fishd, Google)

Comment 4 2009-09-16 09:19:07 PDT

Comment on attachment 39645 [details] patch > + virtual bool operator==(const EventListener& other) { > + return this == &other; > + } nit: the opening "{" should be on the next line per webkit style

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