third_party/WebKit/Source/core/events/EventTarget.h - Issue 1942723004: Change EventTarget callback APIs for add/RemoveEventListenerInternal.

Unified Diff: third_party/WebKit/Source/core/events/EventTarget.h

Issue 1942723004: Change EventTarget callback APIs for add/RemoveEventListenerInternal. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master

Patch Set: Fix gypi Created 4 years, 8 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View side-by-side diff with in-line comments

Download patch

« third_party/WebKit/Source/core/events/EventListenerMap.cpp ('K') | « third_party/WebKit/Source/core/events/EventListenerMap.cpp ('k') | third_party/WebKit/Source/core/events/EventTarget.cpp » ('j') | third_party/WebKit/Source/core/svg/SVGElement.cpp » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: third_party/WebKit/Source/core/events/EventTarget.h

diff --git a/third_party/WebKit/Source/core/events/EventTarget.h b/third_party/WebKit/Source/core/events/EventTarget.h

index 827faa27bc756f05a97cfcc0991302a38498cf99..347d4e6a742af7c170c61a466fa2efef4cd2dc39 100644

--- a/third_party/WebKit/Source/core/events/EventTarget.h

+++ b/third_party/WebKit/Source/core/events/EventTarget.h

@@ -117,9 +117,9 @@ public:

bool addEventListener(const AtomicString& eventType, EventListener*, const EventListenerOptionsOrBoolean&);

bool addEventListener(const AtomicString& eventType, EventListener*, EventListenerOptions&);

- bool removeEventListener(const AtomicString& eventType, EventListener*, bool useCapture = false);

- bool removeEventListener(const AtomicString& eventType, EventListener*, const EventListenerOptionsOrBoolean&);

- bool removeEventListener(const AtomicString& eventType, EventListener*, EventListenerOptions&);

+ bool removeEventListener(const AtomicString& eventType, const EventListener*, bool useCapture = false);

+ bool removeEventListener(const AtomicString& eventType, const EventListener*, const EventListenerOptionsOrBoolean&);

+ bool removeEventListener(const AtomicString& eventType, const EventListener*, EventListenerOptions&);

virtual void removeAllEventListeners();

DispatchEventResult dispatchEvent(Event*);

@@ -152,7 +152,9 @@ protected:

EventTarget();

virtual bool addEventListenerInternal(const AtomicString& eventType, EventListener*, const EventListenerOptions&);

- virtual bool removeEventListenerInternal(const AtomicString& eventType, EventListener*, const EventListenerOptions&);

+ virtual bool removeEventListenerInternal(const AtomicString& eventType, const EventListener*, const EventListenerOptions&);

+ virtual void addedEventListener(const AtomicString& eventType, const RegisteredEventListener&);

bokan 2016/05/03 15:52:46 Please add comments describing what these are for.

dtapuska 2016/05/03 20:53:04 Done.

+ virtual void removedEventListener(const AtomicString& eventType, const RegisteredEventListener&);

virtual DispatchEventResult dispatchEventInternal(Event*);

// Subclasses should likely not override these themselves; instead, they should subclass EventTargetWithInlineData.