[App banners] Be less strict about navigations

chrome/browser/banners/app_banner_manager.cc

Index: chrome/browser/banners/app_banner_manager.cc
diff --git a/chrome/browser/banners/app_banner_manager.cc b/chrome/browser/banners/app_banner_manager.cc
index c16251fdf2896b7f0cf735a7fec6955760874198..fd9ed53e968db2fb80c49d363a1453b7cb3492a0 100644
--- a/chrome/browser/banners/app_banner_manager.cc
+++ b/chrome/browser/banners/app_banner_manager.cc
@@ -6,6 +6,7 @@
 
 #include "base/metrics/field_trial.h"
 #include "chrome/browser/banners/app_banner_data_fetcher.h"
+#include "chrome/browser/banners/app_banner_settings_helper.h"
 #include "content/public/browser/navigation_details.h"
 #include "content/public/browser/render_frame_host.h"
 #include "content/public/browser/web_contents.h"
@@ -35,6 +36,13 @@ void AppBannerManager::DidFinishLoad(
   if (render_frame_host->GetParent())
     return;
 
+  if (data_fetcher_.get() && data_fetcher_->is_active()
+        && AppBannerSettingsHelper::URLsAreForTheSamePage(
+              data_fetcher_->validated_url(),
+              validated_url)) {
+    return;
+  }
+
   // A secure scheme is required to show banners, so exit early if we see the
   // URL is invalid.
   if (!validated_url.SchemeIsSecure() && !gDisableSecureCheckForTesting)