Change a CHECK to DCHECK in LocalSiteCharacteristicsDataStore

This CHECK hasn't tripped in the recent Canary builds, it's safe to turn
it back into a DCHECK.

Bug: 966059
Change-Id: Ieeb23543146fb276d601790638b3d3fbd1880c44
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1760279
Auto-Submit: Sébastien Marchand <sebmarchand@chromium.org>
Reviewed-by: François Doray <fdoray@chromium.org>
Commit-Queue: Sébastien Marchand <sebmarchand@chromium.org>
Cr-Commit-Position: refs/heads/master@{#689136}

chrome/browser/resource_coordinator/local_site_characteristics_data_store.cc

@@ -165,11 +165,8 @@ void LocalSiteCharacteristicsDataStore::OnURLsDeleted(
   } else {
     std::vector<url::Origin> origins_to_remove;
 
-    // CHECK_EQ instead of DCHECK_EQ to determine if
-    // https://crbug.com/966059 still happens in production.
-    // TODO(sebmarchand): Remove once https://crbug.com/966059 is fixed.
-    CHECK_EQ(deletion_info.deleted_urls_origin_map().size(),
-             CountOriginsInURLRows(deletion_info.deleted_rows()));
+    DCHECK_EQ(deletion_info.deleted_urls_origin_map().size(),
+              CountOriginsInURLRows(deletion_info.deleted_rows()));
     for (const auto& it : deletion_info.deleted_urls_origin_map()) {
       const url::Origin origin = url::Origin::Create(it.first);
       const int remaining_visits_in_history = it.second.first;