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

content/public/browser/content_browser_client.cc

Index: content/public/browser/content_browser_client.cc
diff --git a/content/public/browser/content_browser_client.cc b/content/public/browser/content_browser_client.cc
index 529f49fa7ff5469f6a1b4a0858b5de96a5ee8a2e..66f1c4a694c1aca8ba5eb2f53954011e9ac81f0e 100644
--- a/content/public/browser/content_browser_client.cc
+++ b/content/public/browser/content_browser_client.cc
@@ -91,10 +91,10 @@ bool ContentBrowserClient::ShouldAllowOpenURL(SiteInstance* site_instance,
   return true;
 }
 
-bool ContentBrowserClient::
-    ShouldFrameShareParentSiteInstanceDespiteTopDocumentIsolation(
-        const GURL& url,
-        SiteInstance* parent_site_instance) {
+bool ContentBrowserClient::ShouldIsolateFrameFromMainContent(
+    RenderFrameHost* frame,
+    const GURL& dest_url,
+    SiteInstance* main_frame_site_instance) {
   return false;

[Łukasz Anforowicz, 2017/06/30 15:28:37]

I am not sure what the default should be here.  It doesn't matter that much for
browser tests and chrome, only for other embedders.  I think |false| is okay.

 }