PlzNavigate: Show error pages when the navigation failed before commit

content/browser/browser_side_navigation_browsertest.cc

 // Copyright 2014 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 "base/basictypes.h"
 #include "base/command_line.h"
 #include "base/strings/stringprintf.h"
 #include "content/browser/web_contents/web_contents_impl.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/common/content_switches.h"
@@ skipping 141 matching lines @@
     EXPECT_TRUE(WaitForLoadStop(shell()->web_contents()));
     EXPECT_EQ(url, observer.last_navigation_url());
     EXPECT_TRUE(observer.last_navigation_succeeded());
   }
 
   // The RenderFrameHost should not have changed.
   EXPECT_EQ(initial_rfh, static_cast<WebContentsImpl*>(shell()->web_contents())
                              ->GetFrameTree()->root()->current_frame_host());
 }
 
+// Ensure that browser side navigation handles navigation failures.
+IN_PROC_BROWSER_TEST_F(BrowserSideNavigationBrowserTest, FailedNavigation) {
+  // Perform a navigation with no live renderer.
+  {
+    TestNavigationObserver observer(shell()->web_contents());
+    GURL url(embedded_test_server()->GetURL("/title1.html"));
+    NavigateToURL(shell(), url);
+    EXPECT_EQ(url, observer.last_navigation_url());
+    EXPECT_TRUE(observer.last_navigation_succeeded());
+  }
+
+  // Now navigate to an unreachable url.
+  {
+    TestNavigationObserver observer(shell()->web_contents());
+    GURL url("http://127.0.0.1:4");

[clamy, 2015/04/10 14:04:49]

The url was chosen to get a network error (otherwise the test server returns a
404 which does not result in a navigation failure).

[Charlie Reis, 2015/04/10 22:54:44]

I think we have better ways to do this.  Please see
NavigationControllerBrowserTest.ErrorPageReplacement:

  GURL error_url(
      net::URLRequestFailedJob::GetMockHttpUrl(net::ERR_CONNECTION_RESET));
  net::URLRequestFailedJob::AddUrlHandler();

(I'm not 100% sure how that works, but I think this is what it's for.)

[clamy, 2015/04/14 11:56:25]

Done.

+    NavigateToURL(shell(), url);
+    EXPECT_EQ(url, observer.last_navigation_url());
+    EXPECT_EQ(1, observer.navigations_failed());
+    // TODO(clamy): when start/stop loading events are properly reported, update
+    // this test to check for the status of the last navigation.
+  }
+}
+
 }  // namespace content