Don't refer browser-initiated navigations to web-safe URLs to delegate.

chrome/browser/apps/guest_view/web_view_browsertest.cc

 // Copyright 2013 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 <queue>
 
 #include "base/location.h"
 #include "base/path_service.h"
 #include "base/process/process.h"
 #include "base/single_thread_task_runner.h"
@@ skipping 1575 matching lines @@
   GetGuestWebContents()->GetDelegate()->OpenURLFromTab(
       GetGuestWebContents(), params);
 
   ASSERT_TRUE(load_listener.WaitUntilSatisfied());
 
   // Verify that the <webview> ends up at about:blank.
   EXPECT_EQ(GURL(url::kAboutBlankURL),
             GetGuestWebContents()->GetLastCommittedURL());
 }
 
+// A navigation to a web-safe URL should succeed, even if it is not renderer-
+// initiated, such as a navigation from the PDF viewer.
+IN_PROC_BROWSER_TEST_F(WebViewTest, OpenURLFromTab_CurrentTab_Succeed) {
+  LoadAppWithGuest("web_view/simple");
+
+  // Verify that OpenURLFromTab with a window disposition of CURRENT_TAB will
+  // navigate the current <webview>.
+  ExtensionTestMessageListener load_listener("WebViewTest.LOADSTOP", false);
+
+  GURL test_url("http://www.google.com");
+  // Navigating to a file URL is forbidden inside a <webview>.

[Charlie Reis, 2015/07/17 00:05:15]

This comment is stale.  The navigation is supposed to be allowed, right?

[wjmaclean, 2015/07/17 01:29:46]

Done.

+  content::OpenURLParams params(test_url,
+                                content::Referrer(),
+                                CURRENT_TAB,
+                                ui::PAGE_TRANSITION_AUTO_TOPLEVEL,
+                                false /* is_renderer_initiated */);
+  GetGuestWebContents()->GetDelegate()->OpenURLFromTab(
+      GetGuestWebContents(), params);
+
+  ASSERT_TRUE(load_listener.WaitUntilSatisfied());
+
+  // Verify that the <webview> ends up at about:blank.

[Charlie Reis, 2015/07/17 00:05:15]

This comment is stale.

[wjmaclean, 2015/07/17 01:29:46]

Done.

+  EXPECT_EQ(test_url, GetGuestWebContents()->GetLastCommittedURL());
+}
+
 IN_PROC_BROWSER_TEST_F(WebViewNewWindowTest, OpenURLFromTab_NewWindow_Abort) {
   LoadAppWithGuest("web_view/simple");
 
   // Verify that OpenURLFromTab with a window disposition of NEW_BACKGROUND_TAB
   // will trigger the <webview>'s New Window API.
   ExtensionTestMessageListener new_window_listener(
       "WebViewTest.NEWWINDOW", false);
 
   // Navigating to a file URL is forbidden inside a <webview>.
   content::OpenURLParams params(GURL("file://foo"),
@@ skipping 961 matching lines @@
   // Generate and send synthetic touch event.
   FocusWaiter waiter(aura_webview);
   content::SimulateTouchPressAt(GetEmbedderWebContents(),
                                 guest_rect.CenterPoint());
 
   // Wait for the TouchStart to propagate and restore focus. Test times out
   // on failure.
   waiter.Wait();
 }
 #endif