Revert "Bug 1956140 - Implement active-view-transition pseudo class. r=view-transitions-reviewers,emilio" for causing WPT failures @pseudo-classes-after-part.html

pstanciu · pstanciu@mozilla.com · commit a2a376c3a8ec · 2025-06-11T10:15:42.000Z
This reverts commit 2d996f3.
diff --git a/dom/base/Document.cpp b/dom/base/Document.cpp
@@ -18494,12 +18494,6 @@ already_AddRefed<ViewTransition> Document::StartViewTransition(
   // Step 6: Set document's active view transition to transition.
   mActiveViewTransition = transition;
 
-  // Enable :active-view-transition to allow associated styles to
-  // be applied during the view transition.
-  if (auto* root = this->GetRootElement()) {
-    root->AddStates(ElementState::ACTIVE_VIEW_TRANSITION);
-  }
-
   EnsureViewTransitionOperationsHappen();
 
   // Step 7: return transition
diff --git a/dom/base/rust/lib.rs b/dom/base/rust/lib.rs
@@ -137,9 +137,6 @@ bitflags! {
         /// https://drafts.csswg.org/selectors-4/#open-state
         /// Match whether an openable element is currently open
         const OPEN = 1u64 << 49;
-        /// For :active-view-transition.
-        /// <https://www.w3.org/TR/css-view-transitions-2/#the-active-view-transition-pseudo>
-        const ACTIVE_VIEW_TRANSITION = 1u64 << 50;
 
         /// Some convenience unions.
         const DIR_STATES = Self::LTR.bits() | Self::RTL.bits();
diff --git a/dom/view-transitions/ViewTransition.cpp b/dom/view-transitions/ViewTransition.cpp
@@ -1474,12 +1474,6 @@ void ViewTransition::ClearActiveTransition(bool aIsDocumentHidden) {
   MOZ_ASSERT(mDocument);
   MOZ_ASSERT(mDocument->GetActiveViewTransition() == this);
 
-  // Ensure that any styles associated with :active-view-transition no longer
-  // apply.
-  if (auto* root = mDocument->GetRootElement()) {
-    root->RemoveStates(ElementState::ACTIVE_VIEW_TRANSITION);
-  }
-
   // Step 3
   ClearNamedElements();
 
diff --git a/servo/components/style/gecko/non_ts_pseudo_class_list.rs b/servo/components/style/gecko/non_ts_pseudo_class_list.rs
@@ -45,7 +45,6 @@ macro_rules! apply_non_ts_list {
                 ("focus-visible", FocusVisible, FOCUSRING, _),
                 ("has-slotted", HasSlotted, HAS_SLOTTED, _),
                 ("hover", Hover, HOVER, _),
-                ("active-view-transition", ActiveViewTransition, ACTIVE_VIEW_TRANSITION, _),
                 ("-moz-drag-over", MozDragOver, DRAGOVER, _),
                 ("target", Target, URLTARGET, _),
                 ("indeterminate", Indeterminate, INDETERMINATE, _),
diff --git a/servo/components/style/gecko/selector_parser.rs b/servo/components/style/gecko/selector_parser.rs
@@ -157,9 +157,6 @@ impl NonTSPseudoClass {
         if matches!(*self, Self::HasSlotted) {
             return static_prefs::pref!("layout.css.has-slotted-selector.enabled");
         }
-        if matches!(*self, Self::ActiveViewTransition) {
-            return static_prefs::pref!("dom.viewTransitions.enabled");
-        }
         !self.has_any_flag(NonTSPseudoClassFlag::PSEUDO_CLASS_ENABLED_IN_UA_SHEETS_AND_CHROME)
     }
 
diff --git a/servo/components/style/gecko/wrapper.rs b/servo/components/style/gecko/wrapper.rs
@@ -2098,7 +2098,6 @@ impl<'le> ::selectors::Element for GeckoElement<'le> {
             NonTSPseudoClass::HasSlotted |
             NonTSPseudoClass::MozAutofillPreview |
             NonTSPseudoClass::MozRevealed |
-            NonTSPseudoClass::ActiveViewTransition |
             NonTSPseudoClass::MozValueEmpty => self.state().intersects(pseudo_class.state_flag()),
             NonTSPseudoClass::Dir(ref dir) => self.state().intersects(dir.element_state()),
             NonTSPseudoClass::AnyLink => self.is_link(),
diff --git a/testing/web-platform/meta/css/css-view-transitions/active-view-transition-on-non-root.html.ini b/testing/web-platform/meta/css/css-view-transitions/active-view-transition-on-non-root.html.ini
@@ -0,0 +1,2 @@
+[active-view-transition-on-non-root.html]
+  expected: [PASS, FAIL]
diff --git a/testing/web-platform/meta/css/css-view-transitions/active-view-transition-pseudo-class-match.html.ini b/testing/web-platform/meta/css/css-view-transitions/active-view-transition-pseudo-class-match.html.ini
@@ -0,0 +1,2 @@
+[active-view-transition-pseudo-class-match.html]
+  expected: FAIL

Original file line number	Diff line number	Diff line change
`@@ -157,9 +157,6 @@ impl NonTSPseudoClass {`
`157`	`157`	`if matches!(*self, Self::HasSlotted) {`
`158`	`158`	`return static_prefs::pref!("layout.css.has-slotted-selector.enabled");`
`159`	`159`	`}`
`160`		`- if matches!(*self, Self::ActiveViewTransition) {`
`161`		`- return static_prefs::pref!("dom.viewTransitions.enabled");`
`162`		`- }`
`163`	`160`	`!self.has_any_flag(NonTSPseudoClassFlag::PSEUDO_CLASS_ENABLED_IN_UA_SHEETS_AND_CHROME)`
`164`	`161`	`}`
`165`	`162`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,2 @@`
	`1`	`+[active-view-transition-on-non-root.html]`
	`2`	`+ expected: [PASS, FAIL]`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,2 @@`
	`1`	`+[active-view-transition-pseudo-class-match.html]`
	`2`	`+ expected: FAIL`