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

content/public/browser/content_browser_client.h

Index: content/public/browser/content_browser_client.h
diff --git a/content/public/browser/content_browser_client.h b/content/public/browser/content_browser_client.h
index 571f9937a8fea459440b36d9bed1576e880e0d55..ed7cfd42d984b8193f2c69c7194cdccc9704c7d8 100644
--- a/content/public/browser/content_browser_client.h
+++ b/content/public/browser/content_browser_client.h
@@ -9,6 +9,7 @@
 
 #include <map>
 #include <memory>
+#include <set>
 #include <string>
 #include <utility>
 #include <vector>
@@ -256,13 +257,14 @@ class CONTENT_EXPORT ContentBrowserClient {
                                         bool* is_renderer_initiated,
                                         content::Referrer* referrer) {}
 
-  // Allows the embedder to override top document isolation for specific frames.
-  // |url| is the URL being loaded in the subframe, and |parent_site_instance|
-  // is the SiteInstance of the parent frame. Called only for subframes and only
-  // when top document isolation mode is enabled.
-  virtual bool ShouldFrameShareParentSiteInstanceDespiteTopDocumentIsolation(
-      const GURL& url,
-      SiteInstance* parent_site_instance);
+  // Called in TopDocumentIsolation mode to let the embedder decide whether the
+  // |frame| should be isolated from the main content (if the frame is navigated
+  // to |dest_url| that is cross-site from the top-level frame).
+  // |main_frame_site_instance| is the SiteInstance of the top-level frame.
+  virtual bool ShouldIsolateFrameForTopDocumentIsolation(
+      RenderFrameHost* frame,
+      const GURL& dest_url,
+      SiteInstance* main_frame_site_instance);
 
   // Returns whether a new view for a given |site_url| can be launched in a
   // given |process_host|.