Mirrors/servo
1
0
Fork 0
mirror of https://github.com/servo/servo.git synced 2025-08-06 06:00:15 +01:00
Code Issues Projects Releases Packages Wiki Activity

Change resolution queries to compare in unconverted dppx units.

Browse source 
MozReview-Commit-ID: 7wYlixTQTIC
This commit is contained in:
Brad Werth 2017-09-28 11:50:42 -07:00 • committed by Brad Werth
parent 3dc4614991
commit b8e78d20bf
1 changed files with 12 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

14
components/style/gecko/media_queries.rs
View file

@ -373,8 +373,18 @@ impl MediaExpressionValue {
Some(MediaExpressionValue::BoolInteger(i == 1))
}
nsMediaFeature_ValueType::eResolution => {
debug_assert!(css_value.mUnit == nsCSSUnit::eCSSUnit_Inch);
Some(MediaExpressionValue::Resolution(Resolution::Dpi(css_value.float_unchecked())))
// This is temporarily more complicated to allow Gecko Bug
// 1376931 to land. Parts of that bug will supply pixel values
// and expect them to be passed through without conversion.
// After all parts of that bug have landed, Bug 1404097 will
// return this function to once again only allow one type of
// value to be accepted: this time, only pixel values.
let res = match css_value.mUnit {
nsCSSUnit::eCSSUnit_Pixel => Resolution::Dppx(css_value.float_unchecked()),
nsCSSUnit::eCSSUnit_Inch => Resolution::Dpi(css_value.float_unchecked()),
_ => unreachable!(),
};
Some(MediaExpressionValue::Resolution(res))
}
nsMediaFeature_ValueType::eEnumerated => {
let value = css_value.integer_unchecked() as i16;