servo: Merge #17153 - stylo: disable @viewport (from Manishearth:stylo-vp-disable); r=emilio
authorManish Goregaokar <manishearth@gmail.com>
Sun, 04 Jun 2017 08:56:26 -0700
changeset 410332 5599c3c468c47b35804f8619b632cfa4d4f74b3d
parent 410331 350993973191e37ef5d4da63ebf9bfa1ab726759
child 410333 b9bb7e5e804fb2ca7eba25b5abdab1378a7e052f
push id7391
push usermtabara@mozilla.com
push dateMon, 12 Jun 2017 13:08:53 +0000
treeherdermozilla-beta@2191d7f87e2e [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersemilio
milestone55.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
servo: Merge #17153 - stylo: disable @viewport (from Manishearth:stylo-vp-disable); r=emilio Firefox doesn't support @viewport, we shouldn't either. r=emilio https://bugzilla.mozilla.org/show_bug.cgi?id=1347410 Source-Repo: https://github.com/servo/servo Source-Revision: 7176e3ae9e399c413eed0559c68c0ba1945040bb
servo/components/style/stylesheets.rs
--- a/servo/components/style/stylesheets.rs
+++ b/servo/components/style/stylesheets.rs
@@ -1754,17 +1754,17 @@ impl<'a, 'b> NestedRuleParser<'a, 'b> {
 
 #[cfg(feature = "servo")]
 fn is_viewport_enabled() -> bool {
     PREFS.get("layout.viewport.enabled").as_boolean().unwrap_or(false)
 }
 
 #[cfg(not(feature = "servo"))]
 fn is_viewport_enabled() -> bool {
-    true
+    false // Gecko doesn't support @viewport
 }
 
 impl<'a, 'b> AtRuleParser for NestedRuleParser<'a, 'b> {
     type Prelude = AtRulePrelude;
     type AtRule = CssRule;
 
     fn parse_prelude(
         &mut self,