Ensure that when a new navigation cancels an existing one, the old navigation's DidFinishNavigati...

content/browser/frame_host/navigation_handle_impl_browsertest.cc

 // Copyright 2016 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/memory/weak_ptr.h"
 #include "content/browser/frame_host/navigation_handle_impl.h"
 #include "content/browser/web_contents/web_contents_impl.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/browser/web_contents_observer.h"
 #include "content/public/common/browser_side_navigation_policy.h"
@@ skipping 1102 matching lines @@
   // EXPECT_EQ(net::ERR_BLOCKED_BY_CLIENT, observer.net_error_code());
 
   // Only one navigation is expected to happen.
   std::vector<GURL> started_navigation = {kRedirectingUrl};
   EXPECT_EQ(started_navigation, logger.started_navigation_urls());
 
   std::vector<GURL> finished_navigation = {kUrl};
   EXPECT_EQ(finished_navigation, logger.finished_navigation_urls());
 }
 
+// Tests that when a navigation starts while there's an existing one, the first
+// one has the right error code set on its navigation handle.
+IN_PROC_BROWSER_TEST_F(NavigationHandleImplBrowserTest, ErrorCodeOnCancel) {
+  GURL slow_url = embedded_test_server()->GetURL("/slow?60");
+  NavigationHandleObserver observer(shell()->web_contents(), slow_url);
+  shell()->LoadURL(slow_url);
+
+  GURL url2(embedded_test_server()->GetURL("/title1.html"));
+  TestNavigationObserver same_tab_observer(
+      shell()->web_contents(), 1);
+  shell()->LoadURL(url2);
+  same_tab_observer.Wait();
+
+  EXPECT_EQ(net::ERR_ABORTED, observer.net_error_code());
+}
+
 // This class allows running tests with PlzNavigate enabled, regardless of
 // default test configuration.
 class PlzNavigateNavigationHandleImplBrowserTest : public ContentBrowserTest {
  public:
   PlzNavigateNavigationHandleImplBrowserTest() {}
 
   void SetUpCommandLine(base::CommandLine* command_line) override {
     command_line->AppendSwitch(switches::kEnableBrowserSideNavigation);
   }
 };
@@ skipping 63 matching lines @@
     NavigationHandleObserver observer(shell()->web_contents(), error_url);
     EXPECT_FALSE(NavigateToURL(shell(), error_url));
     EXPECT_TRUE(observer.has_committed());
     EXPECT_TRUE(observer.is_error());
     EXPECT_NE(site_instance,
               shell()->web_contents()->GetMainFrame()->GetSiteInstance());
   }
 }
 
 }  // namespace content