Auto merge of #8548 - Ms2ger:unused-tna, r=pcwalton

Remove the unused arguments to hit_test and mouse_over.

I don't think this code is called when there is no document element, but I
added assertions to make sure we notice in case I was wrong.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8548)
<!-- Reviewable:end -->
This commit is contained in:
bors-servo 2015-11-17 02:05:52 +05:30
commit 20d26853e1
3 changed files with 9 additions and 21 deletions

View file

@ -483,30 +483,19 @@ impl Document {
}
pub fn hit_test(&self, point: &Point2D<f32>) -> Option<UntrustedNodeAddress> {
let root = self.GetDocumentElement();
let root = match root.r() {
Some(root) => root,
None => return None,
};
let root = root.upcast::<Node>();
let address = match self.window.layout().hit_test(root.to_trusted_node_address(), *point) {
assert!(self.GetDocumentElement().is_some());
match self.window.layout().hit_test(*point) {
Ok(HitTestResponse(node_address)) => Some(node_address),
Err(()) => {
debug!("layout query error");
None
}
};
address
}
}
pub fn get_nodes_under_mouse(&self, point: &Point2D<f32>) -> Vec<UntrustedNodeAddress> {
let root = self.GetDocumentElement();
let root = match root.r() {
Some(root) => root,
None => return vec!(),
};
let root = root.upcast::<Node>();
match self.window.layout().mouse_over(root.to_trusted_node_address(), *point) {
assert!(self.GetDocumentElement().is_some());
match self.window.layout().mouse_over(*point) {
Ok(MouseOverResponse(node_address)) => node_address,
Err(()) => vec!(),
}