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

Store Window.active_timers in a HashMap

Browse source 
A HashMap allows easily looking up a timer and canceling it in
Window.ClearTimeout.

Fixes #1477.
This commit is contained in:
Martin Robinson 2014-03-29 16:30:17 -07:00
parent 83c2d92397
commit 1a8a3cc271
2 changed files with 16 additions and 12 deletions
Download patch file Download diff file Expand all files Collapse all files

20
src/components/script/dom/window.rs
View file

@ -25,7 +25,7 @@ use js::jsval::JSVal;
use js::jsval::{NullValue, ObjectValue};
use js::JSPROP_ENUMERATE;
use collections::hashmap::HashSet;
use collections::hashmap::HashMap;
use std::cast;
use std::cmp;
use std::comm::Chan;
@ -79,7 +79,7 @@ pub struct Window {
location: Option<JS<Location>>,
navigator: Option<JS<Navigator>>,
image_cache_task: ImageCacheTask,
active_timers: ~HashSet<TimerHandle>,
active_timers: ~HashMap<i32, TimerHandle>,
next_timer_handle: i32,
priv extra: Untraceable
}
@ -116,8 +116,8 @@ impl Window {
impl Drop for Window {
fn drop(&mut self) {
self.extra.timer_chan.send(TimerMessageClose);
for handle in self.active_timers.iter() {
handle.cancel();
for timer_handle in self.active_timers.values() {
timer_handle.cancel();
}
}
}
@ -254,14 +254,16 @@ impl Window {
}));
}
});
self.active_timers.insert(TimerHandle { handle: handle, cancel_chan: Some(cancel_chan) });
self.active_timers.insert(handle, TimerHandle { handle: handle, cancel_chan: Some(cancel_chan) });
handle
}
pub fn ClearTimeout(&mut self, handle: i32) {
// FIXME(#1477): active_timers should be a HashMap and this should
// cancel the removed timer.
self.active_timers.remove(&TimerHandle { handle: handle, cancel_chan: None });
let timer_handle = self.active_timers.pop(&handle);
match timer_handle {
Some(handle) => handle.cancel(),
None => { }
}
}
pub fn damage_and_reflow(&self, damage: DocumentDamageLevel) {
@ -310,7 +312,7 @@ impl Window {
location: None,
navigator: None,
image_cache_task: image_cache_task,
active_timers: ~HashSet::new(),
active_timers: ~HashMap::new(),
next_timer_handle: 0
};