style: Container Relative Units: Set flag for USES_VIEWPORT_UNITS only when viewport fallback is actually used

Differential Revision: https://phabricator.services.mozilla.com/D159866
This commit is contained in:
David Shin 2022-10-20 18:37:39 +00:00 committed by Martin Robinson
parent 15cf28815e
commit 14eece6f5a
3 changed files with 3 additions and 11 deletions

View file

@ -375,9 +375,7 @@ pub struct ContainerSizeQueryResult {
impl ContainerSizeQueryResult {
fn get_viewport_size(context: &Context) -> Size2D<Au> {
use crate::values::specified::ViewportVariant;
context
.device()
.au_viewport_size_for_viewport_unit_resolution(ViewportVariant::Small)
context.viewport_size_for_viewport_unit_resolution(ViewportVariant::Small)
}
fn get_logical_viewport_size(context: &Context) -> LogicalSize<Au> {

View file

@ -5,7 +5,6 @@
//! `<length>` computed values, and related ones.
use super::{Context, Number, ToComputedValue};
use crate::computed_value_flags::ComputedValueFlags;
use crate::values::animated::ToAnimatedValue;
use crate::values::computed::NonNegativeNumber;
use crate::values::generics::length as generics;
@ -53,16 +52,9 @@ impl specified::NoCalcLength {
length.to_computed_value(context, base_size)
},
specified::NoCalcLength::ViewportPercentage(length) => {
context
.builder
.add_flags(ComputedValueFlags::USES_VIEWPORT_UNITS);
length.to_computed_value(context)
},
specified::NoCalcLength::ContainerRelative(length) => {
// Fallback uses small viewport size.
context
.builder
.add_flags(ComputedValueFlags::USES_VIEWPORT_UNITS);
length.to_computed_value(context)
},
specified::NoCalcLength::ServoCharacterWidth(length) => {

View file

@ -361,6 +361,8 @@ impl<'a> Context<'a> {
&self,
variant: ViewportVariant,
) -> default::Size2D<Au> {
self.builder
.add_flags(ComputedValueFlags::USES_VIEWPORT_UNITS);
self.builder
.device
.au_viewport_size_for_viewport_unit_resolution(variant)