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

Side by Side Diff: chrome/browser/extensions/api/sync_file_system/sync_file_system_browsertest.cc

Issue 1782053004: Change how the quota system computes the total poolsize for temporary storage (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 4 years 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
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 <stdint.h> 5 #include <stdint.h>
6 #include <utility> 6 #include <utility>
7 7
8 #include "base/json/json_reader.h" 8 #include "base/json/json_reader.h"
9 #include "base/macros.h" 9 #include "base/macros.h"
10 #include "base/run_loop.h" 10 #include "base/run_loop.h"
11 #include "base/threading/thread_task_runner_handle.h" 11 #include "base/threading/thread_task_runner_handle.h"
12 #include "base/values.h" 12 #include "base/values.h"
13 #include "chrome/browser/apps/app_browsertest_util.h" 13 #include "chrome/browser/apps/app_browsertest_util.h"
14 #include "chrome/browser/extensions/extension_service.h" 14 #include "chrome/browser/extensions/extension_service.h"
15 #include "chrome/browser/signin/fake_signin_manager_builder.h" 15 #include "chrome/browser/signin/fake_signin_manager_builder.h"
16 #include "chrome/browser/sync_file_system/drive_backend/sync_engine.h" 16 #include "chrome/browser/sync_file_system/drive_backend/sync_engine.h"
17 #include "chrome/browser/sync_file_system/local/local_file_sync_service.h" 17 #include "chrome/browser/sync_file_system/local/local_file_sync_service.h"
18 #include "chrome/browser/sync_file_system/sync_file_system_service.h" 18 #include "chrome/browser/sync_file_system/sync_file_system_service.h"
19 #include "chrome/browser/sync_file_system/sync_file_system_service_factory.h" 19 #include "chrome/browser/sync_file_system/sync_file_system_service_factory.h"
20 #include "components/drive/service/fake_drive_service.h" 20 #include "components/drive/service/fake_drive_service.h"
21 #include "content/public/browser/storage_partition.h"
21 #include "extensions/test/extension_test_message_listener.h" 22 #include "extensions/test/extension_test_message_listener.h"
22 #include "extensions/test/result_catcher.h" 23 #include "extensions/test/result_catcher.h"
23 #include "storage/browser/quota/quota_manager.h" 24 #include "storage/browser/quota/quota_manager.h"
24 #include "third_party/leveldatabase/src/helpers/memenv/memenv.h" 25 #include "third_party/leveldatabase/src/helpers/memenv/memenv.h"
25 #include "third_party/leveldatabase/src/include/leveldb/env.h" 26 #include "third_party/leveldatabase/src/include/leveldb/env.h"
26 27
27 namespace sync_file_system { 28 namespace sync_file_system {
28 29
29 namespace { 30 namespace {
30 31
(...skipping 23 matching lines...) Expand all
54 55
55 } // namespace 56 } // namespace
56 57
57 class SyncFileSystemTest : public extensions::PlatformAppBrowserTest, 58 class SyncFileSystemTest : public extensions::PlatformAppBrowserTest,
58 public drive::FakeDriveService::ChangeObserver { 59 public drive::FakeDriveService::ChangeObserver {
59 public: 60 public:
60 SyncFileSystemTest() 61 SyncFileSystemTest()
61 : remote_service_(NULL) { 62 : remote_service_(NULL) {
62 } 63 }
63 64
64 void SetUpInProcessBrowserTestFixture() override {
65 ExtensionApiTest::SetUpInProcessBrowserTestFixture();
66 real_minimum_preserved_space_ =
67 storage::QuotaManager::kMinimumPreserveForSystem;
68 storage::QuotaManager::kMinimumPreserveForSystem = 0;
69 }
70
71 void TearDownInProcessBrowserTestFixture() override {
72 storage::QuotaManager::kMinimumPreserveForSystem =
73 real_minimum_preserved_space_;
74 ExtensionApiTest::TearDownInProcessBrowserTestFixture();
75 }
76
77 scoped_refptr<base::SequencedTaskRunner> MakeSequencedTaskRunner() { 65 scoped_refptr<base::SequencedTaskRunner> MakeSequencedTaskRunner() {
78 scoped_refptr<base::SequencedWorkerPool> worker_pool = 66 scoped_refptr<base::SequencedWorkerPool> worker_pool =
79 content::BrowserThread::GetBlockingPool(); 67 content::BrowserThread::GetBlockingPool();
80 68
81 return worker_pool->GetSequencedTaskRunnerWithShutdownBehavior( 69 return worker_pool->GetSequencedTaskRunnerWithShutdownBehavior(
82 worker_pool->GetSequenceToken(), 70 worker_pool->GetSequenceToken(),
83 base::SequencedWorkerPool::SKIP_ON_SHUTDOWN); 71 base::SequencedWorkerPool::SKIP_ON_SHUTDOWN);
84 } 72 }
85 73
86 void SetUpOnMainThread() override { 74 void SetUpOnMainThread() override {
(...skipping 19 matching lines...) Expand all
106 NULL, // task_logger 94 NULL, // task_logger
107 NULL, // notification_manager 95 NULL, // notification_manager
108 extension_service, 96 extension_service,
109 fake_signin_manager_.get(), // signin_manager 97 fake_signin_manager_.get(), // signin_manager
110 NULL, // token_service 98 NULL, // token_service
111 NULL, // request_context 99 NULL, // request_context
112 std::move(drive_service_factory), in_memory_env_.get()); 100 std::move(drive_service_factory), in_memory_env_.get());
113 remote_service_->SetSyncEnabled(true); 101 remote_service_->SetSyncEnabled(true);
114 factory->set_mock_remote_file_service( 102 factory->set_mock_remote_file_service(
115 std::unique_ptr<RemoteFileSyncService>(remote_service_)); 103 std::unique_ptr<RemoteFileSyncService>(remote_service_));
104
105 const int kMeg = 1024 * 1024;
106 storage::QuotaSettings settings;
107 settings.pool_size = 25 * kMeg;
108 settings.per_host_quota = 5 * kMeg;
109 settings.must_remain_available = 0;
110 settings.refresh_interval = base::TimeDelta::Max();
111 content::BrowserContext::GetDefaultStoragePartition(context)
112 ->GetQuotaManager()
113 ->SetQuotaSettings(settings);
116 } 114 }
117 115
118 // drive::FakeDriveService::ChangeObserver override. 116 // drive::FakeDriveService::ChangeObserver override.
119 void OnNewChangeAvailable() override { 117 void OnNewChangeAvailable() override {
120 sync_engine()->OnNotificationReceived(); 118 sync_engine()->OnNotificationReceived();
121 } 119 }
122 120
123 SyncFileSystemService* sync_file_system_service() { 121 SyncFileSystemService* sync_file_system_service() {
124 return SyncFileSystemServiceFactory::GetForProfile(browser()->profile()); 122 return SyncFileSystemServiceFactory::GetForProfile(browser()->profile());
125 } 123 }
(...skipping 23 matching lines...) Expand all
149 } 147 }
150 148
151 private: 149 private:
152 base::ScopedTempDir base_dir_; 150 base::ScopedTempDir base_dir_;
153 std::unique_ptr<leveldb::Env> in_memory_env_; 151 std::unique_ptr<leveldb::Env> in_memory_env_;
154 152
155 std::unique_ptr<FakeSigninManagerForTesting> fake_signin_manager_; 153 std::unique_ptr<FakeSigninManagerForTesting> fake_signin_manager_;
156 154
157 drive_backend::SyncEngine* remote_service_; 155 drive_backend::SyncEngine* remote_service_;
158 156
159 int64_t real_minimum_preserved_space_;
160
161 DISALLOW_COPY_AND_ASSIGN(SyncFileSystemTest); 157 DISALLOW_COPY_AND_ASSIGN(SyncFileSystemTest);
162 }; 158 };
163 159
164 IN_PROC_BROWSER_TEST_F(SyncFileSystemTest, AuthorizationTest) { 160 IN_PROC_BROWSER_TEST_F(SyncFileSystemTest, AuthorizationTest) {
165 ExtensionTestMessageListener open_failure( 161 ExtensionTestMessageListener open_failure(
166 "checkpoint: Failed to get syncfs", true); 162 "checkpoint: Failed to get syncfs", true);
167 ExtensionTestMessageListener bar_created( 163 ExtensionTestMessageListener bar_created(
168 "checkpoint: \"/bar\" created", true); 164 "checkpoint: \"/bar\" created", true);
169 ExtensionTestMessageListener foo_created( 165 ExtensionTestMessageListener foo_created(
170 "checkpoint: \"/foo\" created", true); 166 "checkpoint: \"/foo\" created", true);
(...skipping 32 matching lines...) Expand 10 before | Expand all | Expand 10 after
203 199
204 sync_engine()->GoogleSigninSucceeded("test_account", "tester", "testing"); 200 sync_engine()->GoogleSigninSucceeded("test_account", "tester", "testing");
205 WaitUntilIdle(); 201 WaitUntilIdle();
206 202
207 bar_created.Reply("resume"); 203 bar_created.Reply("resume");
208 204
209 EXPECT_TRUE(catcher.GetNextResult()); 205 EXPECT_TRUE(catcher.GetNextResult());
210 } 206 }
211 207
212 } // namespace sync_file_system 208 } // namespace sync_file_system
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698