Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(13)

Unified Diff: components/safe_browsing_db/v4_local_database_manager.cc

Issue 2480203002: ui: Cleanup class/struct forward declarations (Closed)
Patch Set: Sync CL to position 430550 Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « components/profile_metrics/counts.cc ('k') | components/safe_browsing_db/v4_protocol_manager_util.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/safe_browsing_db/v4_local_database_manager.cc
diff --git a/components/safe_browsing_db/v4_local_database_manager.cc b/components/safe_browsing_db/v4_local_database_manager.cc
index 5e624abba7995bd74d50b78b96c8297b392d23a3..b9676df6725c8013e5375a804c3ef5678a942790 100644
--- a/components/safe_browsing_db/v4_local_database_manager.cc
+++ b/components/safe_browsing_db/v4_local_database_manager.cc
@@ -524,6 +524,11 @@ void V4LocalDatabaseManager::RespondToClient(
if (check->client_callback_type == ClientCallbackType::CHECK_BROWSE_URL) {
DCHECK_EQ(1u, check->urls.size());
+ // TODO(vakh): Remove these CHECKs after fixing bugs 660293, 660359.
+ CHECK(check.get());
+ CHECK(check->client);
+ CHECK_LE(1u, check->urls.size());
+ CHECK(check->urls[0].is_valid());
check->client->OnCheckBrowseUrlResult(
check->urls[0], check->result_threat_type, check->url_metadata);
} else if (check->client_callback_type ==
« no previous file with comments | « components/profile_metrics/counts.cc ('k') | components/safe_browsing_db/v4_protocol_manager_util.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698