Bug 1394551 - stylo: Don't convert property declarations to lowercase r?xidorn draft
authorNazım Can Altınova <canaltinova@gmail.com>
Wed, 30 Aug 2017 17:08:33 -0700
changeset 657854 8ab614b90633d16d356ef34375872d032fa6ed4b
parent 655241 6c3510bac83299cbaffd2d5ecfa8ef114da6c0c4
child 657855 200484fa2b2402001674e9495d9ffe08810ef0bb
push id77646
push userbmo:canaltinova@gmail.com
push dateSat, 02 Sep 2017 06:01:35 +0000
reviewersxidorn
bugs1394551
milestone57.0a1
Bug 1394551 - stylo: Don't convert property declarations to lowercase r?xidorn Property declarations are already being converted to lowercase inside gfxFontFeatureValueSet::AppendFeatureValueHashEntry and there is no point of doing it twice. MozReview-Commit-ID: I59w1RC4bKK
servo/components/style/stylesheets/font_feature_values_rule.rs
--- a/servo/components/style/stylesheets/font_feature_values_rule.rs
+++ b/servo/components/style/stylesheets/font_feature_values_rule.rs
@@ -203,17 +203,17 @@ impl<'a, 'b, 'i, T> DeclarationParser<'i
 {
     type Declaration = ();
     type Error = SelectorParseError<'i, StyleParseError<'i>>;
 
     fn parse_value<'t>(&mut self, name: CowRcStr<'i>, input: &mut Parser<'i, 't>)
                        -> Result<(), ParseError<'i>> {
         let value = input.parse_entirely(|i| T::parse(self.context, i))?;
         let new = FFVDeclaration {
-            name: Atom::from(&*name).to_ascii_lowercase(),
+            name: Atom::from(&*name),
             value: value,
         };
         update_or_push(&mut self.declarations, new);
         Ok(())
     }
 }
 
 macro_rules! font_feature_values_blocks {