Commit fe336278 authored by Victor Costan's avatar Victor Costan Committed by Commit Bot

Notifications: fix TODO in notification_database.cc.

Change-Id: If9bd995760570480c0ab0c13de09e6e84d07a8fa
Reviewed-on: https://chromium-review.googlesource.com/1098687Reviewed-by: default avatarPeter Beverloo <peter@chromium.org>
Commit-Queue: Victor Costan <pwnall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#566972}
parent f77f6b8d
...@@ -424,11 +424,7 @@ NotificationDatabase::DeleteAllNotificationDataInternal( ...@@ -424,11 +424,7 @@ NotificationDatabase::DeleteAllNotificationDataInternal(
batch.Delete(iter->key()); batch.Delete(iter->key());
// Silently remove the notification if it doesn't have an ID assigned. DCHECK(!notification_database_data.notification_id.empty());
// TODO(peter): Remove this clause when Chrome 55 has branched.
if (notification_database_data.notification_id.empty())
continue;
deleted_notification_ids->insert( deleted_notification_ids->insert(
notification_database_data.notification_id); notification_database_data.notification_id);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment