drive: Move IsDriveEnabled() and DisableDrive() to DriveSystemService

chrome/browser/chromeos/gdata/drive_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_CHROMEOS_GDATA_DRIVE_SYSTEM_SERVICE_H_
 #define CHROME_BROWSER_CHROMEOS_GDATA_DRIVE_SYSTEM_SERVICE_H_
 
 #include <string>
 
 #include "base/memory/scoped_ptr.h"
@@ skipping 34 matching lines @@
   DriveWebAppsRegistry* webapps_registry() { return webapps_registry_.get(); }
 
   // Clears all the local cache files and in-memory data, and remounts the file
   // system.
   void ClearCacheAndRemountFileSystem(
       const base::Callback<void(bool)>& callback);
 
   // ProfileKeyedService override:
   virtual void Shutdown() OVERRIDE;
 
+  // Returns true if Drive is enabled for the specified profile.
+  // Must be called on UI thread.
+  static bool IsDriveEnabled(Profile* profile);
+
  private:
   explicit DriveSystemService(Profile* profile);
   virtual ~DriveSystemService();
 
   // Initializes the object. This function should be called before any
   // other functions.
   void Initialize(DriveServiceInterface* drive_service,
                   const FilePath& cache_root);
 
   // Registers remote file system proxy for drive mount point.
   void AddDriveMountPoint();
   // Unregisters drive mount point from File API.
   void RemoveDriveMountPoint();
 
   // Adds back the drive mount point. Used to implement ClearCache().
   void AddBackDriveMountPoint(const base::Callback<void(bool)>& callback,
                               DriveFileError error,
                               const FilePath& file_path);
 
   // Called when cache initialization is done. Continues initialization if
   // the cache initialization is successful.
   void OnCacheInitialized(bool success);
 
+  // Disables Drive for the specified profile. Used to disable Drive when
+  // needed (ex. initialization of the Drive cache failed).
+  // Must be called on UI thread.
+  static void DisableDrive(Profile* profile);

[hashimoto, 2012/10/11 04:09:15]

Can't this method be a free function in the nameless namespace?

[satorux1, 2012/10/11 04:25:31]

Good point. Done!

+
   friend class DriveSystemServiceFactory;
 
   Profile* profile_;
   scoped_refptr<base::SequencedTaskRunner> blocking_task_runner_;
   DriveCache* cache_;
   scoped_ptr<DriveServiceInterface> drive_service_;
   scoped_ptr<DriveUploader> uploader_;
   scoped_ptr<DriveWebAppsRegistry> webapps_registry_;
   scoped_ptr<DriveFileSystemInterface> file_system_;
   scoped_ptr<FileWriteHelper> file_write_helper_;
@@ skipping 39 matching lines @@
   virtual ~DriveSystemServiceFactory();
 
   // ProfileKeyedServiceFactory:
   virtual ProfileKeyedService* BuildServiceInstanceFor(
       Profile* profile) const OVERRIDE;
 };
 
 }  // namespace gdata
 
 #endif  // CHROME_BROWSER_CHROMEOS_GDATA_DRIVE_SYSTEM_SERVICE_H_