Revert "Bug 1817051 - Part 3: Start to sample scroll timeline in the HTML loop. r=firefox-style-system-reviewers,layout-reviewers,hiro,emilio" on request for causing timeout

This reverts commit 42e172e.

Revert "Bug 1817051 - Part 2: Add scroll timeline into AnimationTimelinesController. r=hiro,layout-reviewers"

This reverts commit 4bdbdb5.

Revert "Bug 1817051 - Part 1: Add AnimationTimelinesController. r=layout-reviewers,hiro,dom-core,smaug"

This reverts commit 85c927d.

Author: chorotan

dom/animation/AnimationEventDispatcher.h

@@ -16,7 +16,6 @@
 #include "mozilla/ProfilerMarkers.h"
 #include "mozilla/Variant.h"
 #include "mozilla/dom/AnimationPlaybackEvent.h"
-#include "mozilla/dom/Document.h"
 #include "mozilla/dom/KeyframeEffect.h"
 #include "nsCycleCollectionParticipant.h"
 #include "nsPresContext.h"

dom/animation/AnimationTimelinesController.cpp

@@ -47,7 +47,7 @@ DocumentTimeline::DocumentTimeline(Document* aDocument,
       mDocument(aDocument),
       mOriginTime(aOriginTime) {
   if (mDocument) {
-    mDocument->TimelinesController().AddDocumentTimeline(*this);
+    mDocument->Timelines().insertBack(this);
   }
   // Ensure mLastRefreshDriverTime is valid.
   UpdateLastRefreshDriverTime();

dom/animation/AnimationTimelinesController.h

@@ -51,6 +51,7 @@ void ElementAnimationData::ClearAllAnimationCollections() {
   mElementData.mTransitions = nullptr;
   mElementData.mScrollTimelines = nullptr;
   mElementData.mViewTimelines = nullptr;
+  mElementData.mProgressTimelineScheduler = nullptr;
   ClearAllPseudos(false);
 }
 
@@ -76,6 +77,7 @@ void ElementAnimationData::ClearAllPseudos(bool aOnlyViewTransitions) {
     data->mTransitions = nullptr;
     data->mScrollTimelines = nullptr;
     data->mViewTimelines = nullptr;
+    data->mProgressTimelineScheduler = nullptr;
 
     if (data->IsEmpty()) {
       iter.Remove();
@@ -176,6 +178,13 @@ void ElementAnimationData::ClearViewTimelineCollectionFor(
   });
 }
 
+void ElementAnimationData::ClearProgressTimelineScheduler(
+    const PseudoStyleRequest& aRequest) {
+  WithDataForRemoval(aRequest, [](PerElementOrPseudoData& aData) {
+    aData.mProgressTimelineScheduler = nullptr;
+  });
+}
+
 ElementAnimationData::PerElementOrPseudoData::PerElementOrPseudoData() =
     default;
 ElementAnimationData::PerElementOrPseudoData::~PerElementOrPseudoData() =
@@ -228,4 +237,11 @@ ElementAnimationData::PerElementOrPseudoData::DoEnsureViewTimelines(
   return *mViewTimelines;
 }
 
+dom::ProgressTimelineScheduler& ElementAnimationData::PerElementOrPseudoData::
+    DoEnsureProgressTimelineScheduler() {
+  MOZ_ASSERT(!mProgressTimelineScheduler);
+  mProgressTimelineScheduler = MakeUnique<dom::ProgressTimelineScheduler>();
+  return *mProgressTimelineScheduler;
+}
+
 }  // namespace mozilla

dom/animation/DocumentTimeline.cpp

@@ -23,6 +23,7 @@ namespace dom {
 class Element;
 class CSSAnimation;
 class CSSTransition;
+class ProgressTimelineScheduler;
 class ScrollTimeline;
 class ViewTimeline;
 }  // namespace dom
@@ -46,15 +47,31 @@ class ElementAnimationData {
     //
     // So it should be fine to create timeline objects only on the elements and
     // pseudo elements which support animations.
-    //
-    // Note: TimelineCollection owns and manages the named progress timeline
-    // generated by specifying scroll-timeline-name property and
-    // view-timeline-name property on this element. However, the anonymous
-    // progress timelines (e.g. animation-timeline:scroll()) are owned by
-    // Animation objects only.
     UniquePtr<ScrollTimelineCollection> mScrollTimelines;
     UniquePtr<ViewTimelineCollection> mViewTimelines;
 
+    // This is different from |mScrollTimelines|. We use this to schedule all
+    // scroll-driven animations (which use anonymous/named scroll timelines or
+    // anonymous/name view timelines) for a specific scroll source (which is the
+    // element with ScrollContainerFrame).
+    //
+    // TimelineCollection owns and manages the named progress timeline generated
+    // by specifying scroll-timeline-name property and view-timeline-name
+    // property on this element. However, the anonymous progress timelines (e.g.
+    // animation-timeline:scroll()) are owned by Animation objects only.
+    //
+    // Note:
+    // 1. For named scroll timelines. The element which specifies
+    //    scroll-timeline-name is the scroll source. However, for named view
+    //    timelines, the element which specifies view-timeline-name may not be
+    //    the scroll source because we use its nearest scroll container as the
+    //    scroll source.
+    // 2. For anonymous progress timelines, we don't keep their timeline obejcts
+    //    in TimelineCollection.
+    // So, per 1) and 2), we use |mProgressTimelineScheduler| for the scroll
+    // source element to schedule scroll-driven animations.
+    UniquePtr<dom::ProgressTimelineScheduler> mProgressTimelineScheduler;
+
     PerElementOrPseudoData();
     ~PerElementOrPseudoData();
 
@@ -67,10 +84,12 @@ class ElementAnimationData {
         dom::Element&, const PseudoStyleRequest&);
     ViewTimelineCollection& DoEnsureViewTimelines(dom::Element&,
                                                   const PseudoStyleRequest&);
+    dom::ProgressTimelineScheduler& DoEnsureProgressTimelineScheduler();
 
     bool IsEmpty() const {
       return !mEffectSet && !mAnimations && !mTransitions &&
-             !mScrollTimelines && !mViewTimelines;
+             !mScrollTimelines && !mViewTimelines &&
+             !mProgressTimelineScheduler;
     }
 
     void Traverse(nsCycleCollectionTraversalCallback&);
@@ -242,6 +261,25 @@ class ElementAnimationData {
     return data.DoEnsureViewTimelines(aOwner, aRequest);
   }
 
+  dom::ProgressTimelineScheduler* GetProgressTimelineScheduler(
+      const PseudoStyleRequest& aRequest) const {
+    if (auto* data = GetData(aRequest)) {
+      return data->mProgressTimelineScheduler.get();
+    }
+    return nullptr;
+  }
+
+  void ClearProgressTimelineScheduler(const PseudoStyleRequest& aRequest);
+
+  dom::ProgressTimelineScheduler& EnsureProgressTimelineScheduler(
+      const PseudoStyleRequest& aRequest) {
+    auto& data = GetOrCreateData(aRequest);
+    if (auto* collection = data.mProgressTimelineScheduler.get()) {
+      return *collection;
+    }
+    return data.DoEnsureProgressTimelineScheduler();
+  }
+
   ElementAnimationData() = default;
 };