Skip to content

Commit

Permalink
servo: Merge #18422 - style: Remove stale todo about handling scrollb…
Browse files Browse the repository at this point in the history
…ars in viewport units (from emilio:stale-todo); r=bholley

This is effectively not going to happen, per
w3c/csswg-drafts#1766

Source-Repo: https://github.com/servo/servo
Source-Revision: 9c0abfea239077bc490ed8a2d4a7c32ba782942d
  • Loading branch information
emilio committed Sep 8, 2017
1 parent bcd88a9 commit 8220c03
Showing 1 changed file with 2 additions and 5 deletions.
7 changes: 2 additions & 5 deletions servo/components/style/gecko/media_queries.rs
Original file line number Diff line number Diff line change
Expand Up @@ -156,11 +156,8 @@ impl Device {

/// Returns the current viewport size in app units.
pub fn au_viewport_size(&self) -> Size2D<Au> {
unsafe {
// TODO(emilio): Need to take into account scrollbars.
let area = &self.pres_context().mVisibleArea;
Size2D::new(Au(area.width), Au(area.height))
}
let area = &self.pres_context().mVisibleArea;
Size2D::new(Au(area.width), Au(area.height))
}

/// Returns the current viewport size in app units, recording that it's been
Expand Down

0 comments on commit 8220c03

Please sign in to comment.