Return a Vec from EventTarget::get_listeners_for

This commit is contained in:
Anthony Ramine 2016-02-29 10:15:55 +01:00 committed by Ms2ger
parent 590316a468
commit fc2cf31d5a
2 changed files with 26 additions and 30 deletions

View file

@ -67,7 +67,7 @@ fn dispatch_to_listeners(event: &Event, target: &EventTarget, chain: &[&EventTar
/* capturing */
event.set_phase(EventPhase::Capturing);
for cur_target in chain.iter().rev() {
if let Some(listeners) = cur_target.get_listeners_for(&type_, Some(ListenerPhase::Capturing)) {
let listeners = cur_target.get_listeners_for(&type_, Some(ListenerPhase::Capturing));
event.set_current_target(cur_target);
for listener in &listeners {
handle_event(window.r(), listener, *cur_target, event);
@ -81,7 +81,6 @@ fn dispatch_to_listeners(event: &Event, target: &EventTarget, chain: &[&EventTar
return;
}
}
}
assert!(!event.stop_propagation());
assert!(!event.stop_immediate());
@ -90,8 +89,7 @@ fn dispatch_to_listeners(event: &Event, target: &EventTarget, chain: &[&EventTar
event.set_phase(EventPhase::AtTarget);
event.set_current_target(target);
if let Some(listeners) = target.get_listeners_for(&type_, None) {
for listener in listeners {
for listener in target.get_listeners_for(&type_, None) {
handle_event(window.r(), &listener, target, event);
if event.stop_immediate() {
@ -101,7 +99,6 @@ fn dispatch_to_listeners(event: &Event, target: &EventTarget, chain: &[&EventTar
if event.stop_propagation() {
return;
}
}
assert!(!event.stop_propagation());
assert!(!event.stop_immediate());
@ -113,7 +110,7 @@ fn dispatch_to_listeners(event: &Event, target: &EventTarget, chain: &[&EventTar
event.set_phase(EventPhase::Bubbling);
for cur_target in chain {
if let Some(listeners) = cur_target.get_listeners_for(&type_, Some(ListenerPhase::Bubbling)) {
let listeners = cur_target.get_listeners_for(&type_, Some(ListenerPhase::Bubbling));
event.set_current_target(cur_target);
for listener in &listeners {
handle_event(window.r(), listener, *cur_target, event);
@ -128,7 +125,6 @@ fn dispatch_to_listeners(event: &Event, target: &EventTarget, chain: &[&EventTar
}
}
}
}
// See https://dom.spec.whatwg.org/#concept-event-dispatch for the full dispatch algorithm
pub fn dispatch_event(target: &EventTarget, pseudo_target: Option<&EventTarget>,

View file

@ -304,8 +304,8 @@ impl EventTarget {
pub fn get_listeners_for(&self,
type_: &Atom,
desired_phase: Option<ListenerPhase>)
-> Option<Vec<CompiledEventListener>> {
self.handlers.borrow_mut().get_mut(type_).map(|listeners| {
-> Vec<CompiledEventListener> {
self.handlers.borrow_mut().get_mut(type_).map_or(vec![], |listeners| {
listeners.get_listeners(desired_phase, self, type_)
})
}