Mirrors/servo
1
0
Fork 0
mirror of https://github.com/servo/servo.git synced 2025-08-03 04:30:10 +01:00
Code Issues Projects Releases Packages Wiki Activity

Make the argument to dispatch_event_with_target non-optional.

Browse source 
The name of the method makes it clear it's supposed to be used with a target
override, so we might as well enforce that.
This commit is contained in:
Ms2ger 2014-12-19 11:34:11 +01:00
parent 7317518c17
commit bce3b172e7
2 changed files with 5 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

8
components/script/dom/eventtarget.rs
View file

@ -108,7 +108,7 @@ impl EventTarget {
pub trait EventTargetHelpers {
fn dispatch_event_with_target(self,
target: Option<JSRef<EventTarget>>,
target: JSRef<EventTarget>,
event: JSRef<Event>) -> bool;
fn dispatch_event(self, event: JSRef<Event>) -> bool;
fn set_inline_event_listener(self,
@ -130,13 +130,13 @@ pub trait EventTargetHelpers {
impl<'a> EventTargetHelpers for JSRef<'a, EventTarget> {
fn dispatch_event_with_target(self,
target: Option<JSRef<EventTarget>>,
target: JSRef<EventTarget>,
event: JSRef<Event>) -> bool {
dispatch_event(self, target, event)
dispatch_event(self, Some(target), event)
}
fn dispatch_event(self, event: JSRef<Event>) -> bool {
self.dispatch_event_with_target(None, event)
dispatch_event(self, None, event)
}
fn set_inline_event_listener(self,