Bug 1645339 - Use range-based for with nsTObserverArray in xpfe. r=smaug

Differential Revision: https://phabricator.services.mozilla.com/D79505
ambroff · Jun 16, 2020 · fcfc4b9 · fcfc4b9
1 parent 6eb160f
commit fcfc4b9
Showing 1 changed file with 4 additions and 6 deletions.
diff --git a/xpfe/appshell/nsWindowMediator.cpp b/xpfe/appshell/nsWindowMediator.cpp
@@ -74,9 +74,8 @@ NS_IMETHODIMP nsWindowMediator::RegisterWindow(nsIAppWindow* inWindow) {
   // Create window info struct and add to list of windows
   nsWindowInfo* windowInfo = new nsWindowInfo(inWindow, mTimeStamp);
 
-  ListenerArray::ForwardIterator iter(mListeners);
-  while (iter.HasMore()) {
-    iter.GetNext()->OnOpenWindow(inWindow);
+  for (const auto& listener : mListeners.ForwardRange()) {
+    listener->OnOpenWindow(inWindow);
   }
 
   if (mOldestWindow)
@@ -105,9 +104,8 @@ nsresult nsWindowMediator::UnregisterWindow(nsWindowInfo* inInfo) {
   }
 
   nsIAppWindow* window = inInfo->mWindow.get();
-  ListenerArray::ForwardIterator iter(mListeners);
-  while (iter.HasMore()) {
-    iter.GetNext()->OnCloseWindow(window);
+  for (const auto& listener : mListeners.ForwardRange()) {
+    listener->OnCloseWindow(window);
   }
 
   // Remove from the lists and free up