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

chrome/browser/browsing_data/browsing_data_quota_helper_impl.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/browsing_data/browsing_data_quota_helper_impl.h"
 
 #include <map>
 #include <set>
 
 #include "base/barrier_closure.h"
@@ skipping 131 matching lines @@
 }
 
 void BrowsingDataQuotaHelperImpl::OnGetHostsUsageComplete(
     const FetchResultCallback& callback,
     QuotaInfoMap* quota_info) {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
 
   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);
   }
 
   BrowserThread::PostTask(BrowserThread::UI, FROM_HERE,
                           base::Bind(callback, result));
 }
 
 void BrowsingDataQuotaHelperImpl::RevokeHostQuotaOnIOThread(
     const std::string& host) {
   quota_manager_->SetPersistentHostQuota(
       host, 0, base::Bind(&BrowsingDataQuotaHelperImpl::DidRevokeHostQuota,
                           weak_factory_.GetWeakPtr()));
 }
 
 void BrowsingDataQuotaHelperImpl::DidRevokeHostQuota(
     storage::QuotaStatusCode /*status*/,
     int64_t /*quota*/) {}