[iOS] Fix OverlayPresenterObserver callbacks.

The iteration loops mistakenly early returned if any observer didn't
support a request.  This prevents code after an observer iteration
from running if any of the observers don't support that request.

Bug: none
Change-Id: I8fe428ecbdaf54f416cf4bde547bc6aca9d10351
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1992362
Commit-Queue: Kurt Horimoto <kkhorimoto@chromium.org>
Reviewed-by: Mike Dougherty <michaeldo@chromium.org>
Auto-Submit: Kurt Horimoto <kkhorimoto@chromium.org>
Cr-Commit-Position: refs/heads/master@{#729628}

ios/chrome/browser/overlays/overlay_presenter_impl.mm

@@ -208,9 +208,8 @@ void OverlayPresenterImpl::PresentOverlayForActiveRequest() {
 
   // Notify the observers that the overlay UI is about to be shown.
   for (auto& observer : observers_) {
-    if (!observer.GetRequestSupport(this)->IsRequestSupported(request))
-      return;
-    observer.WillShowOverlay(this, request);
+    if (observer.GetRequestSupport(this)->IsRequestSupported(request))
+      observer.WillShowOverlay(this, request);
   }
 
   // Present the overlay UI via the UI delegate.
@@ -231,9 +230,8 @@ void OverlayPresenterImpl::OverlayWasPresented(
   DCHECK_EQ(presentation_context_, presentation_context);
   DCHECK_EQ(presented_request_, request);
   for (auto& observer : observers_) {
-    if (!observer.GetRequestSupport(this)->IsRequestSupported(request))
-      return;
-    observer.DidShowOverlay(this, request);
+    if (observer.GetRequestSupport(this)->IsRequestSupported(request))
+      observer.DidShowOverlay(this, request);
   }
 }
 
@@ -271,9 +269,8 @@ void OverlayPresenterImpl::OverlayWasDismissed(
 
   // Notify the observers that the overlay UI was hidden.
   for (auto& observer : observers_) {
-    if (!observer.GetRequestSupport(this)->IsRequestSupported(request))
-      return;
-    observer.DidHideOverlay(this, request);
+    if (observer.GetRequestSupport(this)->IsRequestSupported(request))
+      observer.DidHideOverlay(this, request);
   }
 
   // Only show the next overlay if the active request has changed, either