history service: Turn DCHECKs to CHECKs to try to diagnose crash bug.

Bug: 1009795
Change-Id: Ib33c1242228e674442f21dea8a56ef456d84b886
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1899231Reviewed-by: Scott Violet <sky@chromium.org>
Commit-Queue: Matt Falkenhagen <falken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#712948}

components/history/core/browser/history_service.cc

@@ -491,8 +491,10 @@ base::CancelableTaskTracker::TaskId HistoryService::GetFaviconsForURL(
     favicon_base::FaviconResultsCallback callback,
     base::CancelableTaskTracker* tracker) {
   TRACE_EVENT0("browser", "HistoryService::GetFaviconsForURL");
-  DCHECK(backend_task_runner_) << "History service being called after cleanup";
-  DCHECK(thread_checker_.CalledOnValidThread());
+  // TODO(crbug.com/1009795): Return the CHECKs to DCHECKs after the bug is
+  // fixed.
+  CHECK(thread_checker_.CalledOnValidThread());
+  CHECK(backend_task_runner_) << "History service being called after cleanup";
   return tracker->PostTaskAndReplyWithResult(
       backend_task_runner_.get(), FROM_HERE,
       base::BindOnce(&HistoryBackend::GetFaviconsForURL, history_backend_,
@@ -640,8 +642,10 @@ void HistoryService::SetOnDemandFavicons(
     const GURL& icon_url,
     const std::vector<SkBitmap>& bitmaps,
     base::OnceCallback<void(bool)> callback) {
-  DCHECK(backend_task_runner_) << "History service being called after cleanup";
-  DCHECK(thread_checker_.CalledOnValidThread());
+  // TODO(crbug.com/1009795): Return the CHECKs to DCHECKs after the bug is
+  // fixed.
+  CHECK(thread_checker_.CalledOnValidThread());
+  CHECK(backend_task_runner_) << "History service being called after cleanup";
   if (history_client_ && !history_client_->CanAddURL(page_url)) {
     std::move(callback).Run(false);
     return;