Add RemoteFileSyncService interface

chrome/browser/sync_file_system/sync_file_system_service.h

 // Copyright (c) 2012 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_SYNC_FILE_SYSTEM_SERVICE_H_
 #define CHROME_BROWSER_SYNC_FILE_SYSTEM_SYNC_FILE_SYSTEM_SERVICE_H_
 
 #include <string>
 
 #include "base/basictypes.h"
 #include "base/callback_forward.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/singleton.h"
+#include "base/memory/weak_ptr.h"
 #include "chrome/browser/profiles/profile_keyed_service.h"
 #include "chrome/browser/profiles/profile_keyed_service_factory.h"
+#include "chrome/browser/sync_file_system/remote_file_provider.h"
 #include "webkit/fileapi/syncable/sync_status_code.h"
 
 class GURL;
 
 namespace fileapi {
 class FileSystemContext;
+class FileSystemURL;
 }
 
 namespace sync_file_system {
 
 class LocalFileSyncService;
 
-class SyncFileSystemService : public ProfileKeyedService {
+class SyncFileSystemService
+    : public ProfileKeyedService,
+      public RemoteFileObserver,
+      public base::SupportsWeakPtr<SyncFileSystemService> {
  public:
   typedef base::Callback<void(fileapi::SyncStatusCode status)> StatusCallback;
 
   // ProfileKeyedService overrides.
   virtual void Shutdown() OVERRIDE;
 
+  // RemoteFileObserver override.
+  virtual void OnRemoteChangeAvailable(RemoteFileProvider* remote_file_provider)
+      OVERRIDE;
+
+  void RegisterRemoteFileProvider(
+      const std::string& service_name,
+      scoped_ptr<RemoteFileProvider> remote_file_provider);
+
   void InitializeForApp(
       fileapi::FileSystemContext* file_system_context,
       const std::string& service_name,
       const GURL& app_url,
       const StatusCallback& callback);
 
  private:
   friend class SyncFileSystemServiceFactory;
+  typedef std::map<std::string, RemoteFileProvider*> RemoteFileProviderMap;
 
   explicit SyncFileSystemService(Profile* profile);
   virtual ~SyncFileSystemService();
 
   void Initialize(scoped_ptr<LocalFileSyncService> local_file_service);
 
+  void MayStartRemoteSync();
+  void DidDownloadFile(scoped_ptr<RemoteChange> change,
+                       fileapi::SyncStatusCode status,
+                       const FilePath& file_path);
+  void DidDeleteFile(const fileapi::FileSystemURL& url,
+                     fileapi::SyncStatusCode status);
+
   Profile* profile_;
 
   scoped_ptr<LocalFileSyncService> local_file_service_;
+  RemoteFileProviderMap remote_file_providers_;
+  std::set<RemoteFileProvider*> pending_providers_;

[kinuko, 2012/10/17 09:22:57]

I haven't looked closely at the .cc file but what this 'pending' means?

[tzik, 2012/10/18 07:18:08]

Removed.

In this version of patchset, I considered multiprovider case, and stored
providers that have pending changes here.

+  bool sync_is_running_;
 
   DISALLOW_COPY_AND_ASSIGN(SyncFileSystemService);
 };
 
 class SyncFileSystemServiceFactory : public ProfileKeyedServiceFactory {
  public:
   static SyncFileSystemService* GetForProfile(Profile* profile);
   static SyncFileSystemService* FindForProfile(Profile* profile);
   static SyncFileSystemServiceFactory* GetInstance();
 
  private:
   friend struct DefaultSingletonTraits<SyncFileSystemServiceFactory>;
   SyncFileSystemServiceFactory();
   virtual ~SyncFileSystemServiceFactory();
 
   // ProfileKeyedServiceFactory overrides.
   virtual ProfileKeyedService* BuildServiceInstanceFor(
       Profile* profile) const OVERRIDE;
 };
 
 }  // namespace sync_file_system
 
 #endif  // CHROME_BROWSER_SYNC_FILE_SYSTEM_SYNC_FILE_SYSTEM_SERVICE_H_