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

Side by Side Diff: chrome/browser/browsing_data/browsing_data_local_storage_helper.cc

Issue 459233002: Browsing Data Deletion: Style fixes (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Re-apply comment tweaks Created 6 years, 4 months 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/browsing_data/browsing_data_local_storage_helper.h" 5 #include "chrome/browser/browsing_data/browsing_data_local_storage_helper.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "chrome/browser/browsing_data/browsing_data_helper.h" 8 #include "chrome/browser/browsing_data/browsing_data_helper.h"
9 #include "chrome/browser/profiles/profile.h" 9 #include "chrome/browser/profiles/profile.h"
10 #include "content/public/browser/browser_thread.h" 10 #include "content/public/browser/browser_thread.h"
(...skipping 17 matching lines...) Expand all
28 GetDOMStorageContext()), 28 GetDOMStorageContext()),
29 is_fetching_(false) { 29 is_fetching_(false) {
30 DCHECK(dom_storage_context_); 30 DCHECK(dom_storage_context_);
31 } 31 }
32 32
33 BrowsingDataLocalStorageHelper::~BrowsingDataLocalStorageHelper() { 33 BrowsingDataLocalStorageHelper::~BrowsingDataLocalStorageHelper() {
34 } 34 }
35 35
36 void BrowsingDataLocalStorageHelper::StartFetching( 36 void BrowsingDataLocalStorageHelper::StartFetching(
37 const base::Callback<void(const std::list<LocalStorageInfo>&)>& callback) { 37 const base::Callback<void(const std::list<LocalStorageInfo>&)>& callback) {
38 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); 38 DCHECK_CURRENTLY_ON(BrowserThread::UI);
39 DCHECK(!is_fetching_); 39 DCHECK(!is_fetching_);
40 DCHECK_EQ(false, callback.is_null()); 40 DCHECK(!callback.is_null());
41 41
42 is_fetching_ = true; 42 is_fetching_ = true;
43 completion_callback_ = callback; 43 completion_callback_ = callback;
44 dom_storage_context_->GetLocalStorageUsage( 44 dom_storage_context_->GetLocalStorageUsage(
45 base::Bind( 45 base::Bind(
46 &BrowsingDataLocalStorageHelper::GetUsageInfoCallback, this)); 46 &BrowsingDataLocalStorageHelper::GetUsageInfoCallback, this));
47 } 47 }
48 48
49 void BrowsingDataLocalStorageHelper::DeleteOrigin(const GURL& origin) { 49 void BrowsingDataLocalStorageHelper::DeleteOrigin(const GURL& origin) {
50 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); 50 DCHECK_CURRENTLY_ON(BrowserThread::UI);
51 dom_storage_context_->DeleteLocalStorage(origin); 51 dom_storage_context_->DeleteLocalStorage(origin);
52 } 52 }
53 53
54 void BrowsingDataLocalStorageHelper::GetUsageInfoCallback( 54 void BrowsingDataLocalStorageHelper::GetUsageInfoCallback(
55 const std::vector<content::LocalStorageUsageInfo>& infos) { 55 const std::vector<content::LocalStorageUsageInfo>& infos) {
56 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); 56 DCHECK_CURRENTLY_ON(BrowserThread::UI);
57 57
58 for (size_t i = 0; i < infos.size(); ++i) { 58 for (size_t i = 0; i < infos.size(); ++i) {
59 // Non-websafe state is not considered browsing data. 59 // Non-websafe state is not considered browsing data.
60 const content::LocalStorageUsageInfo& info = infos[i]; 60 const content::LocalStorageUsageInfo& info = infos[i];
61 if (BrowsingDataHelper::HasWebScheme(info.origin)) { 61 if (BrowsingDataHelper::HasWebScheme(info.origin)) {
62 local_storage_info_.push_back( 62 local_storage_info_.push_back(
63 LocalStorageInfo(info.origin, info.data_size, info.last_modified)); 63 LocalStorageInfo(info.origin, info.data_size, info.last_modified));
64 } 64 }
65 } 65 }
66 66
(...skipping 13 matching lines...) Expand all
80 //--------------------------------------------------------- 80 //---------------------------------------------------------
81 81
82 CannedBrowsingDataLocalStorageHelper::CannedBrowsingDataLocalStorageHelper( 82 CannedBrowsingDataLocalStorageHelper::CannedBrowsingDataLocalStorageHelper(
83 Profile* profile) 83 Profile* profile)
84 : BrowsingDataLocalStorageHelper(profile), 84 : BrowsingDataLocalStorageHelper(profile),
85 profile_(profile) { 85 profile_(profile) {
86 } 86 }
87 87
88 CannedBrowsingDataLocalStorageHelper* 88 CannedBrowsingDataLocalStorageHelper*
89 CannedBrowsingDataLocalStorageHelper::Clone() { 89 CannedBrowsingDataLocalStorageHelper::Clone() {
90 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); 90 DCHECK_CURRENTLY_ON(BrowserThread::UI);
91 CannedBrowsingDataLocalStorageHelper* clone = 91 CannedBrowsingDataLocalStorageHelper* clone =
92 new CannedBrowsingDataLocalStorageHelper(profile_); 92 new CannedBrowsingDataLocalStorageHelper(profile_);
93 93
94 clone->pending_local_storage_info_ = pending_local_storage_info_; 94 clone->pending_local_storage_info_ = pending_local_storage_info_;
95 return clone; 95 return clone;
96 } 96 }
97 97
98 void CannedBrowsingDataLocalStorageHelper::AddLocalStorage( 98 void CannedBrowsingDataLocalStorageHelper::AddLocalStorage(
99 const GURL& origin) { 99 const GURL& origin) {
100 if (BrowsingDataHelper::HasWebScheme(origin)) 100 if (BrowsingDataHelper::HasWebScheme(origin))
(...skipping 12 matching lines...) Expand all
113 return pending_local_storage_info_.size(); 113 return pending_local_storage_info_.size();
114 } 114 }
115 115
116 const std::set<GURL>& 116 const std::set<GURL>&
117 CannedBrowsingDataLocalStorageHelper::GetLocalStorageInfo() const { 117 CannedBrowsingDataLocalStorageHelper::GetLocalStorageInfo() const {
118 return pending_local_storage_info_; 118 return pending_local_storage_info_;
119 } 119 }
120 120
121 void CannedBrowsingDataLocalStorageHelper::StartFetching( 121 void CannedBrowsingDataLocalStorageHelper::StartFetching(
122 const base::Callback<void(const std::list<LocalStorageInfo>&)>& callback) { 122 const base::Callback<void(const std::list<LocalStorageInfo>&)>& callback) {
123 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); 123 DCHECK_CURRENTLY_ON(BrowserThread::UI);
124 DCHECK(!callback.is_null()); 124 DCHECK(!callback.is_null());
125 125
126 std::list<LocalStorageInfo> result; 126 std::list<LocalStorageInfo> result;
127 for (std::set<GURL>::iterator iter = pending_local_storage_info_.begin(); 127 for (std::set<GURL>::iterator iter = pending_local_storage_info_.begin();
128 iter != pending_local_storage_info_.end(); ++iter) { 128 iter != pending_local_storage_info_.end(); ++iter) {
129 result.push_back( 129 result.push_back(
130 LocalStorageInfo(*iter, 0, base::Time())); 130 LocalStorageInfo(*iter, 0, base::Time()));
131 } 131 }
132 132
133 BrowserThread::PostTask( 133 BrowserThread::PostTask(
134 BrowserThread::UI, FROM_HERE, base::Bind(callback, result)); 134 BrowserThread::UI, FROM_HERE, base::Bind(callback, result));
135 } 135 }
136 136
137 void CannedBrowsingDataLocalStorageHelper::DeleteOrigin(const GURL& origin) { 137 void CannedBrowsingDataLocalStorageHelper::DeleteOrigin(const GURL& origin) {
138 pending_local_storage_info_.erase(origin); 138 pending_local_storage_info_.erase(origin);
139 BrowsingDataLocalStorageHelper::DeleteOrigin(origin); 139 BrowsingDataLocalStorageHelper::DeleteOrigin(origin);
140 } 140 }
141 141
142 CannedBrowsingDataLocalStorageHelper::~CannedBrowsingDataLocalStorageHelper() {} 142 CannedBrowsingDataLocalStorageHelper::~CannedBrowsingDataLocalStorageHelper() {}
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698