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

auto merge of #4443 : Ms2ger/servo/dispatch_event_with_target, r=Manishearth

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:
bors-servo 2014-12-19 04:33:46 -07:00
commit 11b27361c9
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 { pub trait EventTargetHelpers {
fn dispatch_event_with_target(self, fn dispatch_event_with_target(self,
target: Option<JSRef<EventTarget>>, target: JSRef<EventTarget>,
event: JSRef<Event>) -> bool; event: JSRef<Event>) -> bool;
fn dispatch_event(self, event: JSRef<Event>) -> bool; fn dispatch_event(self, event: JSRef<Event>) -> bool;
fn set_inline_event_listener(self, fn set_inline_event_listener(self,
@ -130,13 +130,13 @@ pub trait EventTargetHelpers {
impl<'a> EventTargetHelpers for JSRef<'a, EventTarget> { impl<'a> EventTargetHelpers for JSRef<'a, EventTarget> {
fn dispatch_event_with_target(self, fn dispatch_event_with_target(self,
target: Option<JSRef<EventTarget>>, target: JSRef<EventTarget>,
event: JSRef<Event>) -> bool { event: JSRef<Event>) -> bool {
dispatch_event(self, target, event) dispatch_event(self, Some(target), event)
} }
fn dispatch_event(self, event: JSRef<Event>) -> bool { 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, fn set_inline_event_listener(self,

2
components/script/script_task.rs
View file

@ -859,7 +859,7 @@ impl ScriptTask {
EventBubbles::DoesNotBubble, EventBubbles::DoesNotBubble,
EventCancelable::NotCancelable).root(); EventCancelable::NotCancelable).root();
let wintarget: JSRef<EventTarget> = EventTargetCast::from_ref(*window); let wintarget: JSRef<EventTarget> = EventTargetCast::from_ref(*window);
let _ = wintarget.dispatch_event_with_target(Some(doctarget), *event); let _ = wintarget.dispatch_event_with_target(doctarget, *event);
*page.fragment_name.borrow_mut() = final_url.fragment.clone(); *page.fragment_name.borrow_mut() = final_url.fragment.clone();