Remove legacy in DriveSyncClient.

chrome/browser/chromeos/drive/drive_sync_client.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_DRIVE_DRIVE_SYNC_CLIENT_H_
 #define CHROME_BROWSER_CHROMEOS_DRIVE_DRIVE_SYNC_CLIENT_H_
 
-#include <deque>
+#include <set>
 #include <string>
 #include <vector>
 
-#include "base/callback_forward.h"
 #include "base/memory/weak_ptr.h"
 #include "base/time.h"
-#include "chrome/browser/chromeos/drive/drive_cache.h"
 #include "chrome/browser/chromeos/drive/drive_cache_observer.h"
 #include "chrome/browser/chromeos/drive/drive_file_system_observer.h"
 #include "chrome/browser/chromeos/drive/drive_resource_metadata.h"
-#include "net/base/network_change_notifier.h"
-
-class Profile;
-class PrefChangeRegistrar;
 
 namespace drive {
 
+class DriveCache;
+class DriveCacheEntry;
 class DriveEntryProto;
 class DriveFileSystemInterface;
-class DrivePrefetcher;
 
 // The DriveSyncClient is used to synchronize pinned files on Drive and the
 // cache on the local drive. The sync client works as follows.
 //
 // When the user pins files on Drive, this client is notified about the files
 // that get pinned, and queues tasks and starts fetching these files in the
 // background.
 //
 // When the user unpins files on Drive, this client is notified about the
 // files that get unpinned, cancels tasks if these are still in the queue.
 //
 // If the user logs out before fetching of the pinned files is complete, this
 // client resumes fetching operations next time the user logs in, based on
 // the states left in the cache.
-class DriveSyncClient
-    : public DriveFileSystemObserver,
-      public DriveCacheObserver {
+class DriveSyncClient : public DriveFileSystemObserver,
+                        public DriveCacheObserver {
  public:
   // Types of sync tasks.
   enum SyncType {
     FETCH,  // Fetch a file from the Drive server.
     UPLOAD,  // Upload a file to the Drive server.
   };
 
-  // |profile| is used to access user preferences.
   // |file_system| is used access the

[hashimoto, 2013/04/19 03:59:06]

nit: This comment is quite wrong. (what is used to access the cache is |cache|.)
I think we can drop this kind of comment when we don't want to note something
special.

[kinaba, 2013/04/19 04:10:17]

Done.

   // cache (ex. store a file to the cache when the file is downloaded).
-  DriveSyncClient(Profile* profile,
-                  DriveFileSystemInterface* file_system,
+  DriveSyncClient(DriveFileSystemInterface* file_system,
                   DriveCache* cache);
   virtual ~DriveSyncClient();
 
   // Initializes the DriveSyncClient.
   void Initialize();
 
   // DriveFileSystemInterface::Observer overrides.
   virtual void OnInitialLoadFinished() OVERRIDE;
   virtual void OnFeedFromServerLoaded() OVERRIDE;
 
@@ skipping 75 matching lines @@
                            DriveFileError error,
                            const base::FilePath& local_path,
                            const std::string& ununsed_mime_type,
                            DriveFileType file_type);
 
   // Called when the file for |resource_id| is uploaded.
   // Calls DoSyncLoop() to go back to the sync loop.
   void OnUploadFileComplete(const std::string& resource_id,
                             DriveFileError error);
 
-  Profile* profile_;
   DriveFileSystemInterface* file_system_;  // Owned by DriveSystemService.
   DriveCache* cache_;  // Owned by DriveSystemService.
 
   // List of the resource ids of resources which have a fetch task created.
   std::set<std::string> fetch_list_;
 
   // List of the resource ids of resources which have a upload task created.
   std::set<std::string> upload_list_;
 
   // Fetch tasks which have been created, but not started yet.  If they are
   // removed before starting, they will be cancelled.
   std::set<std::string> pending_fetch_list_;
 
   // The delay is used for delaying processing SyncTasks in DoSyncLoop().
   base::TimeDelta delay_;
 
   // Note: This should remain the last member so it'll be destroyed and
   // invalidate its weak pointers before any other members are destroyed.
   base::WeakPtrFactory<DriveSyncClient> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(DriveSyncClient);
 };
 
 }  // namespace drive
 
 #endif  // CHROME_BROWSER_CHROMEOS_DRIVE_DRIVE_SYNC_CLIENT_H_