aboutsummaryrefslogtreecommitdiffstats
path: root/components
diff options
context:
space:
mode:
authorbors-servo <metajack+bors@gmail.com>2014-12-11 08:07:14 -0700
committerbors-servo <metajack+bors@gmail.com>2014-12-11 08:07:14 -0700
commitd67bcfa7ce7b37e7ed45a53f1daddbc6b2f2ddf2 (patch)
tree5e3e747d70857f89c6c2f9197d8446ee70699b00 /components
parent80756a11d2fc9dbc67eda580e7ed9c22b354647e (diff)
parent949e7918e0cb96e1b0d304a51ac6aa5739eca1d6 (diff)
downloadservo-d67bcfa7ce7b37e7ed45a53f1daddbc6b2f2ddf2.tar.gz
servo-d67bcfa7ce7b37e7ed45a53f1daddbc6b2f2ddf2.zip
auto merge of #4330 : Ms2ger/servo/page, r=jdm
Diffstat (limited to 'components')
-rw-r--r--components/script/page.rs24
1 files changed, 11 insertions, 13 deletions
diff --git a/components/script/page.rs b/components/script/page.rs
index c8e46463077..d634e86a84a 100644
--- a/components/script/page.rs
+++ b/components/script/page.rs
@@ -434,11 +434,10 @@ impl Page {
pub fn hit_test(&self, point: &Point2D<f32>) -> Option<UntrustedNodeAddress> {
let frame = self.frame();
let document = frame.as_ref().unwrap().document.root();
- let root = document.GetDocumentElement().root();
- if root.is_none() {
- return None;
- }
- let root = root.unwrap();
+ let root = match document.GetDocumentElement().root() {
+ None => return None,
+ Some(root) => root,
+ };
let root: JSRef<Node> = NodeCast::from_ref(*root);
let address = match self.layout().hit_test(root.to_trusted_node_address(), *point) {
Ok(HitTestResponse(node_address)) => {
@@ -455,11 +454,10 @@ impl Page {
pub fn get_nodes_under_mouse(&self, point: &Point2D<f32>) -> Option<Vec<UntrustedNodeAddress>> {
let frame = self.frame();
let document = frame.as_ref().unwrap().document.root();
- let root = document.GetDocumentElement().root();
- if root.is_none() {
- return None;
- }
- let root = root.unwrap();
+ let root = match document.GetDocumentElement().root() {
+ None => return None,
+ Some(root) => root,
+ };
let root: JSRef<Node> = NodeCast::from_ref(*root);
let address = match self.layout().mouse_over(root.to_trusted_node_address(), *point) {
Ok(MouseOverResponse(node_address)) => {
@@ -475,9 +473,9 @@ impl Page {
fn should_move_clip_rect(clip_rect: Rect<Au>, new_viewport: Rect<f32>) -> bool{
let clip_rect = Rect(Point2D(geometry::to_frac_px(clip_rect.origin.x) as f32,
- geometry::to_frac_px(clip_rect.origin.y) as f32),
- Size2D(geometry::to_frac_px(clip_rect.size.width) as f32,
- geometry::to_frac_px(clip_rect.size.height) as f32));
+ geometry::to_frac_px(clip_rect.origin.y) as f32),
+ Size2D(geometry::to_frac_px(clip_rect.size.width) as f32,
+ geometry::to_frac_px(clip_rect.size.height) as f32));
// We only need to move the clip rect if the viewport is getting near the edge of
// our preexisting clip rect. We use half of the size of the viewport as a heuristic