From d5ba19696aa47f6b29473cd8fef7b38d441cbe3e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Emilio=20Cobos=20=C3=81lvarez?= Date: Mon, 17 Sep 2018 00:54:43 +0000 Subject: [PATCH] style: Make ExtremumLength::valid_for static. Differential Revision: https://phabricator.services.mozilla.com/D5975 --- components/style/values/computed/length.rs | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/components/style/values/computed/length.rs b/components/style/values/computed/length.rs index f2d43e2ada9..5acc2688224 100644 --- a/components/style/values/computed/length.rs +++ b/components/style/values/computed/length.rs @@ -953,7 +953,7 @@ impl ExtremumLength { /// TODO: After these values are supported for both axes (and maybe /// unprefixed, see bug 1322780) all this complexity can go away, and /// everything can be derived (no need for uncacheable stuff). - fn valid_for(&self, wm: WritingMode, longhand: LonghandId) -> bool { + fn valid_for(wm: WritingMode, longhand: LonghandId) -> bool { // We only make sense on the inline axis. match longhand { // FIXME(emilio): The flex-basis thing is not quite clear... @@ -1018,7 +1018,7 @@ impl ToComputedValue for specified::MozLength { .rule_cache_conditions .borrow_mut() .set_writing_mode_dependency(context.builder.writing_mode); - if !ext.valid_for( + if !ExtremumLength::valid_for( context.builder.writing_mode, context.for_non_inherited_property.unwrap(), ) { @@ -1080,7 +1080,7 @@ impl ToComputedValue for specified::MaxLength { .rule_cache_conditions .borrow_mut() .set_writing_mode_dependency(context.builder.writing_mode); - if !ext.valid_for( + if !ExtremumLength::valid_for( context.builder.writing_mode, context.for_non_inherited_property.unwrap(), ) {