squahed-commits

Enable baseline-source tests in css-inline-minor code changes wpt: Enable tests in `/css/css-inline/baseline-source/` (servo#31913) * updated-baseline-source * updated-for-tests-with-fmt rebased-for-tests
MunishMummadi · Mar 29, 2024 · d69f0cd · d69f0cd
1 parent 85f1b9e
commit d69f0cd
Show file tree

Hide file tree

Showing 9 changed files with 2 additions and 228 deletions.
diff --git a/components/layout_2020/flow/inline.rs b/components/layout_2020/flow/inline.rs
@@ -2144,10 +2144,9 @@ impl IndependentFormattingContext {
     /// Picks either the first or the last baseline, depending on `baseline-source`.
     /// <https://drafts.csswg.org/css-inline/#baseline-source>
     fn pick_baseline(&self, baselines: &Baselines) -> Option<Au> {
-        // Checking the baseline source
         match self.style().clone_baseline_source() {
-            BaselineSource::First => return baselines.first,
-            BaselineSource::Last => return baselines.last,
+            BaselineSource::First => baselines.first,
+            BaselineSource::Last => baselines.last,
             BaselineSource::Auto => {
                 // The existing logic for `Auto` remains unchanged.
                 if let Self::NonReplaced(non_replaced) = self {

diff --git a/...s/wpt/meta-legacy-layout/css/css-inline/baseline-source/baseline-source-computed.html.ini b/...s/wpt/meta-legacy-layout/css/css-inline/baseline-source/baseline-source-computed.html.ini
diff --git a/...ta-legacy-layout/css/css-inline/baseline-source/baseline-source-no-interpolation.html.ini b/...ta-legacy-layout/css/css-inline/baseline-source/baseline-source-no-interpolation.html.ini
@@ -37,90 +37,3 @@
 
   [Web Animations: property <baseline-source> from [initial\] to [last\] at (1.5) should be [last\]]
     expected: FAIL
-
-  [CSS Transitions with transition-behavior:allow-discrete: property <baseline-source> from [initial\] to [last\] at (0.5) should be [last\]]
-    expected: FAIL
-
-  [CSS Transitions with transition-behavior:allow-discrete: property <baseline-source> from [initial\] to [last\] at (0.6) should be [last\]]
-    expected: FAIL
-
-  [CSS Transitions with transition-behavior:allow-discrete: property <baseline-source> from [initial\] to [last\] at (1) should be [last\]]
-    expected: FAIL
-
-  [CSS Transitions with transition-behavior:allow-discrete: property <baseline-source> from [initial\] to [last\] at (1.5) should be [last\]]
-    expected: FAIL
-
-  [CSS Transitions with transition-property:all and transition-behavor:allow-discrete: property <baseline-source> from [initial\] to [last\] at (0.5) should be [last\]]
-    expected: FAIL
-
-  [CSS Transitions with transition-property:all and transition-behavor:allow-discrete: property <baseline-source> from [initial\] to [last\] at (0.6) should be [last\]]
-    expected: FAIL
-
-  [CSS Transitions with transition-property:all and transition-behavor:allow-discrete: property <baseline-source> from [initial\] to [last\] at (1) should be [last\]]
-    expected: FAIL
-
-  [CSS Transitions with transition-property:all and transition-behavor:allow-discrete: property <baseline-source> from [initial\] to [last\] at (1.5) should be [last\]]
-    expected: FAIL
-
-  [CSS Transitions: property <baseline-source> from [initial\] to [last\] at (-0.3) should be [last\]]
-    expected: FAIL
-
-  [CSS Transitions: property <baseline-source> from [initial\] to [last\] at (0) should be [last\]]
-    expected: FAIL
-
-  [CSS Transitions: property <baseline-source> from [initial\] to [last\] at (0.3) should be [last\]]
-    expected: FAIL
-
-  [CSS Transitions: property <baseline-source> from [initial\] to [last\] at (0.5) should be [last\]]
-    expected: FAIL
-
-  [CSS Transitions: property <baseline-source> from [initial\] to [last\] at (0.6) should be [last\]]
-    expected: FAIL
-
-  [CSS Transitions: property <baseline-source> from [initial\] to [last\] at (1) should be [last\]]
-    expected: FAIL
-
-  [CSS Transitions: property <baseline-source> from [initial\] to [last\] at (1.5) should be [last\]]
-    expected: FAIL
-
-  [CSS Transitions with transition: all: property <baseline-source> from [initial\] to [last\] at (-0.3) should be [last\]]
-    expected: FAIL
-
-  [CSS Transitions with transition: all: property <baseline-source> from [initial\] to [last\] at (0) should be [last\]]
-    expected: FAIL
-
-  [CSS Transitions with transition: all: property <baseline-source> from [initial\] to [last\] at (0.3) should be [last\]]
-    expected: FAIL
-
-  [CSS Transitions with transition: all: property <baseline-source> from [initial\] to [last\] at (0.5) should be [last\]]
-    expected: FAIL
-
-  [CSS Transitions with transition: all: property <baseline-source> from [initial\] to [last\] at (0.6) should be [last\]]
-    expected: FAIL
-
-  [CSS Transitions with transition: all: property <baseline-source> from [initial\] to [last\] at (1) should be [last\]]
-    expected: FAIL
-
-  [CSS Transitions with transition: all: property <baseline-source> from [initial\] to [last\] at (1.5) should be [last\]]
-    expected: FAIL
-
-  [CSS Animations: property <baseline-source> from [initial\] to [last\] at (-0.3) should be [initial\]]
-    expected: FAIL
-
-  [CSS Animations: property <baseline-source> from [initial\] to [last\] at (0) should be [initial\]]
-    expected: FAIL
-
-  [CSS Animations: property <baseline-source> from [initial\] to [last\] at (0.3) should be [initial\]]
-    expected: FAIL
-
-  [CSS Animations: property <baseline-source> from [initial\] to [last\] at (0.5) should be [last\]]
-    expected: FAIL
-
-  [CSS Animations: property <baseline-source> from [initial\] to [last\] at (0.6) should be [last\]]
-    expected: FAIL
-
-  [CSS Animations: property <baseline-source> from [initial\] to [last\] at (1) should be [last\]]
-    expected: FAIL
-
-  [CSS Animations: property <baseline-source> from [initial\] to [last\] at (1.5) should be [last\]]
-    expected: FAIL
diff --git a/tests/wpt/meta-legacy-layout/css/css-inline/baseline-source/baseline-source-valid.html.ini b/tests/wpt/meta-legacy-layout/css/css-inline/baseline-source/baseline-source-valid.html.ini
diff --git a/tests/wpt/meta/css/css-inline/baseline-source/baseline-source-computed.html.ini b/tests/wpt/meta/css/css-inline/baseline-source/baseline-source-computed.html.ini
diff --git a/tests/wpt/meta/css/css-inline/baseline-source/baseline-source-first-001.html.ini b/tests/wpt/meta/css/css-inline/baseline-source/baseline-source-first-001.html.ini
@@ -1,16 +1,4 @@
 [baseline-source-first-001.html]
-  [.target > * 1]
-    expected: FAIL
-
-  [.target > * 3]
-    expected: FAIL
-
-  [.target > * 5]
-    expected: FAIL
-
-  [.target > * 7]
-    expected: FAIL
-
   [.target > * 9]
     expected: FAIL
 

diff --git a/...meta/css/css-inline/baseline-source/baseline-source-first-textarea-001.tentative.html.ini b/...meta/css/css-inline/baseline-source/baseline-source-first-textarea-001.tentative.html.ini
@@ -2,9 +2,6 @@
   [.target > * 1]
     expected: FAIL
 
-  [.target > * 3]
-    expected: FAIL
-
   [.target > * 5]
     expected: FAIL
 

diff --git a/tests/wpt/meta/css/css-inline/baseline-source/baseline-source-no-interpolation.html.ini b/tests/wpt/meta/css/css-inline/baseline-source/baseline-source-no-interpolation.html.ini
@@ -37,90 +37,3 @@
 
   [Web Animations: property <baseline-source> from [initial\] to [last\] at (1.5) should be [last\]]
     expected: FAIL
-
-  [CSS Transitions with transition-behavior:allow-discrete: property <baseline-source> from [initial\] to [last\] at (0.5) should be [last\]]
-    expected: FAIL
-
-  [CSS Transitions with transition-behavior:allow-discrete: property <baseline-source> from [initial\] to [last\] at (0.6) should be [last\]]
-    expected: FAIL
-
-  [CSS Transitions with transition-behavior:allow-discrete: property <baseline-source> from [initial\] to [last\] at (1) should be [last\]]
-    expected: FAIL
-
-  [CSS Transitions with transition-behavior:allow-discrete: property <baseline-source> from [initial\] to [last\] at (1.5) should be [last\]]
-    expected: FAIL
-
-  [CSS Transitions with transition-property:all and transition-behavor:allow-discrete: property <baseline-source> from [initial\] to [last\] at (0.5) should be [last\]]
-    expected: FAIL
-
-  [CSS Transitions with transition-property:all and transition-behavor:allow-discrete: property <baseline-source> from [initial\] to [last\] at (0.6) should be [last\]]
-    expected: FAIL
-
-  [CSS Transitions with transition-property:all and transition-behavor:allow-discrete: property <baseline-source> from [initial\] to [last\] at (1) should be [last\]]
-    expected: FAIL
-
-  [CSS Transitions with transition-property:all and transition-behavor:allow-discrete: property <baseline-source> from [initial\] to [last\] at (1.5) should be [last\]]
-    expected: FAIL
-
-  [CSS Transitions: property <baseline-source> from [initial\] to [last\] at (-0.3) should be [last\]]
-    expected: FAIL
-
-  [CSS Transitions: property <baseline-source> from [initial\] to [last\] at (0) should be [last\]]
-    expected: FAIL
-
-  [CSS Transitions: property <baseline-source> from [initial\] to [last\] at (0.3) should be [last\]]
-    expected: FAIL
-
-  [CSS Transitions: property <baseline-source> from [initial\] to [last\] at (0.5) should be [last\]]
-    expected: FAIL
-
-  [CSS Transitions: property <baseline-source> from [initial\] to [last\] at (0.6) should be [last\]]
-    expected: FAIL
-
-  [CSS Transitions: property <baseline-source> from [initial\] to [last\] at (1) should be [last\]]
-    expected: FAIL
-
-  [CSS Transitions: property <baseline-source> from [initial\] to [last\] at (1.5) should be [last\]]
-    expected: FAIL
-
-  [CSS Transitions with transition: all: property <baseline-source> from [initial\] to [last\] at (-0.3) should be [last\]]
-    expected: FAIL
-
-  [CSS Transitions with transition: all: property <baseline-source> from [initial\] to [last\] at (0) should be [last\]]
-    expected: FAIL
-
-  [CSS Transitions with transition: all: property <baseline-source> from [initial\] to [last\] at (0.3) should be [last\]]
-    expected: FAIL
-
-  [CSS Transitions with transition: all: property <baseline-source> from [initial\] to [last\] at (0.5) should be [last\]]
-    expected: FAIL
-
-  [CSS Transitions with transition: all: property <baseline-source> from [initial\] to [last\] at (0.6) should be [last\]]
-    expected: FAIL
-
-  [CSS Transitions with transition: all: property <baseline-source> from [initial\] to [last\] at (1) should be [last\]]
-    expected: FAIL
-
-  [CSS Transitions with transition: all: property <baseline-source> from [initial\] to [last\] at (1.5) should be [last\]]
-    expected: FAIL
-
-  [CSS Animations: property <baseline-source> from [initial\] to [last\] at (-0.3) should be [initial\]]
-    expected: FAIL
-
-  [CSS Animations: property <baseline-source> from [initial\] to [last\] at (0) should be [initial\]]
-    expected: FAIL
-
-  [CSS Animations: property <baseline-source> from [initial\] to [last\] at (0.3) should be [initial\]]
-    expected: FAIL
-
-  [CSS Animations: property <baseline-source> from [initial\] to [last\] at (0.5) should be [last\]]
-    expected: FAIL
-
-  [CSS Animations: property <baseline-source> from [initial\] to [last\] at (0.6) should be [last\]]
-    expected: FAIL
-
-  [CSS Animations: property <baseline-source> from [initial\] to [last\] at (1) should be [last\]]
-    expected: FAIL
-
-  [CSS Animations: property <baseline-source> from [initial\] to [last\] at (1.5) should be [last\]]
-    expected: FAIL
diff --git a/tests/wpt/meta/css/css-inline/baseline-source/baseline-source-valid.html.ini b/tests/wpt/meta/css/css-inline/baseline-source/baseline-source-valid.html.ini