PlzNavigate: don't stop all loaders when renderer-initiated nav fails

content/browser/frame_host/navigation_request.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 "content/browser/frame_host/navigation_request.h"
 
 #include <utility>
 
 #include "base/memory/ptr_util.h"
 #include "content/browser/appcache/appcache_navigation_handle.h"
@@ skipping 201 matching lines @@
   // While the navigation was started via the LoadURL path it may have come from
   // the renderer in the first place as part of OpenURL.
   bool browser_initiated = !entry.is_renderer_initiated();
 
   CommonNavigationParams common_params = entry.ConstructCommonNavigationParams(
       frame_entry, request_body, dest_url, dest_referrer, navigation_type,
       previews_state, navigation_start);
 
   std::unique_ptr<NavigationRequest> navigation_request(new NavigationRequest(
       frame_tree_node, common_params,
-      BeginNavigationParams(entry.extra_headers(), net::LOAD_NORMAL,
-                            false,  // has_user_gestures
-                            false,  // skip_service_worker
-                            REQUEST_CONTEXT_TYPE_LOCATION,
-                            blink::WebMixedContentContextType::Blockable,
-                            // TODO(arthursonzogni): It can be true for form
-                            // resubmission when the user reloads the page. This
-                            // needs to be fixed.
-                            false,  // is_form_submission
-                            initiator),
+      BeginNavigationParams(
+          entry.extra_headers(), net::LOAD_NORMAL,
+          false,  // has_user_gestures
+          false,  // skip_service_worker
+          REQUEST_CONTEXT_TYPE_LOCATION,
+          blink::WebMixedContentContextType::Blockable,
+          // TODO(arthursonzogni): It can be true for form
+          // resubmission when the user reloads the page. This
+          // needs to be fixed.
+          false,  // is_form_submission
+          initiator,
+          kInvalidRenderNavigationId),  // renderer_navigation_id
       entry.ConstructRequestNavigationParams(
           frame_entry, common_params.url, common_params.method,
           is_history_navigation_in_new_child,
           entry.GetSubframeUniqueNames(frame_tree_node),
           frame_tree_node->has_committed_real_load(),
           controller->GetPendingEntryIndex() == -1,
           controller->GetIndexOfEntry(&entry),
           controller->GetLastCommittedEntryIndex(),
           controller->GetEntryCount()),
       browser_initiated,
@@ skipping 581 matching lines @@
   DCHECK_EQ(request_params_.has_user_gesture, begin_params_.has_user_gesture);
 
   render_frame_host->CommitNavigation(response_.get(), std::move(body_),
                                       common_params_, request_params_,
                                       is_view_source_);
 
   frame_tree_node_->ResetNavigationRequest(true, true);
 }
 
 }  // namespace content