Skip to content
This repository has been archived by the owner on Nov 7, 2018. It is now read-only.

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 ef6b8ce commit bde1e6e
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 bde1e6e

Please sign in to comment.