Bug 1635319 Part 3 - Use range-based for loops instead of ChildListIterator in other layout directories. r=mats

Differential Revision: https://phabricator.services.mozilla.com/D75654

layout/style/nsFontFaceUtils.cpp

@@ -182,11 +182,8 @@ void nsFontFaceUtils::MarkDirtyForFontChange(nsIFrame* aSubtreeRoot,
           }
         }
 
-        nsIFrame::ChildListIterator lists(f);
-        for (; !lists.IsDone(); lists.Next()) {
-          nsFrameList::Enumerator childFrames(lists.CurrentList());
-          for (; !childFrames.AtEnd(); childFrames.Next()) {
-            nsIFrame* kid = childFrames.get();
+        for (const auto& childList : f->GetChildLists()) {
+          for (nsIFrame* kid : childList.mList) {
             stack.AppendElement(std::make_pair(kid, alreadyScheduled));
           }
         }

layout/xul/nsDeckFrame.cpp

@@ -200,9 +200,8 @@ void nsDeckFrame::Animate(nsIFrame* aParentBox, bool start) {
       imgFrame->StopAnimation();
   }
 
-  for (nsIFrame::ChildListIterator childLists(aParentBox); !childLists.IsDone();
-       childLists.Next()) {
-    for (nsIFrame* child : childLists.CurrentList()) {
+  for (const auto& childList : aParentBox->GetChildLists()) {
+    for (nsIFrame* child : childList.mList) {
       Animate(child, start);
     }
   }

layout/xul/nsScrollbarFrame.cpp

@@ -272,11 +272,8 @@ int32_t nsScrollbarFrame::MoveToNewPosition() {
     return curpos;
   }
   // notify all nsSliderFrames of the change
-  nsIFrame::ChildListIterator childLists(this);
-  for (; !childLists.IsDone(); childLists.Next()) {
-    nsFrameList::Enumerator childFrames(childLists.CurrentList());
-    for (; !childFrames.AtEnd(); childFrames.Next()) {
-      nsIFrame* f = childFrames.get();
+  for (const auto& childList : nsIFrame::GetChildLists()) {
+    for (nsIFrame* f : childList.mList) {
       nsSliderFrame* sliderFrame = do_QueryFrame(f);
       if (sliderFrame) {
         sliderFrame->AttributeChanged(kNameSpaceID_None, nsGkAtoms::curpos,