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

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

Issue 26594002: Chrome sync: Put WeakPtrFactory members at the end (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 2 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 "base/memory/scoped_ptr.h" 8 #include "base/memory/scoped_ptr.h"
9 #include "base/memory/weak_ptr.h" 9 #include "base/memory/weak_ptr.h"
10 #include "base/observer_list.h" 10 #include "base/observer_list.h"
(...skipping 127 matching lines...) Expand 10 before | Expand all | Expand 10 after
138 // The owner of the SyncEngine is responsible for their lifetime. 138 // The owner of the SyncEngine is responsible for their lifetime.
139 // I.e. the owner should declare the dependency explicitly by calling 139 // I.e. the owner should declare the dependency explicitly by calling
140 // BrowserContextKeyedService::DependsOn(). 140 // BrowserContextKeyedService::DependsOn().
141 drive::DriveNotificationManager* notification_manager_; 141 drive::DriveNotificationManager* notification_manager_;
142 ExtensionService* extension_service_; 142 ExtensionService* extension_service_;
143 143
144 ObserverList<SyncServiceObserver> service_observers_; 144 ObserverList<SyncServiceObserver> service_observers_;
145 ObserverList<FileStatusObserver> file_status_observers_; 145 ObserverList<FileStatusObserver> file_status_observers_;
146 RemoteChangeProcessor* remote_change_processor_; 146 RemoteChangeProcessor* remote_change_processor_;
147 147
148 SyncTaskManager task_manager_;
148 base::WeakPtrFactory<SyncEngine> weak_ptr_factory_; 149 base::WeakPtrFactory<SyncEngine> weak_ptr_factory_;
149 SyncTaskManager task_manager_;
150 150
151 DISALLOW_COPY_AND_ASSIGN(SyncEngine); 151 DISALLOW_COPY_AND_ASSIGN(SyncEngine);
152 }; 152 };
153 153
154 } // namespace drive_backend 154 } // namespace drive_backend
155 } // namespace sync_file_system 155 } // namespace sync_file_system
156 156
157 #endif // CHROME_BROWSER_SYNC_FILE_SYSTEM_DRIVE_BACKEND_SYNC_ENGINE_H_ 157 #endif // CHROME_BROWSER_SYNC_FILE_SYSTEM_DRIVE_BACKEND_SYNC_ENGINE_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698