Remove has-committed-navigation DCHECK in LoadPostCommitErrorPage

We hit some cases where we got an empty URL after initial commit. This
is fine since we'll change the URL to about:blank anyway, so this CL
removes the DCHECK we're hitting.

Bug: 1159779
Change-Id: Ib0ff0bb6f876f5530bec091acb0d0ed7a9a952f3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2596947Reviewed-by: Nasko Oskov <nasko@chromium.org>
Commit-Queue: Rakina Zata Amni <rakina@chromium.org>
Cr-Commit-Position: refs/heads/master@{#839016}

content/browser/renderer_host/navigation_controller_impl.cc

@@ -3697,11 +3697,11 @@ void NavigationControllerImpl::LoadPostCommitErrorPage(
 
   mojom::CommonNavigationParamsPtr common_params =
       CreateCommonNavigationParams();
-  // |url| might be empty if LoadPostCommitErrorPage happens before the frame
-  // actually committed (e.g. iframe with "src" set to a slow-responding URL).
-  // We should rewrite the URL to about:blank in this case, as the renderer will
-  // only think a page is an error page if it has a non-empty unreachable URL.
-  DCHECK(!url.is_empty() || !rfhi->has_committed_any_navigation());
+  // |url| might be empty, such as when LoadPostCommitErrorPage happens before
+  // the frame actually committed (e.g. iframe with "src" set to a
+  // slow-responding URL). We should rewrite the URL to about:blank in this
+  // case, as the renderer will only think a page is an error page if it has a
+  // non-empty unreachable URL.
   common_params->url = url.is_empty() ? GURL("about:blank") : url;
   mojom::CommitNavigationParamsPtr commit_params =
       CreateCommitNavigationParams();