drive: Make callback parameters mandatory

Looking at the code, most callback parameters always take callbacks
hence can be mandatory. DriveSystemService::ClearCacheAndRemountFileSystem()
was an exception, so add a dummy callback for now with a TODO comment.

BUG=126634
TEST=none

Review URL: https://codereview.chromium.org/11446056

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@171715 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 9 insertions, 0 deletions

diff --git a/chrome/browser/chromeos/drive/drive_scheduler.h b/chrome/browser/chromeos/drive/drive_scheduler.h
index 7f1c6f8..5f8c2ba 100644
--- a/chrome/browser/chromeos/drive/drive_scheduler.h
+++ b/chrome/browser/chromeos/drive/drive_scheduler.h
@@ -89,17 +89,21 @@ class DriveScheduler
   void Initialize();
 
   // Adds a GetAccountMetadata operation to the queue.
+  // |callback| must not be null.
   void GetAccountMetadata(const google_apis::GetDataCallback& callback);
 
   // Adds a GetApplicationInfo operation to the queue.
+  // |callback| must not be null.
   void GetApplicationInfo(const google_apis::GetDataCallback& callback);
 
   // Adds a copy operation to the queue.
+  // |callback| must not be null.
   void Copy(const FilePath& src_file_path,
             const FilePath& dest_file_path,
             const FileOperationCallback& callback);
 
   // Adds a GetDocuments operation to the queue.
+  // |callback| must not be null.
   void GetDocuments(const GURL& feed_url,
                     int64 start_changestamp,
                     const std::string& search_query,
@@ -108,26 +112,31 @@ class DriveScheduler
                     const google_apis::GetDataCallback& callback);
 
   // Adds a transfer operation to the queue.
+  // |callback| must not be null.
   void TransferFileFromRemoteToLocal(const FilePath& remote_src_file_path,
                                      const FilePath& local_dest_file_path,
                                      const FileOperationCallback& callback);
 
   // Adds a transfer operation to the queue.
+  // |callback| must not be null.
   void TransferFileFromLocalToRemote(const FilePath& local_src_file_path,
                                      const FilePath& remote_dest_file_path,
                                      const FileOperationCallback& callback);
 
   // Adds a transfer operation to the queue.
+  // |callback| must not be null.
   void TransferRegularFile(const FilePath& local_src_file_path,
                            const FilePath& remote_dest_file_path,
                            const FileOperationCallback& callback);
 
   // Adds a move operation to the queue.
+  // |callback| must not be null.
   void Move(const FilePath& src_file_path,
             const FilePath& dest_file_path,
             const FileOperationCallback& callback);
 
   // Adds a remove operation to the queue.
+  // |callback| must not be null.
   void Remove(const FilePath& file_path,
               bool is_recursive,
               const FileOperationCallback& callback);