ui/base/events/event_target.cc - Issue 10916210: events: Inlcude the event-phase and event-result in Event.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Unified Diff: ui/base/events/event_target.cc

Issue 10916210: events: Inlcude the event-phase and event-result in Event. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src

Patch Set: test Created 8 years, 3 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:

Index: ui/base/events/event_target.cc

diff --git a/ui/base/events/event_target.cc b/ui/base/events/event_target.cc

index be564e26aa9ac1509a87ba85991f408cec3a9d2a..fe515946bf295f0a27578e9c8626dbcc5294091f 100644

--- a/ui/base/events/event_target.cc

+++ b/ui/base/events/event_target.cc

@@ -37,8 +37,7 @@ void EventTarget::RemovePostTargetHandler(EventHandler* handler) {

}

void EventTarget::GetPreTargetHandlers(EventHandlerList* list) {

- EventTarget* target = this->GetParentTarget() ? this->GetParentTarget()

- : this;

+ EventTarget* target = this;

Ben Goodger (Google) 2012/09/10 18:07:14 I don't think you meant for this to sneak into thi

sadrul 2012/09/10 18:35:24 Whoops. Removed.

while (target) {

EventHandlerList::reverse_iterator it, rend;

for (it = target->pre_target_list().rbegin(),