Adding usage entry to chrome://settings/cookies.

BUG=88644,91816,91836
TEST='BrowsingDataQuotaHelperTest.*'


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=95607
Reverted: http://src.chromium.org/viewvc/chrome?view=rev&revision=95779

Review URL: http://codereview.chromium.org/7387007

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@95959 0039d316-1c4b-4281-b951-d872f2087c98

chrome/browser/browsing_data_quota_helper.cc

+// Copyright (c) 2011 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_quota_helper.h"
+
+BrowsingDataQuotaHelper::QuotaInfo::QuotaInfo()
+    : temporary_usage(0),
+      persistent_usage(0) {}
+
+BrowsingDataQuotaHelper::QuotaInfo::QuotaInfo(const std::string& host)
+    : host(host),
+      temporary_usage(0),
+      persistent_usage(0) {}
+
+BrowsingDataQuotaHelper::QuotaInfo::QuotaInfo(const std::string& host,
+                                              int64 temporary_usage,
+                                              int64 persistent_usage)
+    : host(host),
+      temporary_usage(temporary_usage),
+      persistent_usage(persistent_usage) {}
+
+BrowsingDataQuotaHelper::QuotaInfo::~QuotaInfo() {}
+
+// static
+void BrowsingDataQuotaHelperDeleter::Destruct(
+    const BrowsingDataQuotaHelper* helper) {
+  helper->io_thread_->DeleteSoon(FROM_HERE, helper);
+}
+
+BrowsingDataQuotaHelper::BrowsingDataQuotaHelper(
+    base::MessageLoopProxy* io_thread)
+    : io_thread_(io_thread) {
+}
+
+BrowsingDataQuotaHelper::~BrowsingDataQuotaHelper() {
+}
+
+bool operator <(const BrowsingDataQuotaHelper::QuotaInfo& lhs,
+                const BrowsingDataQuotaHelper::QuotaInfo& rhs) {
+  if (lhs.host != rhs.host)
+    return lhs.host < rhs.host;
+  if (lhs.temporary_usage != rhs.temporary_usage)
+    return lhs.temporary_usage < rhs.temporary_usage;
+  return lhs.persistent_usage < rhs.persistent_usage;
+}
+
+bool operator ==(const BrowsingDataQuotaHelper::QuotaInfo& lhs,
+                 const BrowsingDataQuotaHelper::QuotaInfo& rhs) {
+  return lhs.host == rhs.host &&
+      lhs.temporary_usage == rhs.temporary_usage &&
+      lhs.persistent_usage == rhs.persistent_usage;
+}

chrome/browser/browsing_data_quota_helper.h

+// Copyright (c) 2011 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.
+
+#ifndef CHROME_BROWSER_BROWSING_DATA_QUOTA_HELPER_H_
+#define CHROME_BROWSER_BROWSING_DATA_QUOTA_HELPER_H_
+#pragma once
+
+#include <string>
+#include <vector>
+
+#include "base/callback_old.h"
+#include "base/memory/ref_counted.h"
+#include "base/message_loop_proxy.h"
+#include "base/time.h"
+#include "content/browser/browser_thread.h"
+#include "webkit/quota/quota_types.h"
+
+class Profile;
+
+namespace quota {
+class QuotaManager;
+}
+
+class BrowsingDataQuotaHelper;
+
+struct BrowsingDataQuotaHelperDeleter {
+  static void Destruct(const BrowsingDataQuotaHelper* helper);
+};
+
+// This class is an interface class to bridge between Cookies Tree and Unified
+// Quota System.  This class provides a way to get usage and quota information
+// through the instance.
+//
+// Call Create to create an instance for a profile and call StartFetching with
+// a callback to fetch information asynchronously. If result is no longer needed
+// after StartFetching, call CancelNotification to prevent callback.
+//
+// Parallel fetching is not allowed, a fetching task should start after end of
+// previous task.  All method of this class should called from UI thread.
+class BrowsingDataQuotaHelper
+    : public base::RefCountedThreadSafe<BrowsingDataQuotaHelper,
+                                        BrowsingDataQuotaHelperDeleter> {
+ public:
+  // QuotaInfo contains host-based quota and usage information for persistent
+  // and temporary storage.
+  struct QuotaInfo {
+    QuotaInfo();
+    explicit QuotaInfo(const std::string& host);
+    QuotaInfo(const std::string& host,
+              int64 temporary_usage,
+              int64 persistent_usage);
+    ~QuotaInfo();
+
+    std::string host;
+    int64 temporary_usage;
+    int64 persistent_usage;
+  };
+
+  typedef std::vector<QuotaInfo> QuotaInfoArray;
+  typedef Callback1<const QuotaInfoArray&>::Type FetchResultCallback;
+
+  static BrowsingDataQuotaHelper* Create(Profile* profile);
+
+  virtual void StartFetching(FetchResultCallback* callback) = 0;
+  virtual void CancelNotification() = 0;
+
+  // We don't support deletion now.
+  virtual void DeleteQuotaHost(const std::string& host) {}
+
+ protected:
+  explicit BrowsingDataQuotaHelper(base::MessageLoopProxy* io_thread_);
+  virtual ~BrowsingDataQuotaHelper();
+
+ private:
+  friend class DeleteTask<const BrowsingDataQuotaHelper>;
+  friend struct BrowsingDataQuotaHelperDeleter;
+  scoped_refptr<base::MessageLoopProxy> io_thread_;
+
+  DISALLOW_COPY_AND_ASSIGN(BrowsingDataQuotaHelper);
+};
+
+bool operator <(const BrowsingDataQuotaHelper::QuotaInfo& lhs,
+                const BrowsingDataQuotaHelper::QuotaInfo& rhs);
+bool operator ==(const BrowsingDataQuotaHelper::QuotaInfo& lhs,
+                 const BrowsingDataQuotaHelper::QuotaInfo& rhs);
+
+#endif  // CHROME_BROWSER_BROWSING_DATA_QUOTA_HELPER_H_

chrome/browser/browsing_data_quota_helper_impl.cc

+// Copyright (c) 2011 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_quota_helper_impl.h"
+
+#include <map>
+#include <set>
+
+#include "base/logging.h"
+#include "chrome/browser/profiles/profile.h"
+#include "webkit/quota/quota_manager.h"
+
+// static
+BrowsingDataQuotaHelper* BrowsingDataQuotaHelper::Create(Profile* profile) {
+  return new BrowsingDataQuotaHelperImpl(
+      BrowserThread::GetMessageLoopProxyForThread(BrowserThread::UI),
+      BrowserThread::GetMessageLoopProxyForThread(BrowserThread::IO),
+      profile->GetQuotaManager());
+}
+
+BrowsingDataQuotaHelperImpl::BrowsingDataQuotaHelperImpl(
+    base::MessageLoopProxy* ui_thread,
+    base::MessageLoopProxy* io_thread,
+    quota::QuotaManager* quota_manager)
+    : BrowsingDataQuotaHelper(io_thread),
+      quota_manager_(quota_manager),
+      is_fetching_(false),
+      ui_thread_(ui_thread),
+      io_thread_(io_thread),
+      callback_factory_(ALLOW_THIS_IN_INITIALIZER_LIST(this)) {
+  DCHECK(quota_manager);
+}
+
+BrowsingDataQuotaHelperImpl::~BrowsingDataQuotaHelperImpl() {}
+
+void BrowsingDataQuotaHelperImpl::StartFetching(FetchResultCallback* callback) {
+  DCHECK(callback);
+  DCHECK(!callback_.get());
+  DCHECK(!is_fetching_);
+  callback_.reset(callback);
+  quota_info_.clear();
+  is_fetching_ = true;
+
+  FetchQuotaInfo();
+}
+
+void BrowsingDataQuotaHelperImpl::CancelNotification() {
+  callback_.reset();
+}
+
+void BrowsingDataQuotaHelperImpl::FetchQuotaInfo() {
+  if (!io_thread_->BelongsToCurrentThread()) {
+    io_thread_->PostTask(
+        FROM_HERE,
+        NewRunnableMethod(
+            this,
+            &BrowsingDataQuotaHelperImpl::FetchQuotaInfo));
+    return;
+  }
+
+  quota_manager_->GetOriginsModifiedSince(
+      quota::kStorageTypeTemporary,
+      base::Time(),
+      callback_factory_.NewCallback(
+          &BrowsingDataQuotaHelperImpl::GotOrigins));
+}
+
+void BrowsingDataQuotaHelperImpl::GotOrigins(
+    const std::set<GURL>& origins, quota::StorageType type) {
+  for (std::set<GURL>::const_iterator itr = origins.begin();
+       itr != origins.end();
+       ++itr)
+    pending_hosts_.insert(std::make_pair(itr->host(), type));
+
+  DCHECK(type == quota::kStorageTypeTemporary ||
+         type == quota::kStorageTypePersistent);
+
+  if (type == quota::kStorageTypeTemporary) {
+    quota_manager_->GetOriginsModifiedSince(
+        quota::kStorageTypePersistent,
+        base::Time(),
+        callback_factory_.NewCallback(
+            &BrowsingDataQuotaHelperImpl::GotOrigins));
+  } else {
+    // type == quota::kStorageTypePersistent
+    ProcessPendingHosts();
+  }
+}
+
+void BrowsingDataQuotaHelperImpl::ProcessPendingHosts() {
+  if (pending_hosts_.empty()) {
+    OnComplete();
+    return;
+  }
+
+  PendingHosts::iterator itr = pending_hosts_.begin();
+  std::string host = itr->first;
+  quota::StorageType type = itr->second;
+  pending_hosts_.erase(itr);
+  GetHostUsage(host, type);
+}
+
+void BrowsingDataQuotaHelperImpl::GetHostUsage(const std::string& host,
+                                               quota::StorageType type) {
+  DCHECK(quota_manager_.get());
+  quota_manager_->GetHostUsage(
+      host, type,
+      callback_factory_.NewCallback(
+          &BrowsingDataQuotaHelperImpl::GotHostUsage));
+}
+
+void BrowsingDataQuotaHelperImpl::GotHostUsage(const std::string& host,
+                                               quota::StorageType type,
+                                               int64 usage) {
+  switch (type) {
+    case quota::kStorageTypeTemporary:
+      quota_info_[host].temporary_usage = usage;
+      break;
+    case quota::kStorageTypePersistent:
+      quota_info_[host].persistent_usage = usage;
+      break;
+    default:
+      NOTREACHED();
+  }
+  ProcessPendingHosts();
+}
+
+void BrowsingDataQuotaHelperImpl::OnComplete() {
+  // Check if CancelNotification was called
+  if (!callback_.get())
+    return;
+
+  if (!ui_thread_->BelongsToCurrentThread()) {
+    ui_thread_->PostTask(
+        FROM_HERE,
+        NewRunnableMethod(
+            this,
+            &BrowsingDataQuotaHelperImpl::OnComplete));
+    return;
+  }
+
+  is_fetching_ = false;
+
+  QuotaInfoArray result;
+  result.reserve(quota_info_.size());
+
+  for (std::map<std::string, QuotaInfo>::iterator itr = quota_info_.begin();
+       itr != quota_info_.end();
+       ++itr) {
+    QuotaInfo* info = &itr->second;
+    // Skip unused entries
+    if (info->temporary_usage <= 0 &&
+        info->persistent_usage <= 0)
+      continue;
+
+    info->host = itr->first;
+    result.push_back(*info);
+  }
+
+  callback_->Run(result);
+  callback_.reset();
+}

chrome/browser/browsing_data_quota_helper_impl.h

+// Copyright (c) 2011 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.
+
+#ifndef CHROME_BROWSER_BROWSING_DATA_QUOTA_HELPER_IMPL_H_
+#define CHROME_BROWSER_BROWSING_DATA_QUOTA_HELPER_IMPL_H_
+#pragma once
+
+#include <map>
+#include <set>
+#include <string>
+#include <utility>
+#include <vector>
+
+#include "base/callback_old.h"
+#include "base/memory/ref_counted.h"
+#include "base/memory/scoped_callback_factory.h"
+#include "base/memory/scoped_ptr.h"
+#include "base/time.h"
+#include "chrome/browser/browsing_data_quota_helper.h"
+#include "content/browser/browser_thread.h"
+#include "webkit/quota/quota_types.h"
+
+namespace quota {
+class QuotaManager;
+}
+
+// Implementation of BrowsingDataQuotaHelper.  Since a client of
+// BrowsingDataQuotaHelper should live in UI thread and QuotaManager lives in
+// IO thread, we have to communicate over thread using PostTask.
+class BrowsingDataQuotaHelperImpl : public BrowsingDataQuotaHelper {
+ public:
+  virtual void StartFetching(FetchResultCallback* callback) OVERRIDE;
+  virtual void CancelNotification() OVERRIDE;
+
+ private:
+  void FetchQuotaInfo();
+  void OnComplete();
+
+  void GetHostUsage(const std::string& host, quota::StorageType type);
+  void ProcessPendingHosts();
+
+  // Callback function for GetOriginModifiedSince.
+  void GotOrigins(const std::set<GURL>& origins, quota::StorageType type);
+
+  // Callback function for GetHostUsage.
+  void GotHostUsage(const std::string& host,
+                    quota::StorageType type,
+                    int64 usage);
+
+  explicit BrowsingDataQuotaHelperImpl(base::MessageLoopProxy* ui_thread,
+                                       base::MessageLoopProxy* io_thread,
+                                       quota::QuotaManager* quota_manager);
+  virtual ~BrowsingDataQuotaHelperImpl();
+
+  scoped_refptr<quota::QuotaManager> quota_manager_;
+  scoped_ptr<FetchResultCallback> callback_;
+
+  typedef std::set<std::pair<std::string, quota::StorageType> > PendingHosts;
+  PendingHosts pending_hosts_;
+  std::map<std::string, QuotaInfo> quota_info_;
+
+  bool is_fetching_;
+
+  scoped_refptr<base::MessageLoopProxy> ui_thread_;
+  scoped_refptr<base::MessageLoopProxy> io_thread_;
+  base::ScopedCallbackFactory<BrowsingDataQuotaHelperImpl> callback_factory_;
+
+  friend class BrowsingDataQuotaHelper;
+  friend class BrowsingDataQuotaHelperTest;
+
+  DISALLOW_COPY_AND_ASSIGN(BrowsingDataQuotaHelperImpl);
+};
+
+#endif  // CHROME_BROWSER_BROWSING_DATA_QUOTA_HELPER_IMPL_H_

chrome/browser/browsing_data_quota_helper_unittest.cc

+// Copyright (c) 2011 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 "testing/gtest/include/gtest/gtest.h"
+
+#include "base/memory/scoped_callback_factory.h"
+#include "base/message_loop_proxy.h"
+#include "base/scoped_temp_dir.h"
+#include "chrome/browser/browsing_data_quota_helper_impl.h"
+#include "webkit/quota/mock_storage_client.h"
+#include "webkit/quota/quota_manager.h"
+
+class BrowsingDataQuotaHelperTest : public testing::Test {
+ public:
+  typedef BrowsingDataQuotaHelper::QuotaInfo QuotaInfo;
+  typedef BrowsingDataQuotaHelper::QuotaInfoArray QuotaInfoArray;
+
+  BrowsingDataQuotaHelperTest()
+      : ui_thread_(BrowserThread::UI, &message_loop_),
+        db_thread_(BrowserThread::DB, &message_loop_),
+        io_thread_(BrowserThread::IO, &message_loop_),
+        fetching_completed_(true),
+        callback_factory_(ALLOW_THIS_IN_INITIALIZER_LIST(this)) {}
+
+  virtual ~BrowsingDataQuotaHelperTest() {}
+
+  virtual void SetUp() OVERRIDE {
+    EXPECT_TRUE(dir_.CreateUniqueTempDir());
+    quota_manager_ = new quota::QuotaManager(
+        false, dir_.path(),
+        BrowserThread::GetMessageLoopProxyForThread(BrowserThread::IO),
+        BrowserThread::GetMessageLoopProxyForThread(BrowserThread::DB),
+        NULL);
+    helper_ = new BrowsingDataQuotaHelperImpl(
+        BrowserThread::GetMessageLoopProxyForThread(BrowserThread::UI),
+        BrowserThread::GetMessageLoopProxyForThread(BrowserThread::IO),
+        quota_manager_);
+  }
+
+  virtual void TearDown() OVERRIDE {
+    helper_ = NULL;
+    quota_manager_ = NULL;
+    quota_info_.clear();
+    MessageLoop::current()->RunAllPending();
+  }
+
+ protected:
+  const QuotaInfoArray& quota_info() const {
+    return quota_info_;
+  }
+
+  bool fetching_completed() const {
+    return fetching_completed_;
+  }
+
+  void StartFetching() {
+    fetching_completed_ = false;
+    helper_->StartFetching(
+        callback_factory_.NewCallback(
+            &BrowsingDataQuotaHelperTest::FetchCompleted));
+  }
+
+  void RegisterClient(const quota::MockOriginData* data, std::size_t data_len) {
+    quota::MockStorageClient* client =
+        new quota::MockStorageClient(
+            quota_manager_->proxy(), data, data_len);
+    quota_manager_->proxy()->RegisterClient(client);
+    client->TouchAllOriginsAndNotify();
+  }
+
+ private:
+  void FetchCompleted(const QuotaInfoArray& quota_info) {
+    quota_info_ = quota_info;
+    fetching_completed_ = true;
+  }
+
+  MessageLoop message_loop_;
+  BrowserThread ui_thread_;
+  BrowserThread db_thread_;
+  BrowserThread io_thread_;
+  scoped_refptr<quota::QuotaManager> quota_manager_;
+
+  ScopedTempDir dir_;
+  scoped_refptr<BrowsingDataQuotaHelper> helper_;
+
+  bool fetching_completed_;
+  QuotaInfoArray quota_info_;
+  base::ScopedCallbackFactory<BrowsingDataQuotaHelperTest> callback_factory_;
+
+  DISALLOW_COPY_AND_ASSIGN(BrowsingDataQuotaHelperTest);
+};
+
+TEST_F(BrowsingDataQuotaHelperTest, Empty) {
+  StartFetching();
+  MessageLoop::current()->RunAllPending();
+  EXPECT_TRUE(fetching_completed());
+  EXPECT_TRUE(quota_info().empty());
+}
+
+TEST_F(BrowsingDataQuotaHelperTest, FetchData) {
+  const quota::MockOriginData kOrigins[] = {
+    {"http://example.com/", quota::kStorageTypeTemporary, 1},
+    {"https://example.com/", quota::kStorageTypeTemporary, 10},
+    {"http://example.com/", quota::kStorageTypePersistent, 100},
+    {"http://example2.com/", quota::kStorageTypeTemporary, 1000},
+  };
+
+  RegisterClient(kOrigins, arraysize(kOrigins));
+  StartFetching();
+  MessageLoop::current()->RunAllPending();
+  EXPECT_TRUE(fetching_completed());
+
+  std::set<QuotaInfo> expected, actual;
+  actual.insert(quota_info().begin(), quota_info().end());
+  expected.insert(QuotaInfo("example.com", 11, 100));
+  expected.insert(QuotaInfo("example2.com", 1000, 0));
+  EXPECT_TRUE(expected == actual);
+}

chrome/browser/content_settings/tab_specific_content_settings.cc

@@ -526,5 +526,6 @@ TabSpecificContentSettings::LocalSharedObjectsContainer::GetCookiesTreeModel() {
                               appcaches_->Clone(),
                               indexed_dbs_->Clone(),
                               file_systems_->Clone(),
+                              NULL,
                               true);
 }

chrome/browser/cookies_tree_model.h

@@ -21,6 +21,7 @@
 #include "chrome/browser/browsing_data_file_system_helper.h"
 #include "chrome/browser/browsing_data_indexed_db_helper.h"
 #include "chrome/browser/browsing_data_local_storage_helper.h"
+#include "chrome/browser/browsing_data_quota_helper.h"
 #include "chrome/common/content_settings.h"
 #include "net/base/cookie_monster.h"
 #include "ui/base/models/tree_node_model.h"
@@ -36,6 +37,7 @@ class CookieTreeFileSystemsNode;
 class CookieTreeFileSystemNode;
 class CookieTreeLocalStorageNode;
 class CookieTreeLocalStoragesNode;
+class CookieTreeQuotaNode;
 class CookieTreeSessionStorageNode;
 class CookieTreeSessionStoragesNode;
 class CookieTreeIndexedDBNode;
@@ -69,8 +71,9 @@ class CookieTreeNode : public ui::TreeNode<CookieTreeNode> {
       TYPE_APPCACHE,  // This is used for CookieTreeAppCacheNode.
       TYPE_INDEXED_DBS,  // This is used for CookieTreeIndexedDBsNode.
       TYPE_INDEXED_DB,  // This is used for CookieTreeIndexedDBNode.
-      TYPE_FILE_SYSTEMS, // This is used for CookieTreeFileSystemsNode.
-      TYPE_FILE_SYSTEM, // This is used for CookieTreeFileSystemNode.
+      TYPE_FILE_SYSTEMS,  // This is used for CookieTreeFileSystemsNode.
+      TYPE_FILE_SYSTEM,  // This is used for CookieTreeFileSystemNode.
+      TYPE_QUOTA,  // This is used for CookieTreeQuotaNode.
     };
 
     // TODO(viettrungluu): Figure out whether we want to store |origin| as a
@@ -85,7 +88,8 @@ class CookieTreeNode : public ui::TreeNode<CookieTreeNode> {
             session_storage_info,
         const appcache::AppCacheInfo* appcache_info,
         const BrowsingDataIndexedDBHelper::IndexedDBInfo* indexed_db_info,
-        const BrowsingDataFileSystemHelper::FileSystemInfo* file_system_info)
+        const BrowsingDataFileSystemHelper::FileSystemInfo* file_system_info,
+        const BrowsingDataQuotaHelper::QuotaInfo* quota_info)
         : origin(UTF16ToWideHack(origin)),
           node_type(node_type),
           cookie(cookie),
@@ -94,13 +98,15 @@ class CookieTreeNode : public ui::TreeNode<CookieTreeNode> {
           session_storage_info(session_storage_info),
           appcache_info(appcache_info),
           indexed_db_info(indexed_db_info),
-          file_system_info(file_system_info) {
+          file_system_info(file_system_info),
+          quota_info(quota_info) {
       DCHECK((node_type != TYPE_DATABASE) || database_info);
       DCHECK((node_type != TYPE_LOCAL_STORAGE) || local_storage_info);
       DCHECK((node_type != TYPE_SESSION_STORAGE) || session_storage_info);
       DCHECK((node_type != TYPE_APPCACHE) || appcache_info);
       DCHECK((node_type != TYPE_INDEXED_DB) || indexed_db_info);
       DCHECK((node_type != TYPE_FILE_SYSTEM) || file_system_info);
+      DCHECK((node_type != TYPE_QUOTA) || quota_info);
     }
 #if !defined(WCHAR_T_IS_UTF16)
     DetailedInfo(const std::wstring& origin, NodeType node_type,
@@ -112,7 +118,8 @@ class CookieTreeNode : public ui::TreeNode<CookieTreeNode> {
             session_storage_info,
         const appcache::AppCacheInfo* appcache_info,
         const BrowsingDataIndexedDBHelper::IndexedDBInfo* indexed_db_info,
-        const BrowsingDataFileSystemHelper::FileSystemInfo* file_system_info)
+        const BrowsingDataFileSystemHelper::FileSystemInfo* file_system_info,
+        const BrowsingDataQuotaHelper::QuotaInfo* quota_info)
         : origin(origin),
           node_type(node_type),
           cookie(cookie),
@@ -121,13 +128,15 @@ class CookieTreeNode : public ui::TreeNode<CookieTreeNode> {
           session_storage_info(session_storage_info),
           appcache_info(appcache_info),
           indexed_db_info(indexed_db_info),
-          file_system_info(file_system_info) {
+          file_system_info(file_system_info),
+          quota_info(quota_info) {
       DCHECK((node_type != TYPE_DATABASE) || database_info);
       DCHECK((node_type != TYPE_LOCAL_STORAGE) || local_storage_info);
       DCHECK((node_type != TYPE_SESSION_STORAGE) || session_storage_info);
       DCHECK((node_type != TYPE_APPCACHE) || appcache_info);
       DCHECK((node_type != TYPE_INDEXED_DB) || indexed_db_info);
       DCHECK((node_type != TYPE_FILE_SYSTEM) || file_system_info);
+      DCHECK((node_type != TYPE_QUOTA) || quota_info);
     }
 #endif
 
@@ -141,6 +150,7 @@ class CookieTreeNode : public ui::TreeNode<CookieTreeNode> {
     const appcache::AppCacheInfo* appcache_info;
     const BrowsingDataIndexedDBHelper::IndexedDBInfo* indexed_db_info;
     const BrowsingDataFileSystemHelper::FileSystemInfo* file_system_info;
+    const BrowsingDataQuotaHelper::QuotaInfo* quota_info;
   };
 
   CookieTreeNode() {}
@@ -211,6 +221,8 @@ class CookieTreeOriginNode : public CookieTreeNode {
   CookieTreeAppCachesNode* GetOrCreateAppCachesNode();
   CookieTreeIndexedDBsNode* GetOrCreateIndexedDBsNode();
   CookieTreeFileSystemsNode* GetOrCreateFileSystemsNode();
+  CookieTreeQuotaNode* UpdateOrCreateQuotaNode(
+      BrowsingDataQuotaHelper::QuotaInfo* quota_info);
 
   // Creates an content exception for this origin of type
   // CONTENT_SETTINGS_TYPE_COOKIES.
@@ -233,6 +245,7 @@ class CookieTreeOriginNode : public CookieTreeNode {
   CookieTreeAppCachesNode* appcaches_child_;
   CookieTreeIndexedDBsNode* indexed_dbs_child_;
   CookieTreeFileSystemsNode* file_systems_child_;
+  CookieTreeQuotaNode* quota_child_;
 
   // The URL for which this node was initially created.
   GURL url_;
@@ -498,6 +511,24 @@ class CookieTreeIndexedDBsNode : public CookieTreeNode {
   DISALLOW_COPY_AND_ASSIGN(CookieTreeIndexedDBsNode);
 };
 
+// CookieTreeQuotaNode --------------------------------------------------
+class CookieTreeQuotaNode : public CookieTreeNode {
+ public:
+  // Does not take ownership of quota_info, and quota_info should remain valid
+  // at least as long as the CookieTreeQuotaNode is valid.
+  explicit CookieTreeQuotaNode(BrowsingDataQuotaHelper::QuotaInfo* quota_info);
+  virtual ~CookieTreeQuotaNode();
+
+  virtual void DeleteStoredObjects();
+  virtual DetailedInfo GetDetailedInfo() const;
+
+ private:
+  // quota_info_ is not owned by the node, and is expected to remain valid as
+  // long as the CookieTreeQuotaNode is valid.
+  BrowsingDataQuotaHelper::QuotaInfo* quota_info_;
+
+  DISALLOW_COPY_AND_ASSIGN(CookieTreeQuotaNode);
+};
 
 // CookiesTreeModel -----------------------------------------------------------
 class CookiesTreeModel : public ui::TreeNodeModel<CookieTreeNode> {
@@ -521,6 +552,7 @@ class CookiesTreeModel : public ui::TreeNodeModel<CookieTreeNode> {
       BrowsingDataAppCacheHelper* appcache_helper,
       BrowsingDataIndexedDBHelper* indexed_db_helper,
       BrowsingDataFileSystemHelper* file_system_helper,
+      BrowsingDataQuotaHelper* quota_helper,
       bool use_cookie_source);
   virtual ~CookiesTreeModel();
 
@@ -565,6 +597,7 @@ class CookiesTreeModel : public ui::TreeNodeModel<CookieTreeNode> {
       IndexedDBInfoList;
   typedef std::vector<BrowsingDataFileSystemHelper::FileSystemInfo>
       FileSystemInfoList;
+  typedef std::vector<BrowsingDataQuotaHelper::QuotaInfo> QuotaInfoArray;
 
   void LoadCookies();
   void LoadCookiesWithFilter(const std::wstring& filter);
@@ -579,6 +612,7 @@ class CookiesTreeModel : public ui::TreeNodeModel<CookieTreeNode> {
       const IndexedDBInfoList& indexed_db_info);
   void OnFileSystemModelInfoLoaded(
       const FileSystemInfoList& file_system_info);
+  void OnQuotaModelInfoLoaded(const QuotaInfoArray& quota_info);
 
   void PopulateAppCacheInfoWithFilter(const std::wstring& filter);
   void PopulateDatabaseInfoWithFilter(const std::wstring& filter);
@@ -586,6 +620,7 @@ class CookiesTreeModel : public ui::TreeNodeModel<CookieTreeNode> {
   void PopulateSessionStorageInfoWithFilter(const std::wstring& filter);
   void PopulateIndexedDBInfoWithFilter(const std::wstring& filter);
   void PopulateFileSystemInfoWithFilter(const std::wstring& filter);
+  void PopulateQuotaInfoWithFilter(const std::wstring& filter);
 
   void NotifyObserverBeginBatch();
   void NotifyObserverEndBatch();
@@ -602,10 +637,12 @@ class CookiesTreeModel : public ui::TreeNodeModel<CookieTreeNode> {
   scoped_refptr<BrowsingDataLocalStorageHelper> session_storage_helper_;
   scoped_refptr<BrowsingDataIndexedDBHelper> indexed_db_helper_;
   scoped_refptr<BrowsingDataFileSystemHelper> file_system_helper_;
+  scoped_refptr<BrowsingDataQuotaHelper> quota_helper_;
   LocalStorageInfoList local_storage_info_list_;
   LocalStorageInfoList session_storage_info_list_;
   IndexedDBInfoList indexed_db_info_list_;
   FileSystemInfoList file_system_info_list_;
+  QuotaInfoArray quota_info_list_;
 
   // The CookiesTreeModel maintains a separate list of observers that are
   // specifically of the type CookiesTreeModel::Observer.
@@ -626,6 +663,7 @@ class CookiesTreeModel : public ui::TreeNodeModel<CookieTreeNode> {
   friend class CookieTreeLocalStorageNode;
   friend class CookieTreeIndexedDBNode;
   friend class CookieTreeFileSystemNode;
+  friend class CookieTreeQuotaNode;
 
   DISALLOW_COPY_AND_ASSIGN(CookiesTreeModel);
 };

chrome/browser/mock_browsing_data_quota_helper.cc

+// Copyright (c) 2011 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/mock_browsing_data_quota_helper.h"
+
+MockBrowsingDataQuotaHelper::MockBrowsingDataQuotaHelper(Profile* profile)
+    : BrowsingDataQuotaHelper(BrowserThread::GetMessageLoopProxyForThread(
+        BrowserThread::IO)) {}
+
+MockBrowsingDataQuotaHelper::~MockBrowsingDataQuotaHelper() {}
+
+void MockBrowsingDataQuotaHelper::StartFetching(
+    FetchResultCallback* callback) {
+  callback_.reset(callback);
+}
+
+void MockBrowsingDataQuotaHelper::CancelNotification() {
+  callback_.reset(NULL);
+}
+
+void MockBrowsingDataQuotaHelper::AddHost(
+    const std::string& host,
+    int64 temporary_usage,
+    int64 persistent_usage) {
+  response_.push_back(QuotaInfo(
+      host,
+      temporary_usage,
+      persistent_usage));
+}
+
+void MockBrowsingDataQuotaHelper::AddQuotaSamples() {
+  AddHost("quotahost1", 1, 2);
+  AddHost("quotahost2", 10, 20);
+}
+
+void MockBrowsingDataQuotaHelper::Notify() {
+  CHECK(callback_.get());
+  callback_->Run(response_);
+  callback_.reset();
+  response_.clear();
+}

chrome/browser/mock_browsing_data_quota_helper.h

+// Copyright (c) 2011 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.
+
+#ifndef CHROME_BROWSER_MOCK_BROWSING_DATA_QUOTA_HELPER_H_
+#define CHROME_BROWSER_MOCK_BROWSING_DATA_QUOTA_HELPER_H_
+#pragma once
+
+#include <string>
+#include <vector>
+
+#include "base/compiler_specific.h"
+#include "base/memory/scoped_ptr.h"
+#include "chrome/browser/browsing_data_quota_helper.h"
+
+class MockBrowsingDataQuotaHelper : public BrowsingDataQuotaHelper {
+ public:
+  explicit MockBrowsingDataQuotaHelper(Profile* profile);
+
+  virtual void StartFetching(FetchResultCallback* callback) OVERRIDE;
+  virtual void CancelNotification() OVERRIDE;
+
+  void AddHost(const std::string& host,
+               int64 temporary_usage,
+               int64 persistent_usage);
+  void AddQuotaSamples();
+  void Notify();
+
+ private:
+  virtual ~MockBrowsingDataQuotaHelper();
+
+  scoped_ptr<FetchResultCallback> callback_;
+  std::vector<QuotaInfo> response_;
+  Profile* profile_;
+};
+
+#endif  // CHROME_BROWSER_MOCK_BROWSING_DATA_QUOTA_HELPER_H_

chrome/browser/resources/options/cookies_list.js

@@ -119,6 +119,8 @@ cr.define('options', function() {
       this.siteChild.className = 'cookie-site';
       this.dataChild = this.ownerDocument.createElement('div');
       this.dataChild.className = 'cookie-data';
+      this.sizeChild = this.ownerDocument.createElement('div');
+      this.sizeChild.className = 'cookie-size';
       this.itemsChild = this.ownerDocument.createElement('div');
       this.itemsChild.className = 'cookie-items';
       this.infoChild = this.ownerDocument.createElement('div');
@@ -131,6 +133,7 @@ cr.define('options', function() {
       var content = this.contentElement;
       content.appendChild(this.siteChild);
       content.appendChild(this.dataChild);
+      content.appendChild(this.sizeChild);
       content.appendChild(this.itemsChild);
       this.itemsChild.appendChild(this.infoChild);
       if (this.origin && this.origin.data) {
@@ -250,6 +253,10 @@ cr.define('options', function() {
         else
           text = list[i];
       this.dataChild.textContent = text;
+      if (info.quota && info.quota.totalUsage) {
+        this.sizeChild.textContent = info.quota.totalUsage;
+      }
+
       if (this.expanded)
         this.updateItems_();
     },
@@ -430,18 +437,21 @@ cr.define('options', function() {
         for (var i = 0; i < this.children.length; ++i)
           this.children[i].collectSummaryInfo(info);
       } else if (this.data && !this.data.hasChildren) {
-        if (this.data.type == 'cookie')
+        if (this.data.type == 'cookie') {
           info.cookies++;
-        else if (this.data.type == 'database')
+        } else if (this.data.type == 'database') {
           info.database = true;
-        else if (this.data.type == 'local_storage')
+        } else if (this.data.type == 'local_storage') {
           info.localStorage = true;
-        else if (this.data.type == 'app_cache')
+        } else if (this.data.type == 'app_cache') {
           info.appCache = true;
-        else if (this.data.type == 'indexed_db')
+        } else if (this.data.type == 'indexed_db') {
           info.indexedDb = true;
-        else if (this.data.type == 'file_system')
+        } else if (this.data.type == 'file_system') {
           info.fileSystem = true;
+        } else if (this.data.type == 'quota') {
+          info.quota = this.data;
+        }
       }
     },
 
@@ -474,6 +484,8 @@ cr.define('options', function() {
             text = localStrings.getString('cookie_file_system');
             break;
         }
+        if (!text)
+          return;
         var div = item.ownerDocument.createElement('div');
         div.className = 'cookie-item';
         // Help out screen readers and such: this is a clickable thing.

chrome/browser/resources/options/cookies_view.css

 /*
-Copyright (c) 2010 The Chromium Authors. All rights reserved.
+Copyright (c) 2011 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.
 */
@@ -88,6 +88,12 @@ list.cookie-list > .deletable-item[selected] .cookie-site {
   display: inline-block;
 }
 
+.cookie-size {
+  display: inline-block;
+  float: right;
+  margin-right: 3em;
+}
+
 list.cookie-list > .deletable-item[selected] .cookie-data {
   -webkit-user-select: text;
 }

chrome/browser/ui/webui/cookies_tree_model_util.cc

@@ -41,6 +41,13 @@ static const char kKeyModified[] = "modified";
 static const char kKeyPersistent[] = "persistent";
 static const char kKeyTemporary[] = "temporary";
 
+static const char kKeyTotalUsage[] = "totalUsage";
+static const char kKeyTemporaryUsage[] = "temporaryUsage";
+static const char kKeyPersistentUsage[] = "persistentUsage";
+static const char kKeyPersistentQuota[] = "persistentQuota";
+
+static const int64 kNegligibleUsage = 1024;  // 1KiB
+
 // Encodes a pointer value into a hex string.
 std::string PointerToHexString(const void* pointer) {
   return base::HexEncode(&pointer, sizeof(pointer));
@@ -65,7 +72,7 @@ std::string GetTreeNodeId(CookieTreeNode* node) {
   return PointerToHexString(node);
 }
 
-void GetCookieTreeNodeDictionary(const CookieTreeNode& node,
+bool GetCookieTreeNodeDictionary(const CookieTreeNode& node,
                                  DictionaryValue* dict) {
   // Use node's address as an id for WebUI to look it up.
   dict->SetString(kKeyId, PointerToHexString(&node));
@@ -190,12 +197,36 @@ void GetCookieTreeNodeDictionary(const CookieTreeNode& node,
                               IDS_COOKIES_FILE_SYSTEM_USAGE_NONE));
       break;
     }
+    case CookieTreeNode::DetailedInfo::TYPE_QUOTA: {
+      dict->SetString(kKeyType, "quota");
+      dict->SetString(kKeyIcon, "chrome://theme/IDR_COOKIE_STORAGE_ICON");
+
+      const BrowsingDataQuotaHelper::QuotaInfo& quota_info =
+          *node.GetDetailedInfo().quota_info;
+      if (quota_info.temporary_usage + quota_info.persistent_usage <=
+          kNegligibleUsage)
+        return false;
+
+      dict->SetString(kKeyOrigin, quota_info.host);
+      dict->SetString(kKeyTotalUsage,
+                      UTF16ToUTF8(ui::FormatBytes(
+                          quota_info.temporary_usage +
+                          quota_info.persistent_usage)));
+      dict->SetString(kKeyTemporaryUsage,
+                      UTF16ToUTF8(ui::FormatBytes(
+                          quota_info.temporary_usage)));
+      dict->SetString(kKeyPersistentUsage,
+                      UTF16ToUTF8(ui::FormatBytes(
+                          quota_info.persistent_usage)));
+      break;
+    }
     default:
 #if defined(OS_MACOSX)
       dict->SetString(kKeyIcon, "chrome://theme/IDR_BOOKMARK_BAR_FOLDER");
 #endif
       break;
   }
+  return true;
 }
 
 void GetChildNodeList(CookieTreeNode* parent, int start, int count,
@@ -203,8 +234,10 @@ void GetChildNodeList(CookieTreeNode* parent, int start, int count,
   for (int i = 0; i < count; ++i) {
     DictionaryValue* dict = new DictionaryValue;
     CookieTreeNode* child = parent->GetChild(start + i);
-    GetCookieTreeNodeDictionary(*child, dict);
-    nodes->Append(dict);
+    if (GetCookieTreeNodeDictionary(*child, dict))
+      nodes->Append(dict);
+    else
+      delete dict;
   }
 }
 

chrome/browser/ui/webui/cookies_tree_model_util.h

@@ -21,7 +21,8 @@ namespace cookies_tree_model_util {
 std::string GetTreeNodeId(CookieTreeNode* node);
 
 // Populate given |dict| with cookie tree node properties.
-void GetCookieTreeNodeDictionary(const CookieTreeNode& node,
+// Returns false if the |node| does not need to be shown.
+bool GetCookieTreeNodeDictionary(const CookieTreeNode& node,
                                  base::DictionaryValue* dict);
 
 // Append the children nodes of |parent| in specified range to |nodes| list.

chrome/browser/ui/webui/options/cookies_view_handler.cc

@@ -10,6 +10,7 @@
 #include "chrome/browser/browsing_data_database_helper.h"
 #include "chrome/browser/browsing_data_file_system_helper.h"
 #include "chrome/browser/browsing_data_indexed_db_helper.h"
+#include "chrome/browser/browsing_data_quota_helper.h"
 #include "chrome/browser/browsing_data_local_storage_helper.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/webui/cookies_tree_model_util.h"
@@ -66,7 +67,7 @@ void CookiesViewHandler::GetLocalizedValues(
     { "label_file_system_temporary_usage",
       IDS_COOKIES_FILE_SYSTEM_TEMPORARY_USAGE_LABEL },
     { "label_file_system_persistent_usage",
-      IDS_COOKIES_FILE_SYSTEM_PERSISTENT_USAGE_LABEL }
+      IDS_COOKIES_FILE_SYSTEM_PERSISTENT_USAGE_LABEL },
   };
 
   RegisterStrings(localized_strings, resources, arraysize(resources));
@@ -151,6 +152,7 @@ void CookiesViewHandler::EnsureCookiesTreeModelCreated() {
         new BrowsingDataAppCacheHelper(profile),
         BrowsingDataIndexedDBHelper::Create(profile),
         BrowsingDataFileSystemHelper::Create(profile),
+        BrowsingDataQuotaHelper::Create(profile),
         false));
     cookies_tree_model_->AddCookiesTreeObserver(this);
   }

chrome/chrome_browser.gypi

@@ -314,6 +314,10 @@
         'browser/browsing_data_indexed_db_helper.h',
         'browser/browsing_data_local_storage_helper.cc',
         'browser/browsing_data_local_storage_helper.h',
+        'browser/browsing_data_quota_helper.cc',
+        'browser/browsing_data_quota_helper.h',
+        'browser/browsing_data_quota_helper_impl.cc',
+        'browser/browsing_data_quota_helper_impl.h',
         'browser/browsing_data_remover.cc',
         'browser/browsing_data_remover.h',
         'browser/bug_report_data.cc',

chrome/chrome_tests.gypi

@@ -87,6 +87,8 @@
         'browser/mock_browsing_data_indexed_db_helper.h',
         'browser/mock_browsing_data_local_storage_helper.cc',
         'browser/mock_browsing_data_local_storage_helper.h',
+        'browser/mock_browsing_data_quota_helper.cc',
+        'browser/mock_browsing_data_quota_helper.h',
         'browser/notifications/notification_test_util.cc',
         'browser/notifications/notification_test_util.h',
         'browser/policy/mock_cloud_policy_data_store.cc',
@@ -1318,6 +1320,7 @@
         'browser/browsing_data_file_system_helper_unittest.cc',
         'browser/browsing_data_indexed_db_helper_unittest.cc',
         'browser/browsing_data_local_storage_helper_unittest.cc',
+        'browser/browsing_data_quota_helper_unittest.cc',
         'browser/browsing_data_remover_unittest.cc',
         'browser/chrome_browser_application_mac_unittest.mm',
         'browser/chromeos/cros/network_library.cc',
@@ -2070,6 +2073,8 @@
         '../webkit/fileapi/file_writer_delegate_unittest.cc',
         '../webkit/fileapi/file_system_test_helper.cc',
         '../webkit/fileapi/file_system_test_helper.h',
+        '../webkit/quota/mock_storage_client.cc',
+        '../webkit/quota/mock_storage_client.h',
       ],
       'conditions': [
         ['p2p_apis==1', {

tools/heapcheck/suppressions.txt

@@ -1698,214 +1698,6 @@
    ...
    fun:CrxUpdateService::ProcessPendingItems
 }
-{
-   bug_91816_a
-   Heapcheck:Leak
-   fun:__gnu_cxx::new_allocator::allocate
-   fun:std::_Vector_base::_M_allocate
-   fun:std::vector::_M_insert_aux
-   fun:std::vector::push_back
-   fun:MockBrowsingDataQuotaHelper::AddHost
-   fun:MockBrowsingDataQuotaHelper::AddQuotaSamples
-   fun:::CookiesTreeModelTest::CreateCookiesTreeModelWithInitialSample
-}
-{
-   bug_91816_b
-   Heapcheck:Leak
-   fun:__gnu_cxx::new_allocator::allocate
-   fun:std::_Vector_base::_M_allocate
-   fun:std::vector::_M_insert_aux
-   fun:std::vector::push_back
-   fun:MockBrowsingDataQuotaHelper::AddHost
-   fun:MockBrowsingDataQuotaHelper::AddQuotaSamples
-   fun:::CookiesTreeModelTest_RemoveSingleCookieNode_Test::TestBody
-}
-{
-   bug_91816_c
-   Heapcheck:Leak
-   fun:__gnu_cxx::new_allocator::allocate
-   fun:std::_Vector_base::_M_allocate
-   fun:std::vector::_M_insert_aux
-   fun:std::vector::push_back
-   fun:MockBrowsingDataQuotaHelper::AddHost
-   fun:MockBrowsingDataQuotaHelper::AddQuotaSamples
-   fun:::CookiesTreeModelTest_RemoveSingleCookieNodeOf3_Test::TestBody
-}
-{
-   bug_91816_d
-   Heapcheck:Leak
-   fun:::CookiesTreeModelTest::SetUp
-}
-{
-   bug_91816_e
-   Heapcheck:Leak
-   fun:BrowserThread::GetMessageLoopProxyForThread
-   fun:MockBrowsingDataQuotaHelper*
-   fun:::CookiesTreeModelTest::SetUp
-}
-{
-   bug_91816_f
-   Heapcheck:Leak
-   fun:::CookiesTreeModelTest_OriginOrdering_Test::TestBody
-}
-{
-   bug_91816_g
-   Heapcheck:Leak
-   fun:BrowserThread::GetMessageLoopProxyForThread
-   fun:MockBrowsingDataQuotaHelper*
-   fun:::CookiesTreeModelTest_OriginOrdering_Test::TestBody
-}
-{
-   bug_91816_h
-   Heapcheck:Leak
-   fun:::CookiesTreeModelTest_ContentSettings_Test::TestBody
-}
-{
-   bug_91816_i
-   Heapcheck:Leak
-   fun:BrowserThread::GetMessageLoopProxyForThread
-   fun:MockBrowsingDataQuotaHelper*
-   fun:::CookiesTreeModelTest_ContentSettings_Test::TestBody
-}
-{
-   bug_91836_a
-   Heapcheck:Leak
-   ...
-   fun:std::basic_string::basic_string
-   fun:GURL::ComponentString
-   fun:GURL::host
-   fun:BrowsingDataQuotaHelperImpl::GotOrigins
-   fun:void DispatchToMethod
-   fun:base::ScopedCallbackFactory::CallbackImpl::RunWithParams
-   fun:CallbackRunner::Run
-   fun:quota::QuotaManager::GetModifiedSinceTask::DatabaseTaskCompleted
-   fun:quota::QuotaManager::DatabaseTaskBase::Completed
-   fun:quota::QuotaTask::CallCompleted
-   ...
-   fun:BrowsingDataQuotaHelperTest_FetchData_Test::TestBody
-}
-{
-   bug_91836_b
-   Heapcheck:Leak
-   fun:BrowsingDataQuotaHelperTest::SetUp
-}
-{
-   bug_91836_c
-   Heapcheck:Leak
-   fun:quota::QuotaManager::QuotaManager
-   fun:BrowsingDataQuotaHelperTest::SetUp
-}
-{
-   bug_91836_d
-   Heapcheck:Leak
-   ...
-   fun:std::deque::deque
-   fun:quota::CallbackQueueBase::CallbackQueueBase
-   fun:quota::CallbackQueue3::CallbackQueue3
-   fun:quota::QuotaManager::QuotaManager
-   fun:BrowsingDataQuotaHelperTest::SetUp
-}
-{
-   bug_91836_e
-   Heapcheck:Leak
-   fun:__gnu_cxx::new_allocator::allocate
-   ...
-   fun:std::_Deque_base::_M_initialize_map
-   fun:std::_Deque_base::_Deque_base
-   fun:std::deque::deque
-   fun:quota::CallbackQueueBase::CallbackQueueBase
-   fun:quota::CallbackQueue3::CallbackQueue3
-   fun:quota::UsageTracker::UsageTracker
-   fun:quota::QuotaManager::LazyInitialize
-   fun:quota::QuotaManager::GetOriginsModifiedSince
-   fun:BrowsingDataQuotaHelperImpl::FetchQuotaInfo
-   fun:BrowsingDataQuotaHelperImpl::StartFetching
-   fun:BrowsingDataQuotaHelperTest::StartFetching
-   fun:BrowsingDataQuotaHelperTest_Empty_Test::TestBody
-}
-{
-   bug_91836_f
-   Heapcheck:Leak
-   fun:quota::QuotaManager::LazyInitialize
-   fun:quota::QuotaManager::GetOriginsModifiedSince
-   fun:BrowsingDataQuotaHelperImpl::FetchQuotaInfo
-   fun:BrowsingDataQuotaHelperImpl::StartFetching
-   fun:BrowsingDataQuotaHelperTest::StartFetching
-   fun:BrowsingDataQuotaHelperTest_Empty_Test::TestBody
-}
-{
-   bug_91836_g
-   Heapcheck:Leak
-   ...
-   fun:std::string::append
-   fun:FilePath::Append
-   fun:FilePath::AppendASCII
-   fun:quota::QuotaManager::LazyInitialize
-   fun:quota::QuotaManager::GetOriginsModifiedSince
-   fun:BrowsingDataQuotaHelperImpl::FetchQuotaInfo
-   fun:BrowsingDataQuotaHelperImpl::StartFetching
-   fun:BrowsingDataQuotaHelperTest::StartFetching
-   fun:BrowsingDataQuotaHelperTest_Empty_Test::TestBody
-}
-{
-   bug_91836_h
-   Heapcheck:Leak
-   ...
-   fun:std::map::insert
-   fun:std::map::operator[]
-   fun:BrowsingDataQuotaHelperImpl::GotHostUsage
-   fun:void DispatchToMethod
-   fun:base::ScopedCallbackFactory::CallbackImpl::RunWithParams
-   fun:CallbackRunner::Run
-   fun:quota::CallbackQueue3::Run
-   fun:quota::CallbackQueueMap3::Run
-   fun:quota::UsageTracker::DidGetClientHostUsage
-   fun:void DispatchToMethod
-   fun:base::ScopedCallbackFactory::CallbackImpl::RunWithParams
-   fun:CallbackRunner::Run
-   fun:quota::ClientUsageTracker::GetHostUsage
-   fun:quota::UsageTracker::GetHostUsage
-   fun:quota::QuotaManager::GetHostUsage
-}
-{
-   bug_91836_i
-   Heapcheck:Leak
-   fun:quota::QuotaManager::StartEviction
-   fun:quota::QuotaManager::InitializeTemporaryOriginsInfoTask::DatabaseTaskCompleted
-   fun:quota::QuotaManager::DatabaseTaskBase::Completed
-   fun:quota::QuotaTask::CallCompleted
-   ...
-   fun:BrowsingDataQuotaHelperTest_Empty_Test::TestBody
-}
-{
-   bug_91836_j
-   Heapcheck:Leak
-   fun:BrowserThread::GetMessageLoopProxyForThread
-   fun:BrowsingDataQuotaHelperTest::SetUp
-}
-{
-   bug_91836_k
-   Heapcheck:Leak
-   fun:quota::QuotaDatabase::LazyOpen
-   fun:quota::QuotaDatabase::SetGlobalQuota
-   fun:quota::QuotaManager::InitializeTask::RunOnTargetThread
-   ...
-   fun:BrowsingDataQuotaHelperTest_Empty_Test::TestBody
-}
-{
-   bug_91836_l
-   Heapcheck:Leak
-   fun:__gnu_cxx::new_allocator::allocate
-   fun:std::string::_Rep::_S_create
-   fun:std::string::_S_construct
-   fun:std::string::_S_construct_aux
-   fun:std::string::_S_construct
-   fun:std::basic_string::basic_string
-   fun:file_util::CreateTemporaryDirInDirImpl
-   fun:file_util::CreateNewTempDirectory
-   fun:ScopedTempDir::CreateUniqueTempDir
-   fun:BrowsingDataQuotaHelperTest::SetUp
-}
 {
    bug_91845_a
    Heapcheck:Leak

tools/valgrind/memcheck/suppressions.txt

@@ -5189,210 +5189,6 @@
    fun:_ZN12sync_ui_util25ConstructAboutInformationEP18ProfileSyncServicePN4base15DictionaryValueE
    fun:_ZN71SyncUIUtilTest_ConstructAboutInformationWithUnrecoverableErrorTest_Test8TestBodyEv
 }
-{
-   bug_91816_a
-   Memcheck:Leak
-   fun:_Znw*
-   fun:_ZN12_GLOBAL__N_120CookiesTreeModelTest5SetUpEv
-}
-{
-   bug_91816_b
-   Memcheck:Leak
-   fun:_Znw*
-   fun:_ZN12_GLOBAL__N_140CookiesTreeModelTest_OriginOrdering_Test8TestBodyEv
-}
-{
-   bug_91816_c
-   Memcheck:Leak
-   fun:_Znw*
-   fun:_ZN12_GLOBAL__N_141CookiesTreeModelTest_ContentSettings_Test8TestBodyEv
-}
-{
-   bug_91836_a
-   Memcheck:Leak
-   fun:_Znw*
-   ...
-   fun:_ZN5quota18ClientUsageTracker*DidGetHostUsage*
-   fun:_ZN5quota18ClientUsageTracker*GatherHostUsageTask9CompletedEv
-   fun:_ZN5quota9QuotaTask13CallCompletedEv
-   fun:_ZN5quota18ClientUsageTracker19GatherUsageTaskBase11DidGetUsageEx
-}
-{
-   bug_91836_b
-   Memcheck:Leak
-   fun:_Znw*
-   ...
-   fun:_ZN5quota18ClientUsageTracker*DidGetGlobalUsage*
-   fun:_ZN5quota18ClientUsageTracker*GatherGlobalUsage*
-   fun:_ZN5quota9QuotaTask13CallCompletedEv
-   fun:_ZN5quota18ClientUsageTracker19GatherUsageTaskBase11DidGetUsageEx
-}
-{
-   bug_91836_c
-   Memcheck:Leak
-   fun:_Znw*
-   fun:_ZN5quota12QuotaManager13StartEvictionEv
-   fun:_ZN5quota12QuotaManager34InitializeTemporaryOriginsInfoTask21DatabaseTaskCompletedEv
-   fun:_ZN5quota12QuotaManager16DatabaseTaskBase9CompletedEv
-   fun:_ZN5quota9QuotaTask13CallCompletedEv
-   ...
-   fun:_ZN38BrowsingDataQuotaHelperTest_Empty_Test8TestBodyEv
-}
-{
-   bug_91836_c2
-   Memcheck:Leak
-   fun:_Znw*
-   fun:_ZN5quota12QuotaManager13StartEvictionEv
-   fun:_ZN5quota12QuotaManager34InitializeTemporaryOriginsInfoTask21DatabaseTaskCompletedEv
-   fun:_ZN5quota12QuotaManager16DatabaseTaskBase9CompletedEv
-   fun:_ZN5quota9QuotaTask13CallCompletedEv
-   ...
-   fun:_ZN42BrowsingDataQuotaHelperTest_FetchData_Test8TestBodyEv
-}
-{
-   bug_91836_d
-   Memcheck:Leak
-   fun:_Znw*
-   fun:_ZN9__gnu_cxx13new_allocatorISt13_Rb_tree_nodeISt4pairIKSsN23BrowsingDataQuotaHelper9QuotaInfoEEEE8allocateEjPKv
-   fun:_ZNSt8_Rb_treeISsSt4pairIKSsN23BrowsingDataQuotaHelper9QuotaInfoEESt10_Select1stIS4_ESt4lessISsESaIS4_EE11_M_get_nodeEv
-   fun:_ZNSt8_Rb_treeISsSt4pairIKSsN23BrowsingDataQuotaHelper9QuotaInfoEESt10_Select1stIS4_ESt4lessISsESaIS4_EE14_M_create_nodeERKS4_
-   fun:_ZNSt8_Rb_treeISsSt4pairIKSsN23BrowsingDataQuotaHelper9QuotaInfoEESt10_Select1stIS4_ESt4lessISsESaIS4_EE10_M_insert_EPKSt18_Rb_tree_node_baseSD_RKS4_
-   ...
-   fun:_ZN5quota12QuotaManager12GetHostUsageERKSsNS_11StorageTypeEP14CallbackRunnerI6Tuple3IS2_S3_xEE
-}
-{
-   bug_91836_e
-   Memcheck:Leak
-   fun:_Znw*
-   ...
-   fun:_ZN3sql10Connection18GetUniqueStatementEPKc
-   fun:_ZN3sql10Connection18GetCachedStatementERKNS_11StatementIDEPKc
-   fun:_ZN3sql9MetaTable19PrepareSetStatementEPNS_9StatementEPKc
-   fun:_ZN3sql9MetaTable8SetValueEPKci
-   fun:_ZN3sql9MetaTable16SetVersionNumberEi
-   fun:_ZN3sql9MetaTable4InitEPNS_10ConnectionEii
-   fun:_ZN5quota13QuotaDatabase12CreateSchemaEPN3sql10ConnectionEPNS1_9MetaTableEiiPKNS0_11TableSchemaEjPKNS0_11IndexSchemaEj
-   fun:_ZN5quota13QuotaDatabase21EnsureDatabaseVersionEv
-}
-{
-   bug_91836_f
-   Memcheck:Leak
-   fun:_Znw*
-   fun:_ZN9__gnu_cxx13new_allocatorISt13_Rb_tree_nodeIPN3sql10Connection12StatementRefEEE8allocateEjPKv
-   fun:_ZNSt8_Rb_treeIPN3sql10Connection12StatementRefES3_St9_IdentityIS3_ESt4lessIS3_ESaIS3_EE11_M_get_nodeEv
-   fun:_ZNSt8_Rb_treeIPN3sql10Connection12StatementRefES3_St9_IdentityIS3_ESt4lessIS3_ESaIS3_EE14_M_create_nodeERKS3_
-   fun:_ZNSt8_Rb_treeIPN3sql10Connection12StatementRefES3_St9_IdentityIS3_ESt4lessIS3_ESaIS3_EE10_M_insert_EPKSt18_Rb_tree_node_baseSC_RKS3_
-   fun:_ZNSt8_Rb_treeIPN3sql10Connection12StatementRefES3_St9_IdentityIS3_ESt4lessIS3_ESaIS3_EE16_M_insert_uniqueERKS3_
-   fun:_ZNSt3setIPN3sql10Connection12StatementRefESt4lessIS3_ESaIS3_EE6insertERKS3_
-   fun:_ZN3sql10Connection19StatementRefCreatedEPNS0_12StatementRefE
-   fun:_ZN3sql10Connection12StatementRefC1EPS0_P12sqlite3_stmt
-   fun:_ZN3sql10Connection18GetUniqueStatementEPKc
-   fun:_ZN3sql10Connection18GetCachedStatementERKNS_11StatementIDEPKc
-   fun:_ZN3sql10Connection17CommitTransactionEv
-   fun:_ZN3sql11Transaction6CommitEv
-   fun:_ZN5quota13QuotaDatabase12CreateSchemaEPN3sql10ConnectionEPNS1_9MetaTableEiiPKNS0_11TableSchemaEjPKNS0_11IndexSchemaEj
-   fun:_ZN5quota13QuotaDatabase21EnsureDatabaseVersionEv
-}
-{
-   bug_91836_g
-   Memcheck:Leak
-   fun:_Znw*
-   fun:_ZN9__gnu_cxx13new_allocatorISt13_Rb_tree_nodeISt4pairIKSsN23BrowsingDataQuotaHelper9QuotaInfoEEEE8allocateEmPKv
-   fun:_ZNSt8_Rb_treeISsSt4pairIKSsN23BrowsingDataQuotaHelper9QuotaInfoEESt10_Select1stIS4_ESt4lessISsESaIS4_EE11_M_get_nodeEv
-   fun:_ZNSt8_Rb_treeISsSt4pairIKSsN23BrowsingDataQuotaHelper9QuotaInfoEESt10_Select1stIS4_ESt4lessISsESaIS4_EE14_M_create_nodeERKS4_
-   fun:_ZNSt8_Rb_treeISsSt4pairIKSsN23BrowsingDataQuotaHelper9QuotaInfoEESt10_Select1stIS4_ESt4lessISsESaIS4_EE9_M_insertEPSt18_Rb_tree_node_baseSC_RKS4_
-   ...
-   fun:_ZNSt8_Rb_treeISsSt4pairIKSsN23BrowsingDataQuotaHelper9QuotaInfoEESt10_Select1stIS4_ESt4lessISsESaIS4_EE13insert_uniqueESt17_Rb_tree_iteratorIS4_ERKS4_
-   fun:_ZNSt3mapISsN23BrowsingDataQuotaHelper9QuotaInfoESt4lessISsESaISt4pairIKSsS1_EEE6insertESt17_Rb_tree_iteratorIS6_ERKS6_
-   fun:_ZNSt3mapISsN23BrowsingDataQuotaHelper9QuotaInfoESt4lessISsESaISt4pairIKSsS1_EEEixERS5_
-   fun:_ZN27BrowsingDataQuotaHelperImpl12GotHostUsageERKSsN5quota11StorageTypeEx
-   fun:_Z16DispatchToMethodI27BrowsingDataQuotaHelperImplMS0_FvRKSsN5quota11StorageTypeExES2_S4_xEvPT_T0_RK6Tuple3IT1_T2_T3_E
-   fun:_ZN4base21ScopedCallbackFactoryI27BrowsingDataQuotaHelperImplE12CallbackImplIMS1_FvRKSsN5quota11StorageTypeExE6Tuple3IS5_S7_xEE13RunWithParamsERKSB_
-   fun:_ZN14CallbackRunnerI6Tuple3IRKSsN5quota11StorageTypeExEE3RunISsS4_xEEvRKT_RKT0_RKT1_
-   fun:_ZN5quota14CallbackQueue3IP14CallbackRunnerI6Tuple3IRKSsNS_11StorageTypeExEES4_S5_xE3RunES4_S5_x
-   fun:_ZN5quota17CallbackQueueMap3IP14CallbackRunnerI6Tuple3IRKSsNS_11StorageTypeExEESsS4_S5_xE3RunES4_S4_S5_x
-   fun:_ZN5quota12UsageTracker21DidGetClientHostUsageERKSsNS_11StorageTypeEx
-   fun:_Z16DispatchToMethodIN5quota12UsageTrackerEMS1_FvRKSsNS0_11StorageTypeExES3_S4_xEvPT_T0_RK6Tuple3IT1_T2_T3_E
-   fun:_ZN4base21ScopedCallbackFactoryIN5quota12UsageTrackerEE12CallbackImplIMS2_FvRKSsNS1_11StorageTypeExE6Tuple3IS6_S7_xEE13RunWithParamsERKSB_
-   fun:_ZN14CallbackRunnerI6Tuple3IRKSsN5quota11StorageTypeExEE3RunISsS4_xEEvRKT_RKT0_RKT1_
-   fun:_ZN5quota18ClientUsageTracker12GetHostUsageERKSsP14CallbackRunnerI6Tuple3IS2_NS_11StorageTypeExEE
-   fun:_ZN5quota12UsageTracker12GetHostUsageERKSsP14CallbackRunnerI6Tuple3IS2_NS_11StorageTypeExEE
-   fun:_ZN5quota12QuotaManager12GetHostUsageERKSsNS_11StorageTypeEP14CallbackRunnerI6Tuple3IS2_S3_xEE
-}
-{
-   bug_91836_h
-   Memcheck:Leak
-   fun:_Znw*
-   fun:_ZN3sql10Connection18GetUniqueStatementEPKc
-   fun:_ZN3sql10Connection18GetCachedStatementERKNS_11StatementIDEPKc
-   fun:_ZN5quota12_GLOBAL__N_122PrepareCachedStatementEPN3sql10ConnectionERKNS1_11StatementIDEPKcPNS1_9StatementE
-   fun:_ZN5quota13QuotaDatabase23GetOriginsModifiedSinceENS_11StorageTypeEPSt3setI4GURLSt4lessIS3_ESaIS3_EEN4base4TimeE
-   fun:_ZN5quota12QuotaManager20GetModifiedSinceTask17RunOnTargetThreadEv
-   ...
-   fun:_ZN38BrowsingDataQuotaHelperTest_Empty_Test8TestBodyEv
-}
-{
-   bug_91836_h2
-   Memcheck:Leak
-   fun:_Znw*
-   fun:_ZN3sql10Connection18GetUniqueStatementEPKc
-   fun:_ZN3sql10Connection18GetCachedStatementERKNS_11StatementIDEPKc
-   fun:_ZN3sql9MetaTable19PrepareGetStatementEPNS_9StatementEPKc
-   fun:_ZN3sql9MetaTable8GetValueEPKcPi
-   fun:_ZN5quota13QuotaDatabase28IsOriginDatabaseBootstrappedEv
-   fun:_ZN5quota12QuotaManager14InitializeTask17RunOnTargetThreadEv
-   ...
-   fun:_ZN38BrowsingDataQuotaHelperTest_Empty_Test8TestBodyEv
-}
-{
-   bug_91836_h3
-   Memcheck:Leak
-   fun:_Znw*
-   fun:_ZN3sql10Connection18GetUniqueStatementEPKc
-   fun:_ZN3sql10Connection18GetCachedStatementERKNS_11StatementIDEPKc
-   fun:_ZN5quota12_GLOBAL__N_122PrepareCachedStatementEPN3sql10ConnectionERKNS1_11StatementIDEPKcPNS1_9StatementE
-   fun:_ZN5quota13QuotaDatabase19FindOriginUsedCountERK4GURLNS_11StorageTypeEPi
-   fun:_ZN5quota13QuotaDatabase25SetOriginLastModifiedTimeERK4GURLNS_11StorageTypeEN4base4TimeE
-   fun:_ZN5quota12QuotaManager22UpdateModifiedTimeTask17RunOnTargetThreadEv
-   ...
-   fun:_ZN42BrowsingDataQuotaHelperTest_FetchData_Test8TestBodyEv
-}
-{
-   bug_91836_i
-   Memcheck:Leak
-   fun:_Znw*
-   ...
-   fun:_ZN27BrowsingDataQuotaHelperTest5SetUpEv
-}
-{
-   bug_91836_j
-   Memcheck:Leak
-   fun:malloc
-   fun:sqlite3MemMalloc
-   fun:mallocWithAlarm
-   fun:sqlite3Malloc
-   fun:sqlite3MallocZero
-   fun:sqlite3BtreeOpen
-   fun:openDatabase
-   fun:sqlite3_open
-   fun:_ZN3sql10Connection12OpenInternalERKSs
-   fun:_ZN3sql10Connection4OpenERK8FilePath
-   fun:_ZN5quota13QuotaDatabase8LazyOpenEb
-   fun:_ZN5quota13QuotaDatabase14SetGlobalQuotaENS_11StorageTypeEx
-   fun:_ZN5quota12QuotaManager14InitializeTask17RunOnTargetThreadEv
-}
-{
-   bug_91836_k
-   Memcheck:Leak
-   fun:_Znw*
-   ...
-   fun:_ZN5quota12UsageTrackerC1ERKSt4listIPNS_11QuotaClientESaIS3_EENS_11StorageTypeEPNS_20SpecialStoragePolicyE
-   fun:_ZN5quota12QuotaManager14LazyInitializeEv
-   fun:_ZN5quota12QuotaManager29NotifyStorageModifiedInternalENS_11QuotaClient2IDERK4GURLNS_11StorageTypeExN4base4TimeE
-   fun:_ZN5quota17MockStorageClient24TouchAllOriginsAndNotifyEv
-   fun:_ZN27BrowsingDataQuotaHelperTest14RegisterClientEPKN5quota14MockOriginData*
-   fun:_ZN42BrowsingDataQuotaHelperTest_FetchData_Test8TestBodyEv
-}
 {
    bug_91845_a
    Memcheck:Leak

webkit/quota/mock_storage_client.cc

@@ -84,6 +84,15 @@ void MockStorageClient::ModifyOriginAndNotify(
       id(), origin_url, type, delta, IncrementMockTime());
 }
 
+void MockStorageClient::TouchAllOriginsAndNotify() {
+  for (OriginDataMap::const_iterator itr = origin_data_.begin();
+       itr != origin_data_.end();
+       ++itr) {
+    quota_manager_proxy_->quota_manager()->NotifyStorageModifiedInternal(
+        id(), itr->first.first, itr->first.second, 0, IncrementMockTime());
+  }
+}
+
 void MockStorageClient::AddOriginToErrorSet(
     const GURL& origin_url, StorageType type) {
   error_origins_.insert(make_pair(origin_url, type));

webkit/quota/mock_storage_client.h

@@ -37,6 +37,7 @@ class MockStorageClient : public QuotaClient {
       const GURL& origin_url, StorageType type, int64 size);
   void ModifyOriginAndNotify(
       const GURL& origin_url, StorageType type, int64 delta);
+  void TouchAllOriginsAndNotify();
 
   void AddOriginToErrorSet(const GURL& origin_url, StorageType type);