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

Side by Side Diff: chrome/browser/sync_file_system/drive_backend/sync_engine.h

Issue 269343008: [SyncFS] Add a UMA to measure RTT in registration (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 7 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_SYNC_FILE_SYSTEM_DRIVE_BACKEND_SYNC_ENGINE_H_ 5 #ifndef CHROME_BROWSER_SYNC_FILE_SYSTEM_DRIVE_BACKEND_SYNC_ENGINE_H_
6 #define CHROME_BROWSER_SYNC_FILE_SYSTEM_DRIVE_BACKEND_SYNC_ENGINE_H_ 6 #define CHROME_BROWSER_SYNC_FILE_SYSTEM_DRIVE_BACKEND_SYNC_ENGINE_H_
7 7
8 #include <set> 8 #include <set>
9 #include <string> 9 #include <string>
10 10
(...skipping 145 matching lines...) Expand 10 before | Expand all | Expand 10 after
156 156
157 SyncEngine(scoped_ptr<drive::DriveServiceInterface> drive_service, 157 SyncEngine(scoped_ptr<drive::DriveServiceInterface> drive_service,
158 scoped_ptr<drive::DriveUploaderInterface> drive_uploader, 158 scoped_ptr<drive::DriveUploaderInterface> drive_uploader,
159 base::SequencedTaskRunner* worker_task_runner, 159 base::SequencedTaskRunner* worker_task_runner,
160 drive::DriveNotificationManager* notification_manager, 160 drive::DriveNotificationManager* notification_manager,
161 ExtensionServiceInterface* extension_service, 161 ExtensionServiceInterface* extension_service,
162 SigninManagerBase* signin_manager); 162 SigninManagerBase* signin_manager);
163 163
164 void UpdateRegisteredApps(); 164 void UpdateRegisteredApps();
165 165
166 static void DidRegisterOrigin(base::Time start_time,
167 const SyncStatusCallback& callback,
168 SyncStatusCode status);
tzik 2014/05/08 06:39:10 Can this function move to anonymous namespace in .
peria 2014/05/08 06:51:52 Done.
169
166 scoped_ptr<drive::DriveServiceInterface> drive_service_; 170 scoped_ptr<drive::DriveServiceInterface> drive_service_;
167 scoped_ptr<drive::DriveUploaderInterface> drive_uploader_; 171 scoped_ptr<drive::DriveUploaderInterface> drive_uploader_;
168 172
169 // These external services are not owned by SyncEngine. 173 // These external services are not owned by SyncEngine.
170 // The owner of the SyncEngine is responsible for their lifetime. 174 // The owner of the SyncEngine is responsible for their lifetime.
171 // I.e. the owner should declare the dependency explicitly by calling 175 // I.e. the owner should declare the dependency explicitly by calling
172 // KeyedService::DependsOn(). 176 // KeyedService::DependsOn().
173 drive::DriveNotificationManager* notification_manager_; 177 drive::DriveNotificationManager* notification_manager_;
174 ExtensionServiceInterface* extension_service_; 178 ExtensionServiceInterface* extension_service_;
175 SigninManagerBase* signin_manager_; 179 SigninManagerBase* signin_manager_;
176 180
177 ObserverList<SyncServiceObserver> service_observers_; 181 ObserverList<SyncServiceObserver> service_observers_;
178 ObserverList<FileStatusObserver> file_status_observers_; 182 ObserverList<FileStatusObserver> file_status_observers_;
179 183
180 scoped_ptr<WorkerObserver> worker_observer_; 184 scoped_ptr<WorkerObserver> worker_observer_;
181 scoped_ptr<SyncWorker> sync_worker_; 185 scoped_ptr<SyncWorker> sync_worker_;
182 scoped_refptr<base::SequencedTaskRunner> worker_task_runner_; 186 scoped_refptr<base::SequencedTaskRunner> worker_task_runner_;
183 187
184 base::WeakPtrFactory<SyncEngine> weak_ptr_factory_; 188 base::WeakPtrFactory<SyncEngine> weak_ptr_factory_;
185 DISALLOW_COPY_AND_ASSIGN(SyncEngine); 189 DISALLOW_COPY_AND_ASSIGN(SyncEngine);
186 }; 190 };
187 191
188 } // namespace drive_backend 192 } // namespace drive_backend
189 } // namespace sync_file_system 193 } // namespace sync_file_system
190 194
191 #endif // CHROME_BROWSER_SYNC_FILE_SYSTEM_DRIVE_BACKEND_SYNC_ENGINE_H_ 195 #endif // CHROME_BROWSER_SYNC_FILE_SYSTEM_DRIVE_BACKEND_SYNC_ENGINE_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698