Add a test for subframes redirecting to hosted app URLs.

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 81 matching lines @@
       if (browser_app_id == app->id()) {
         app_browser_ = b;
         break;
       }
     }
 
     ASSERT_TRUE(app_browser_);
     ASSERT_TRUE(app_browser_ != browser());
   }
 
+  void SetUpOnMainThread() override {
+    ExtensionBrowserTest::SetUpOnMainThread();
+    host_resolver()->AddRule("*", "127.0.0.1");
+  }
+
   Browser* app_browser_;
 
  private:
   base::test::ScopedFeatureList scoped_feature_list_;
 
   DISALLOW_COPY_AND_ASSIGN(HostedAppTest);
 };
 
 // Check that the location bar is shown correctly for bookmark apps.
 IN_PROC_BROWSER_TEST_F(HostedAppTest,
@@ skipping 79 matching lines @@
   // Navigate to the app's launch page with the 'www.' prefis; the location bar
   // should be hidden.
   NavigateAndCheckForLocationBar(
       app_browser_, "http://www.example.com/empty.html", false);
 
   // Navigate to different origin; the location bar should now be visible.
   NavigateAndCheckForLocationBar(
       app_browser_, "http://www.foo.com/blah", true);
 }
 
+// Check that a subframe on a regular web page can navigate to a URL that
+// redirects to a hosted app.  https://crbug.com/721949.
+IN_PROC_BROWSER_TEST_F(HostedAppTest, SubframeRedirectsToHostedApp) {
+  ASSERT_TRUE(embedded_test_server()->Start());
+
+  // Set up an app which covers app.com URLs.
+  GURL app_url = embedded_test_server()->GetURL("app.com", "/title1.html");
+  extensions::TestExtensionDir test_app_dir;
+  test_app_dir.WriteManifest(base::StringPrintf(
+      R"( { "name": "Hosted App",
+            "version": "1",
+            "manifest_version": 2,
+            "app": {
+              "launch": {
+                "web_url": "%s"
+              },
+              "urls": ["*://app.com/"]
+            }
+          } )",
+      app_url.spec().c_str()));
+  SetupApp(test_app_dir.UnpackedPath(), false);
+
+  // Navigate a regular tab to a page with a subframe.
+  GURL url = embedded_test_server()->GetURL("foo.com", "/iframe.html");
+  content::WebContents* tab =
+      browser()->tab_strip_model()->GetActiveWebContents();
+  ui_test_utils::NavigateToURL(browser(), url);
+
+  // Navigate the subframe to a URL that redirects to a URL in the hosted app's
+  // web extent.
+  GURL redirect_url = embedded_test_server()->GetURL(
+      "bar.com", "/server-redirect?" + app_url.spec());
+  EXPECT_TRUE(NavigateIframeToURL(tab, "test", redirect_url));
+
+  // Ensure that the frame navigated successfully and that it has correct
+  // content.
+  content::RenderFrameHost* subframe =
+      content::ChildFrameAt(tab->GetMainFrame(), 0);
+  EXPECT_EQ(app_url, subframe->GetLastCommittedURL());
+  std::string result;
+  EXPECT_TRUE(ExecuteScriptAndExtractString(
+      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 SetUpOnMainThread() override {
     scoped_feature_list_.InitAndEnableFeature(features::kTopDocumentIsolation);
     HostedAppTest::SetUpOnMainThread();
-    host_resolver()->AddRule("*", "127.0.0.1");
     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
@@ skipping 123 matching lines @@
   } else {
     // TODO(lukasza): https://crbug.com/718516: Process policy is not
     // well-defined / settled wrt relationship between 1) hosted apps and 2)
     // same-site web content outside of hosted app's extent.  When this test was
     // authored --site-per-process would put |app| in a separate renderer
     // process from |diff_dir| and |same_site|, even though such process
     // placement can be problematic (if |app| tries to synchronously script
     // |diff_dir| and/or |same_site|).
   }
 }