drive: Minor cleanup around ChangeListLoader::LoadFromServerIfNeeded()

chrome/browser/chromeos/drive/change_list_loader.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_CHANGE_LIST_LOADER_H_
 #define CHROME_BROWSER_CHROMEOS_DRIVE_CHANGE_LIST_LOADER_H_
 
 #include <map>
 #include <string>
 #include <utility>
@@ skipping 70 matching lines @@
 
   // Starts retrieving search results for |search_query| from the server.
   // If |next_feed| is set, this is the feed url that will be fetched.
   // If |next_feed| is an empty string, the default URL is used.
   // Upon completion, |callback| is invoked.
   // |callback| must not be null.
   void SearchFromServer(const std::string& search_query,
                         const GURL& next_feed,
                         const LoadFeedListCallback& callback);
 
-  // Initiates the change list loading from the server if the local
-  // changestamp is older than the server changestamp.
-  // See the comment at Load() for |directory_fetch_info| parameter.
-  // |callback| must not be null.
-  // TODO(satorux): make this private. crbug.com/193417
-  void LoadFromServerIfNeeded(const DirectoryFetchInfo& directory_fetch_info,
-                              const FileOperationCallback& callback);
+  // Checks for updates on the server. Does nothing if the change list is now
+  // being loaded or refreshed. |callback| must not be null.
+  void CheckForUpdates(const FileOperationCallback& callback);
 
   // Updates whole directory structure feeds collected in |feed_list|.
   // Record file statistics as UMA histograms.
   //
   // See comments at ChangeListProcessor::ApplyFeeds() for
   // |about_resource| and |is_delta_feed|.
   // |update_finished_callback| must not be null.
   void UpdateFromFeed(scoped_ptr<google_apis::AboutResource> about_resource,
                       ScopedVector<ChangeList> change_lists,
                       bool is_delta_feed,
                       const base::Closure& update_finished_callback);
 
   // Indicates whether there is a feed refreshing server request is in flight.
   bool refreshing() const { return refreshing_; }
 
  private:
+  // Initiates the change list loading from the server if the local
+  // changestamp is older than the server changestamp.
+  // See the comment at Load() for |directory_fetch_info| parameter.
+  // |callback| must not be null.
+  void LoadFromServerIfNeeded(const DirectoryFetchInfo& directory_fetch_info,
+                              const FileOperationCallback& callback);
+
   // Checks the local changestamp. |callback| must not be null.
   void CheckLocalChangestamp(const GetChangestampCallback& callback);
 
   // Part of Load(). Called after checking the local changestamp completes.
   void LoadAfterCheckLocalChangestamp(
       const DirectoryFetchInfo& directory_fetch_info,
       const FileOperationCallback& callback,
       int64 local_changestamp);
 
   // Callback to fetch all the resource list response from the server.
@@ skipping 173 matching lines @@
 
   // 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<ChangeListLoader> weak_ptr_factory_;
   DISALLOW_COPY_AND_ASSIGN(ChangeListLoader);
 };
 
 }  // namespace drive
 
 #endif  // CHROME_BROWSER_CHROMEOS_DRIVE_CHANGE_LIST_LOADER_H_