r118246 added a mode where all navigations were sent to the browser process, so
that for platform apps we could ignore them. However, we still want to allow
platform apps to have iframes that point to app pages or data: URLs, so the
check needs to be made a bit more fine-grained.
The fine-grained check will be implemented in a follow-up CL (via CSP), for now all
frame navigations are allowed.
[email protected],[email protected]
Review URL: https://chromiumcodereview.appspot.com/10387074
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@138304 0039d316-1c4b-4281-b951-d872f2087c98
diff --git a/chrome/browser/extensions/platform_app_browsertest.cc b/chrome/browser/extensions/platform_app_browsertest.cc
index 4fb84ae..aab74f8 100644
--- a/chrome/browser/extensions/platform_app_browsertest.cc
+++ b/chrome/browser/extensions/platform_app_browsertest.cc
@@ -149,6 +149,11 @@
ASSERT_TRUE(RunPlatformAppTest("platform_apps/navigation")) << message_;
}
+IN_PROC_BROWSER_TEST_F(PlatformAppBrowserTest, Iframes) {
+ ASSERT_TRUE(StartTestServer());
+ ASSERT_TRUE(RunPlatformAppTest("platform_apps/iframes")) << message_;
+}
+
// Tests that localStorage and WebSQL are disabled for platform apps.
IN_PROC_BROWSER_TEST_F(PlatformAppBrowserTest, DisallowStorage) {
ASSERT_TRUE(RunPlatformAppTest("platform_apps/storage")) << message_;
