Implement SyncEngine::UpdateRegisteredApps()

chrome/browser/sync_file_system/drive_backend/sync_engine.h

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_SYNC_FILE_SYSTEM_DRIVE_BACKEND_SYNC_ENGINE_H_
 #define CHROME_BROWSER_SYNC_FILE_SYSTEM_DRIVE_BACKEND_SYNC_ENGINE_H_
 
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/weak_ptr.h"
 #include "base/observer_list.h"
 #include "chrome/browser/drive/drive_notification_observer.h"
 #include "chrome/browser/drive/drive_service_interface.h"
 #include "chrome/browser/sync_file_system/drive_backend/sync_engine_context.h"
 #include "chrome/browser/sync_file_system/local_change_processor.h"
 #include "chrome/browser/sync_file_system/remote_file_sync_service.h"
 #include "chrome/browser/sync_file_system/sync_task_manager.h"
 #include "net/base/network_change_notifier.h"
 
-class ExtensionService;
+class ExtensionServiceInterface;
 
 namespace base {
 class SequencedTaskRunner;
 }
 
 namespace drive {
 class DriveServiceInterface;
 class DriveNotificationManager;
 }
 
@@ skipping 12 matching lines @@
                    public drive::DriveServiceObserver,
                    public net::NetworkChangeNotifier::NetworkChangeObserver,
                    public SyncEngineContext {
  public:
   typedef Observer SyncServiceObserver;
 
   SyncEngine(const base::FilePath& base_dir,
              base::SequencedTaskRunner* task_runner,
              scoped_ptr<drive::DriveServiceInterface> drive_service,
              drive::DriveNotificationManager* notification_manager,
-             ExtensionService* extension_service);
+             ExtensionServiceInterface* extension_service);
   virtual ~SyncEngine();
 
   void Initialize();
 
   // RemoteFileSyncService overrides.
   virtual void AddServiceObserver(SyncServiceObserver* observer) OVERRIDE;
   virtual void AddFileStatusObserver(FileStatusObserver* observer) OVERRIDE;
   virtual void RegisterOrigin(
       const GURL& origin,
       const SyncStatusCallback& callback) OVERRIDE;
@@ skipping 49 matching lines @@
 
   // net::NetworkChangeNotifier::NetworkChangeObserver overrides.
   virtual void OnNetworkChanged(
       net::NetworkChangeNotifier::ConnectionType type) OVERRIDE;
 
   // SyncEngineContext overrides.
   virtual drive::DriveServiceInterface* GetDriveService() OVERRIDE;
   virtual MetadataDatabase* GetMetadataDatabase() OVERRIDE;
   virtual RemoteChangeProcessor* GetRemoteChangeProcessor() OVERRIDE;
 
+

[kinuko, 2013/11/18 08:56:18]

nit: extra empty line

[keishi, 2013/11/18 09:15:32]

Done.

  private:
+  friend class SyncEngineTest;
+
   void DoDisableApp(const std::string& app_id,
                     const SyncStatusCallback& callback);
   void DoEnableApp(const std::string& app_id,
                    const SyncStatusCallback& callback);
 
   void DidInitialize(SyncEngineInitializer* initializer,
                      SyncStatusCode status);
   void DidProcessRemoteChange(RemoteToLocalSyncer* syncer,
                               const SyncFileCallback& callback,
                               SyncStatusCode status);
   void DidApplyLocalChange(LocalToRemoteSyncer* syncer,
                            const SyncStatusCallback& callback,
                            SyncStatusCode status);
   void DidFetchChangeList(SyncStatusCallback& callback);
 
   void MaybeStartFetchChanges();
   void UpdateServiceStateFromSyncStatusCode(
       SyncStatusCode state,
       const std::string& description);
   void UpdateServiceState(RemoteServiceState state,
                           const std::string& description);
+  void UpdateRegisteredApps();
 
   base::FilePath base_dir_;
   base::FilePath temporary_file_dir_;
 
   scoped_refptr<base::SequencedTaskRunner> task_runner_;
 
   scoped_ptr<drive::DriveServiceInterface> drive_service_;
   scoped_ptr<MetadataDatabase> metadata_database_;
 
   // These external services are not owned by SyncEngine.
   // The owner of the SyncEngine is responsible for their lifetime.
   // I.e. the owner should declare the dependency explicitly by calling
   // BrowserContextKeyedService::DependsOn().
   drive::DriveNotificationManager* notification_manager_;
-  ExtensionService* extension_service_;
+  ExtensionServiceInterface* extension_service_;
 
   ObserverList<SyncServiceObserver> service_observers_;
   ObserverList<FileStatusObserver> file_status_observers_;
   RemoteChangeProcessor* remote_change_processor_;
 
   RemoteServiceState service_state_;
 
   bool should_check_remote_change_;
   base::TimeTicks time_to_check_changes_;
 
   bool sync_enabled_;
   ConflictResolutionPolicy conflict_resolution_policy_;
   bool network_available_;
 
   scoped_ptr<SyncTaskManager> task_manager_;
 
   base::WeakPtrFactory<SyncEngine> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(SyncEngine);
 };
 
 }  // namespace drive_backend
 }  // namespace sync_file_system
 
 #endif  // CHROME_BROWSER_SYNC_FILE_SYSTEM_DRIVE_BACKEND_SYNC_ENGINE_H_