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

Side by Side Diff: chrome/browser/chromeos/drive/file_system/operation_test_base.h

Issue 150113002: drive: Also lock SearchOperation when locking ChangeListLoader (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 10 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 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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 #ifndef CHROME_BROWSER_CHROMEOS_DRIVE_FILE_SYSTEM_OPERATION_TEST_BASE_H_ 5 #ifndef CHROME_BROWSER_CHROMEOS_DRIVE_FILE_SYSTEM_OPERATION_TEST_BASE_H_
6 #define CHROME_BROWSER_CHROMEOS_DRIVE_FILE_SYSTEM_OPERATION_TEST_BASE_H_ 6 #define CHROME_BROWSER_CHROMEOS_DRIVE_FILE_SYSTEM_OPERATION_TEST_BASE_H_
7 7
8 #include <set> 8 #include <set>
9 9
10 #include "base/files/scoped_temp_dir.h" 10 #include "base/files/scoped_temp_dir.h"
(...skipping 12 matching lines...) Expand all
23 23
24 namespace drive { 24 namespace drive {
25 25
26 class FakeDriveService; 26 class FakeDriveService;
27 class FakeFreeDiskSpaceGetter; 27 class FakeFreeDiskSpaceGetter;
28 class JobScheduler; 28 class JobScheduler;
29 29
30 namespace internal { 30 namespace internal {
31 class ChangeListLoader; 31 class ChangeListLoader;
32 class FileCache; 32 class FileCache;
33 class LoaderController;
33 class ResourceMetadata; 34 class ResourceMetadata;
34 class ResourceMetadataStorage; 35 class ResourceMetadataStorage;
35 } // namespace internal 36 } // namespace internal
36 37
37 namespace file_system { 38 namespace file_system {
38 39
39 // Base fixture class for testing Drive file system operations. It sets up the 40 // Base fixture class for testing Drive file system operations. It sets up the
40 // basic set of Drive internal classes (ResourceMetadata, Cache, etc) on top of 41 // basic set of Drive internal classes (ResourceMetadata, Cache, etc) on top of
41 // FakeDriveService for testing. 42 // FakeDriveService for testing.
42 class OperationTestBase : public testing::Test { 43 class OperationTestBase : public testing::Test {
(...skipping 67 matching lines...) Expand 10 before | Expand all | Expand 10 after
110 LoggingObserver* observer() { return &observer_; } 111 LoggingObserver* observer() { return &observer_; }
111 JobScheduler* scheduler() { return scheduler_.get(); } 112 JobScheduler* scheduler() { return scheduler_.get(); }
112 base::SequencedTaskRunner* blocking_task_runner() { 113 base::SequencedTaskRunner* blocking_task_runner() {
113 return blocking_task_runner_.get(); 114 return blocking_task_runner_.get();
114 } 115 }
115 internal::ResourceMetadata* metadata() { return metadata_.get(); } 116 internal::ResourceMetadata* metadata() { return metadata_.get(); }
116 FakeFreeDiskSpaceGetter* fake_free_disk_space_getter() { 117 FakeFreeDiskSpaceGetter* fake_free_disk_space_getter() {
117 return fake_free_disk_space_getter_.get(); 118 return fake_free_disk_space_getter_.get();
118 } 119 }
119 internal::FileCache* cache() { return cache_.get(); } 120 internal::FileCache* cache() { return cache_.get(); }
121 internal::LoaderController* loader_controller() {
122 return loader_controller_.get();
123 }
120 internal::ChangeListLoader* change_list_loader() { 124 internal::ChangeListLoader* change_list_loader() {
121 return change_list_loader_.get(); 125 return change_list_loader_.get();
122 } 126 }
123 127
124 private: 128 private:
125 content::TestBrowserThreadBundle thread_bundle_; 129 content::TestBrowserThreadBundle thread_bundle_;
126 scoped_refptr<base::SequencedTaskRunner> blocking_task_runner_; 130 scoped_refptr<base::SequencedTaskRunner> blocking_task_runner_;
127 scoped_ptr<TestingPrefServiceSimple> pref_service_; 131 scoped_ptr<TestingPrefServiceSimple> pref_service_;
128 base::ScopedTempDir temp_dir_; 132 base::ScopedTempDir temp_dir_;
129 133
130 LoggingObserver observer_; 134 LoggingObserver observer_;
131 scoped_ptr<FakeDriveService> fake_drive_service_; 135 scoped_ptr<FakeDriveService> fake_drive_service_;
132 scoped_ptr<JobScheduler> scheduler_; 136 scoped_ptr<JobScheduler> scheduler_;
133 scoped_ptr<internal::ResourceMetadataStorage, 137 scoped_ptr<internal::ResourceMetadataStorage,
134 test_util::DestroyHelperForTests> metadata_storage_; 138 test_util::DestroyHelperForTests> metadata_storage_;
135 scoped_ptr<internal::ResourceMetadata, test_util::DestroyHelperForTests> 139 scoped_ptr<internal::ResourceMetadata, test_util::DestroyHelperForTests>
136 metadata_; 140 metadata_;
137 scoped_ptr<FakeFreeDiskSpaceGetter> fake_free_disk_space_getter_; 141 scoped_ptr<FakeFreeDiskSpaceGetter> fake_free_disk_space_getter_;
138 scoped_ptr<internal::FileCache, test_util::DestroyHelperForTests> cache_; 142 scoped_ptr<internal::FileCache, test_util::DestroyHelperForTests> cache_;
143 scoped_ptr<internal::LoaderController> loader_controller_;
139 scoped_ptr<internal::ChangeListLoader> change_list_loader_; 144 scoped_ptr<internal::ChangeListLoader> change_list_loader_;
140 }; 145 };
141 146
142 } // namespace file_system 147 } // namespace file_system
143 } // namespace drive 148 } // namespace drive
144 149
145 #endif // CHROME_BROWSER_CHROMEOS_DRIVE_FILE_SYSTEM_OPERATION_TEST_BASE_H_ 150 #endif // CHROME_BROWSER_CHROMEOS_DRIVE_FILE_SYSTEM_OPERATION_TEST_BASE_H_
OLDNEW
« no previous file with comments | « chrome/browser/chromeos/drive/file_system.cc ('k') | chrome/browser/chromeos/drive/file_system/operation_test_base.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698