Revert "Add DumpWithoutCrashing to try find navigations that later hit a CHECK."

This reverts commit bea26525.

Reason for revert: This debugging has made it into M88 but it's a
too noisy to allow into the release. This revert will be cherry-
picked and then the debugging will be restored and improved.

Original change's description:
> Add DumpWithoutCrashing to try find navigations that later hit a CHECK.
>
>
>
> Bug: 1146573
> Change-Id: I695cecab30b60028a6003245da150b1a72089706
> Cq-Do-Not-Cancel-Tryjobs: true
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2531334
> Commit-Queue: Fergal Daly <fergal@chromium.org>
> Reviewed-by: Alex Moshchuk <alexmos@chromium.org>
> Auto-Submit: Fergal Daly <fergal@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#826624}

TBR=alexmos@chromium.org,fergal@chromium.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: 1146573
Change-Id: I8b1c55e206f1b2b0353df2eb8f303b18efd13bc6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2543523Reviewed-by: Fergal Daly <fergal@chromium.org>
Reviewed-by: Alex Moshchuk <alexmos@chromium.org>
Commit-Queue: Fergal Daly <fergal@chromium.org>
Commit-Queue: Alex Moshchuk <alexmos@chromium.org>
Auto-Submit: Fergal Daly <fergal@chromium.org>
Cr-Commit-Position: refs/heads/master@{#828061}

content/browser/renderer_host/navigation_request.cc

@@ -66,7 +66,6 @@
 #include "content/browser/web_package/web_bundle_utils.h"
 #include "content/common/appcache_interfaces.h"
 #include "content/common/content_constants_internal.h"
-#include "content/common/content_navigation_policy.h"
 #include "content/common/frame_messages.h"
 #include "content/common/navigation_params.h"
 #include "content/common/navigation_params_mojom_traits.h"
@@ -3204,22 +3203,6 @@ void NavigationRequest::AddOldPageInfoToCommitParamsIfNeeded() {
 }
 
 void NavigationRequest::CommitNavigation() {
-  // TODO(https://crbug.com/1146573): Remove this when the bug is closed.
-  RenderFrameHostManager* manager = frame_tree_node_->render_manager();
-  if (RenderFrameHost* speculative_rfh = manager->speculative_frame_host()) {
-    RenderFrameHost* current_rfh = manager->current_frame_host();
-    if (speculative_rfh != current_rfh &&
-        speculative_rfh->GetSiteInstance() == current_rfh->GetSiteInstance() &&
-        !ShouldCreateNewHostForSameSiteSubframe()) {
-      SCOPED_CRASH_KEY_STRING256(
-          CommitNavigation, OldSiteInstance,
-          current_rfh->GetSiteInstance()->GetSiteURL().spec());
-      SCOPED_CRASH_KEY_STRING256(
-          CommitNavigation, NewSiteInstance,
-          speculative_rfh->GetSiteInstance()->GetSiteURL().spec());
-      base::debug::DumpWithoutCrashing();
-    }
-  }
   UpdateCommitNavigationParamsHistory();
   DCHECK(NeedsUrlLoader() == !!response_head_ ||
          (was_redirected_ && common_params_->url.IsAboutBlank()));