Count number of origins with data affected by clearing "cookies and site data".

chrome/browser/browsing_data/browsing_data_quota_helper_impl.cc

Index: chrome/browser/browsing_data/browsing_data_quota_helper_impl.cc
diff --git a/chrome/browser/browsing_data/browsing_data_quota_helper_impl.cc b/chrome/browser/browsing_data/browsing_data_quota_helper_impl.cc
index 80b9848b3e753c9257c3be0efe06d6e9d332f263..cafaa58e4d82a4f4df843fd70bccec460e35550c 100644
--- a/chrome/browser/browsing_data/browsing_data_quota_helper_impl.cc
+++ b/chrome/browser/browsing_data/browsing_data_quota_helper_impl.cc
@@ -149,11 +149,13 @@ void BrowsingDataQuotaHelperImpl::OnGetHostsUsageComplete(
   QuotaInfoArray result;
   for (auto& pair : *quota_info) {
     QuotaInfo& info = pair.second;
+    // TODO? This code skips quota entries without size and hides them this way

[msramek, 2017/01/09 12:54:43]

I see. This is a bug somewhere in StoragePartition / QuotaManager. Could you
please file a bug and either remove the TODO from here, or change it to
TODO(crbug.com/123456)?

[dullweber, 2017/01/09 16:05:45]

I created https://bugs.chromium.org/p/chromium/issues/detail?id=679330 and
removed the todo.

+    // but this doesn't fix the issue that there are still quota entries left
+    // behind after deletion.
     // Skip unused entries
     if (info.temporary_usage <= 0 && info.persistent_usage <= 0 &&
         info.syncable_usage <= 0)

[msramek, 2017/01/09 12:54:43]

style nit: I'd prefer keeping the empty line below for better readabilty. While
you're here can you please also add braces? if without braces is only allowed if
the condition fits on one line.

[dullweber, 2017/01/09 16:05:45]

Done.

       continue;
-
     info.host = pair.first;
     result.push_back(info);
   }