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

chrome/browser/ui/extensions/hosted_app_browsertest.cc

 // Copyright 2015 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 <string>
 
 #include "base/command_line.h"
 #include "base/compiler_specific.h"
 #include "base/strings/stringprintf.h"
 #include "base/test/scoped_feature_list.h"
@@ skipping 237 matching lines @@
       subframe, "window.domAutomationController.send(document.body.innerText);",
       &result));
   EXPECT_EQ("This page has no title.", result);
 }
 
 class HostedAppVsTdiTest : public HostedAppTest {
  public:
   HostedAppVsTdiTest() {}
   ~HostedAppVsTdiTest() override {}
 
+  void SetUpCommandLine(base::CommandLine* command_line) override {
+    HostedAppTest::SetUpCommandLine(command_line);
+    content::EnableTopDocumentIsolationForTesting(command_line);
+  }
+
   void SetUpOnMainThread() override {
-    scoped_feature_list_.InitAndEnableFeature(features::kTopDocumentIsolation);
     HostedAppTest::SetUpOnMainThread();
     ASSERT_TRUE(embedded_test_server()->Start());
   }
 
  private:
-  base::test::ScopedFeatureList scoped_feature_list_;
-
   DISALLOW_COPY_AND_ASSIGN(HostedAppVsTdiTest);
 };
 
 // Tests that even with --top-document-isolation, app.site.com (covered by app's
 // web extent) main frame and foo.site.com (not covered by app's web extent)
 // share the same renderer process.  See also https://crbug.com/679011.
 //
 // Relevant frames in the test:
 // - |app| - app.site.com/frame_tree/cross_origin_but_same_site_frames.html
 //           Main frame, launch URL of the hosted app (i.e. app.launch.web_url).
@@ skipping 192 matching lines @@
   // app's web extent should use the isolated origin's SiteInstance and not the
   // app's.
   ui_test_utils::NavigateToURL(browser(), isolated_url);
   content::WebContents* web_contents =
       browser()->tab_strip_model()->GetActiveWebContents();
   EXPECT_EQ(isolated_url.GetOrigin(),
             web_contents->GetMainFrame()->GetSiteInstance()->GetSiteURL());
   EXPECT_NE(web_contents->GetMainFrame()->GetSiteInstance(),
             app->GetSiteInstance());
 }