From 9a55b1e46904eb0e686094654393b596baff6d9c Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Emilio=20Cobos=20=C3=81lvarez?= Date: Fri, 8 Sep 2017 23:24:12 +0200 Subject: [PATCH] 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 --- components/style/gecko/media_queries.rs | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/components/style/gecko/media_queries.rs b/components/style/gecko/media_queries.rs index 067375bef40..e107e684974 100644 --- a/components/style/gecko/media_queries.rs +++ b/components/style/gecko/media_queries.rs @@ -157,11 +157,8 @@ impl Device { /// Returns the current viewport size in app units. pub fn au_viewport_size(&self) -> Size2D { - 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