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

Side by Side Diff: webkit/browser/database/database_quota_client_unittest.cc

Issue 15925005: [Quota][Clean up] Drop non-informative StorageType parameter on GetOriginsForType callback (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 7 years, 6 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 <map> 5 #include <map>
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/files/file_path.h" 8 #include "base/files/file_path.h"
9 #include "base/message_loop.h" 9 #include "base/message_loop.h"
10 #include "base/message_loop_proxy.h" 10 #include "base/message_loop_proxy.h"
(...skipping 174 matching lines...) Expand 10 before | Expand all | Expand 10 after
185 } 185 }
186 186
187 MockDatabaseTracker* mock_tracker() { return mock_tracker_.get(); } 187 MockDatabaseTracker* mock_tracker() { return mock_tracker_.get(); }
188 188
189 189
190 private: 190 private:
191 void OnGetOriginUsageComplete(int64 usage) { 191 void OnGetOriginUsageComplete(int64 usage) {
192 usage_ = usage; 192 usage_ = usage;
193 } 193 }
194 194
195 void OnGetOriginsComplete(const std::set<GURL>& origins, 195 void OnGetOriginsComplete(const std::set<GURL>& origins) {
196 quota::StorageType type) {
197 origins_ = origins; 196 origins_ = origins;
198 type_ = type;
199 } 197 }
200 198
201 void OnDeleteOriginDataComplete(quota::QuotaStatusCode status) { 199 void OnDeleteOriginDataComplete(quota::QuotaStatusCode status) {
202 delete_status_ = status; 200 delete_status_ = status;
203 } 201 }
204 202
205 base::MessageLoop message_loop_; 203 base::MessageLoop message_loop_;
206 int64 usage_; 204 int64 usage_;
207 std::set<GURL> origins_; 205 std::set<GURL> origins_;
208 quota::StorageType type_;
209 quota::QuotaStatusCode delete_status_; 206 quota::QuotaStatusCode delete_status_;
210 scoped_refptr<MockDatabaseTracker> mock_tracker_; 207 scoped_refptr<MockDatabaseTracker> mock_tracker_;
211 base::WeakPtrFactory<DatabaseQuotaClientTest> weak_factory_; 208 base::WeakPtrFactory<DatabaseQuotaClientTest> weak_factory_;
212 }; 209 };
213 210
214 211
215 TEST_F(DatabaseQuotaClientTest, GetOriginUsage) { 212 TEST_F(DatabaseQuotaClientTest, GetOriginUsage) {
216 DatabaseQuotaClient client( 213 DatabaseQuotaClient client(
217 base::MessageLoopProxy::current(), 214 base::MessageLoopProxy::current(),
218 mock_tracker()); 215 mock_tracker());
(...skipping 64 matching lines...) Expand 10 before | Expand all | Expand 10 after
283 mock_tracker()->set_async_delete(false); 280 mock_tracker()->set_async_delete(false);
284 EXPECT_TRUE(DeleteOriginData(&client, kTemp, kOriginA)); 281 EXPECT_TRUE(DeleteOriginData(&client, kTemp, kOriginA));
285 EXPECT_EQ(1, mock_tracker()->delete_called_count()); 282 EXPECT_EQ(1, mock_tracker()->delete_called_count());
286 283
287 mock_tracker()->set_async_delete(true); 284 mock_tracker()->set_async_delete(true);
288 EXPECT_TRUE(DeleteOriginData(&client, kTemp, kOriginA)); 285 EXPECT_TRUE(DeleteOriginData(&client, kTemp, kOriginA));
289 EXPECT_EQ(2, mock_tracker()->delete_called_count()); 286 EXPECT_EQ(2, mock_tracker()->delete_called_count());
290 } 287 }
291 288
292 } // namespace webkit_database 289 } // namespace webkit_database
OLDNEW
« no previous file with comments | « webkit/browser/database/database_quota_client.cc ('k') | webkit/browser/fileapi/file_system_quota_client.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698