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

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

Issue 408143014: Rename OperationObserver to OperationDelegate (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 5 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"
11 #include "chrome/browser/chromeos/drive/drive.pb.h" 11 #include "chrome/browser/chromeos/drive/drive.pb.h"
12 #include "chrome/browser/chromeos/drive/file_change.h" 12 #include "chrome/browser/chromeos/drive/file_change.h"
13 #include "chrome/browser/chromeos/drive/file_errors.h" 13 #include "chrome/browser/chromeos/drive/file_errors.h"
14 #include "chrome/browser/chromeos/drive/file_system/operation_observer.h" 14 #include "chrome/browser/chromeos/drive/file_system/operation_delegate.h"
15 #include "chrome/browser/chromeos/drive/test_util.h" 15 #include "chrome/browser/chromeos/drive/test_util.h"
16 #include "content/public/test/test_browser_thread_bundle.h" 16 #include "content/public/test/test_browser_thread_bundle.h"
17 #include "testing/gtest/include/gtest/gtest.h" 17 #include "testing/gtest/include/gtest/gtest.h"
18 18
19 class TestingPrefServiceSimple; 19 class TestingPrefServiceSimple;
20 20
21 namespace base { 21 namespace base {
22 class SequencedTaskRunner; 22 class SequencedTaskRunner;
23 } // namespace base 23 } // namespace base
24 24
(...skipping 13 matching lines...) Expand all
38 class ResourceMetadataStorage; 38 class ResourceMetadataStorage;
39 } // namespace internal 39 } // namespace internal
40 40
41 namespace file_system { 41 namespace file_system {
42 42
43 // Base fixture class for testing Drive file system operations. It sets up the 43 // Base fixture class for testing Drive file system operations. It sets up the
44 // basic set of Drive internal classes (ResourceMetadata, Cache, etc) on top of 44 // basic set of Drive internal classes (ResourceMetadata, Cache, etc) on top of
45 // FakeDriveService for testing. 45 // FakeDriveService for testing.
46 class OperationTestBase : public testing::Test { 46 class OperationTestBase : public testing::Test {
47 protected: 47 protected:
48 // OperationObserver that records all the events. 48 // OperationDelegate that records all the events.
49 class LoggingObserver : public OperationObserver { 49 class LoggingDelegate : public OperationDelegate {
50 public: 50 public:
51 typedef base::Callback<bool( 51 typedef base::Callback<bool(
52 const std::string& local_id, 52 const std::string& local_id,
53 const FileOperationCallback& callback)> WaitForSyncCompleteHandler; 53 const FileOperationCallback& callback)> WaitForSyncCompleteHandler;
54 54
55 LoggingObserver(); 55 LoggingDelegate();
56 ~LoggingObserver(); 56 ~LoggingDelegate();
57 57
58 // OperationObserver overrides. 58 // OperationDelegate overrides.
59 virtual void OnFileChangedByOperation( 59 virtual void OnFileChangedByOperation(
60 const FileChange& changed_files) OVERRIDE; 60 const FileChange& changed_files) OVERRIDE;
61 virtual void OnEntryUpdatedByOperation( 61 virtual void OnEntryUpdatedByOperation(
62 const std::string& local_id) OVERRIDE; 62 const std::string& local_id) OVERRIDE;
63 virtual void OnDriveSyncError(DriveSyncErrorType type, 63 virtual void OnDriveSyncError(DriveSyncErrorType type,
64 const std::string& local_id) OVERRIDE; 64 const std::string& local_id) OVERRIDE;
65 virtual bool WaitForSyncComplete( 65 virtual bool WaitForSyncComplete(
66 const std::string& local_id, 66 const std::string& local_id,
67 const FileOperationCallback& callback) OVERRIDE; 67 const FileOperationCallback& callback) OVERRIDE;
68 68
(...skipping 48 matching lines...) Expand 10 before | Expand all | Expand 10 after
117 117
118 // Synchronously updates |metadata_| by fetching the change feed from the 118 // Synchronously updates |metadata_| by fetching the change feed from the
119 // |fake_service_|. 119 // |fake_service_|.
120 FileError CheckForUpdates(); 120 FileError CheckForUpdates();
121 121
122 // Accessors for the components. 122 // Accessors for the components.
123 FakeDriveService* fake_service() { 123 FakeDriveService* fake_service() {
124 return fake_drive_service_.get(); 124 return fake_drive_service_.get();
125 } 125 }
126 EventLogger* logger() { return logger_.get(); } 126 EventLogger* logger() { return logger_.get(); }
127 LoggingObserver* observer() { return &observer_; } 127 LoggingDelegate* delegate() { return &delegate_; }
128 JobScheduler* scheduler() { return scheduler_.get(); } 128 JobScheduler* scheduler() { return scheduler_.get(); }
129 base::SequencedTaskRunner* blocking_task_runner() { 129 base::SequencedTaskRunner* blocking_task_runner() {
130 return blocking_task_runner_.get(); 130 return blocking_task_runner_.get();
131 } 131 }
132 FakeFreeDiskSpaceGetter* fake_free_disk_space_getter() { 132 FakeFreeDiskSpaceGetter* fake_free_disk_space_getter() {
133 return fake_free_disk_space_getter_.get(); 133 return fake_free_disk_space_getter_.get();
134 } 134 }
135 internal::FileCache* cache() { return cache_.get(); } 135 internal::FileCache* cache() { return cache_.get(); }
136 internal::ResourceMetadata* metadata() { return metadata_.get(); } 136 internal::ResourceMetadata* metadata() { return metadata_.get(); }
137 internal::LoaderController* loader_controller() { 137 internal::LoaderController* loader_controller() {
138 return loader_controller_.get(); 138 return loader_controller_.get();
139 } 139 }
140 internal::ChangeListLoader* change_list_loader() { 140 internal::ChangeListLoader* change_list_loader() {
141 return change_list_loader_.get(); 141 return change_list_loader_.get();
142 } 142 }
143 143
144 private: 144 private:
145 content::TestBrowserThreadBundle thread_bundle_; 145 content::TestBrowserThreadBundle thread_bundle_;
146 scoped_refptr<base::SequencedTaskRunner> blocking_task_runner_; 146 scoped_refptr<base::SequencedTaskRunner> blocking_task_runner_;
147 scoped_ptr<TestingPrefServiceSimple> pref_service_; 147 scoped_ptr<TestingPrefServiceSimple> pref_service_;
148 base::ScopedTempDir temp_dir_; 148 base::ScopedTempDir temp_dir_;
149 149
150 LoggingObserver observer_; 150 LoggingDelegate delegate_;
151 scoped_ptr<EventLogger> logger_; 151 scoped_ptr<EventLogger> logger_;
152 scoped_ptr<FakeDriveService> fake_drive_service_; 152 scoped_ptr<FakeDriveService> fake_drive_service_;
153 scoped_ptr<JobScheduler> scheduler_; 153 scoped_ptr<JobScheduler> scheduler_;
154 scoped_ptr<internal::ResourceMetadataStorage, 154 scoped_ptr<internal::ResourceMetadataStorage,
155 test_util::DestroyHelperForTests> metadata_storage_; 155 test_util::DestroyHelperForTests> metadata_storage_;
156 scoped_ptr<FakeFreeDiskSpaceGetter> fake_free_disk_space_getter_; 156 scoped_ptr<FakeFreeDiskSpaceGetter> fake_free_disk_space_getter_;
157 scoped_ptr<internal::FileCache, test_util::DestroyHelperForTests> cache_; 157 scoped_ptr<internal::FileCache, test_util::DestroyHelperForTests> cache_;
158 scoped_ptr<internal::ResourceMetadata, test_util::DestroyHelperForTests> 158 scoped_ptr<internal::ResourceMetadata, test_util::DestroyHelperForTests>
159 metadata_; 159 metadata_;
160 scoped_ptr<internal::AboutResourceLoader> about_resource_loader_; 160 scoped_ptr<internal::AboutResourceLoader> about_resource_loader_;
161 scoped_ptr<internal::LoaderController> loader_controller_; 161 scoped_ptr<internal::LoaderController> loader_controller_;
162 scoped_ptr<internal::ChangeListLoader> change_list_loader_; 162 scoped_ptr<internal::ChangeListLoader> change_list_loader_;
163 }; 163 };
164 164
165 } // namespace file_system 165 } // namespace file_system
166 } // namespace drive 166 } // namespace drive
167 167
168 #endif // CHROME_BROWSER_CHROMEOS_DRIVE_FILE_SYSTEM_OPERATION_TEST_BASE_H_ 168 #endif // CHROME_BROWSER_CHROMEOS_DRIVE_FILE_SYSTEM_OPERATION_TEST_BASE_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698