Fix incorrect creation of duplicate navigation entries for repeated page load failures.

content/renderer/render_frame_impl.cc

Index: content/renderer/render_frame_impl.cc
diff --git a/content/renderer/render_frame_impl.cc b/content/renderer/render_frame_impl.cc
index 57a29fc7b4a32506dca003f56547ecfec8849d14..86144ef47032b426bd488dea71fae6c0cd8ba3d8 100644
--- a/content/renderer/render_frame_impl.cc
+++ b/content/renderer/render_frame_impl.cc
@@ -2519,7 +2519,7 @@ void RenderFrameImpl::didFailProvisionalLoad(
       routing_id_, params));
 
   // Don't display an error page if this is simply a cancelled load.  Aside
-  // from being dumb, WebCore doesn't expect it and it will cause a crash.
+  // from being dumb, Blink doesn't expect it and it will cause a crash.
   if (error.reason == net::ERR_ABORTED)
     return;
 
@@ -2551,16 +2551,7 @@ void RenderFrameImpl::didFailProvisionalLoad(
   // 'replace' load.  This is necessary to avoid messing up session history.
   // Otherwise, we do a normal load, which simulates a 'go' navigation as far
   // as session history is concerned.
-  //
-  // AUTO_SUBFRAME loads should always be treated as loads that do not advance
-  // the page id.
-  //
-  // TODO(davidben): This should also take the failed navigation's replacement
-  // state into account, if a location.replace() failed.
-  bool replace =
-      navigation_state->request_params().page_id != -1 ||
-      ui::PageTransitionCoreTypeIs(navigation_state->GetTransitionType(),
-                                   ui::PAGE_TRANSITION_AUTO_SUBFRAME);
+  bool replace = commit_type != blink::WebStandardCommit;
 
   // If we failed on a browser initiated request, then make sure that our error
   // page load is regarded as the same browser initiated request.