Auto merge of #18422 - emilio:stale-todo, r=bholley

style: Remove stale todo about handling scrollbars in viewport units.

This is effectively not going to happen, per
https://github.com/w3c/csswg-drafts/issues/1766

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/18422)
<!-- Reviewable:end -->
This commit is contained in:
bors-servo 2017-09-08 16:30:26 -05:00 committed by GitHub
commit 9c0abfea23

View file

@ -156,12 +156,9 @@ 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))
}
}
/// Returns the current viewport size in app units, recording that it's been
/// used for viewport unit resolution.