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+

Description Vitaly Repeshko 2009-09-16 06:16:48 PDT

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

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

Created attachment 39645 [details]
patch

Comment 2 Dimitri Glazkov (Google) 2009-09-16 07:14:09 PDT

Comment on attachment 39645 [details]
patch

r=me.

Comment 3 Dimitri Glazkov (Google) 2009-09-16 07:30:52 PDT

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

Comment 4 Darin Fisher (:fishd, Google) 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