Remove DumpWithoutCrashing for bug 1090859
I added this DumpWithoutCrashing to investigate
https://crbug.com/1090859. This has never been hit, but I forget to
remove it. In the meantime, the number of crashes quickly vanished to
almost zero and nobody continued working on this bug.
Time to remove it! This reverts:
https://chromium-review.googlesource.com/c/chromium/src/+/2241672
Note: A new different bug appeared recently:
https://bugs.chromium.org/p/chromium/issues/detail?id=1262203
They are a priori not related, because they are failing on a different
CHECK on the same function.
Bug: 1090859
Change-Id: I5a7f9c237713cafa1473176296b8572aa3b36684
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3236734
Auto-Submit: Arthur Sonzogni <arthursonzogni@chromium.org>
Reviewed-by: danakj <danakj@chromium.org>
Commit-Queue: Arthur Sonzogni <arthursonzogni@chromium.org>
Cr-Commit-Position: refs/heads/main@{#935877}
diff --git a/content/browser/renderer_host/navigation_request.cc b/content/browser/renderer_host/navigation_request.cc
index 3ef015f..d4a59f3 100644
--- a/content/browser/renderer_host/navigation_request.cc
+++ b/content/browser/renderer_host/navigation_request.cc
@@ -3336,9 +3336,6 @@
was_redirected_ /* has_followed_redirect */,
false /* url_upgraded_after_redirect */, true /* is_response_check */);
DCHECK_NE(net_error, net::ERR_BLOCKED_BY_CLIENT);
- // TODO(https://crbug.com/1090859): Remove this once the bug has been fixed.
- if (net_error == net::ERR_BLOCKED_BY_CLIENT)
- base::debug::DumpWithoutCrashing();
if (net_error != net::OK) {
OnRequestFailedInternal(network::URLLoaderCompletionStatus(net_error),
false /* skip_throttles */,
