Use FrameIsAd to decide whether to isolate a frame in TopDocumentIsolation mode.

chrome/browser/chrome_content_browser_client.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/chrome_content_browser_client.h"
 
 #include <map>
 #include <set>
 #include <utility>
 #include <vector>
 
 #include "base/base_switches.h"
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/command_line.h"
 #include "base/files/scoped_file.h"
 #include "base/i18n/character_encoding.h"
 #include "base/json/json_reader.h"
 #include "base/lazy_instance.h"
 #include "base/macros.h"
 #include "base/memory/ptr_util.h"
+#include "base/metrics/field_trial_params.h"
 #include "base/metrics/histogram_macros.h"
 #include "base/path_service.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/string_split.h"
 #include "base/strings/string_util.h"
 #include "base/strings/stringprintf.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/threading/sequenced_worker_pool.h"
 #include "base/threading/thread_task_runner_handle.h"
 #include "build/build_config.h"
@@ skipping 16 matching lines @@
 #include "chrome/browser/font_family_cache.h"
 #include "chrome/browser/media/router/media_router_feature.h"
 #include "chrome/browser/media/router/presentation_service_delegate_impl.h"
 #include "chrome/browser/media/router/receiver_presentation_service_delegate_impl.h"
 #include "chrome/browser/media/webrtc/media_capture_devices_dispatcher.h"
 #include "chrome/browser/memory/chrome_memory_coordinator_delegate.h"
 #include "chrome/browser/metrics/chrome_browser_main_extra_parts_metrics.h"
 #include "chrome/browser/nacl_host/nacl_browser_delegate_impl.h"
 #include "chrome/browser/net_benchmarking.h"
 #include "chrome/browser/notifications/platform_notification_service_impl.h"
+#include "chrome/browser/page_load_metrics/ads_detection.h"
 #include "chrome/browser/page_load_metrics/experiments/delay_navigation_throttle.h"
 #include "chrome/browser/page_load_metrics/metrics_navigation_throttle.h"
 #include "chrome/browser/page_load_metrics/page_load_metrics_util.h"
 #include "chrome/browser/password_manager/chrome_password_manager_client.h"
 #include "chrome/browser/permissions/permission_context_base.h"
 #include "chrome/browser/platform_util.h"
 #include "chrome/browser/prerender/prerender_final_status.h"
 #include "chrome/browser/prerender/prerender_manager.h"
 #include "chrome/browser/prerender/prerender_manager_factory.h"
 #include "chrome/browser/prerender/prerender_message_filter.h"
@@ skipping 1188 matching lines @@
     *is_renderer_initiated = false;
     *referrer = content::Referrer();
   }
 
 #if BUILDFLAG(ENABLE_EXTENSIONS)
   ChromeContentBrowserClientExtensionsPart::OverrideNavigationParams(
       site_instance, transition, is_renderer_initiated, referrer);
 #endif
 }
 
-bool ChromeContentBrowserClient::
-    ShouldFrameShareParentSiteInstanceDespiteTopDocumentIsolation(
-        const GURL& url,
-        content::SiteInstance* parent_site_instance) {
-  if (IsNTPSiteInstance(parent_site_instance))
-    return true;
+bool ChromeContentBrowserClient::ShouldIsolateFrameForTopDocumentIsolation(
+    content::NavigationHandle* navigation_handle,
+    content::SiteInstance* main_frame_site_instance) {
+  if (IsNTPSiteInstance(main_frame_site_instance))
+    return false;
 
 #if BUILDFLAG(ENABLE_EXTENSIONS)
-  return ChromeContentBrowserClientExtensionsPart::
-      ShouldFrameShareParentSiteInstanceDespiteTopDocumentIsolation(
-          url, parent_site_instance);
-#else
+  if (ChromeContentBrowserClientExtensionsPart::
+          IsMainFrameSiteInstanceExcludedFromTopDocumentIsolation(
+              main_frame_site_instance)) {
+    return false;
+  }
+#endif
+
+  features::TopDocumentIsolationMode tdiMode =
+      static_cast<features::TopDocumentIsolationMode>(
+          base::GetFieldTrialParamByFeatureAsInt(
+              ::features::kTopDocumentIsolation,
+              ::features::kTopDocumentIsolationModeParam,
+              static_cast<int>(
+                  features::TopDocumentIsolationMode::Unspecified)));
+  switch (tdiMode) {
+    case features::TopDocumentIsolationMode::Unspecified:
+    case features::TopDocumentIsolationMode::Ads:
+      return page_load_metrics::GetDetectedAdTypes(navigation_handle).any();
+
+    case features::TopDocumentIsolationMode::CrossSite:
+      return true;
+  }
+  NOTREACHED();
   return false;
-#endif
 }
 
 bool ChromeContentBrowserClient::IsSuitableHost(
     content::RenderProcessHost* process_host,
     const GURL& site_url) {
   Profile* profile =
       Profile::FromBrowserContext(process_host->GetBrowserContext());
   // This may be NULL during tests. In that case, just assume any site can
   // share any host.
   if (!profile)
@@ skipping 2159 matching lines @@
   // |safe_browsing_service_| may be unavailable in tests.
   if (safe_browsing_service_ && !safe_browsing_url_checker_delegate_) {
     safe_browsing_url_checker_delegate_ =
         new safe_browsing::UrlCheckerDelegateImpl(
             safe_browsing_service_->database_manager(),
             safe_browsing_service_->ui_manager());
   }
 
   return safe_browsing_url_checker_delegate_.get();
 }