drive: Remove polling code from DriveFileSystem

chrome/browser/chromeos/drive/drive_file_system.cc

 // 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.
 
 #include "chrome/browser/chromeos/drive/drive_file_system.h"
 
 #include "base/bind.h"
 #include "base/file_util.h"
 #include "base/json/json_file_value_serializer.h"
 #include "base/message_loop_proxy.h"
@@ skipping 23 matching lines @@
 #include "content/public/browser/notification_details.h"
 
 using content::BrowserThread;
 
 namespace drive {
 namespace {
 
 const char kMimeTypeJson[] = "application/json";
 const char kEmptyFilePath[] = "/dev/null";
 
-// Drive update polling interval for polling only mode (in seconds).
-const int kFastPollingIntervalInSec = 60;
-
-// Drive update polling interval when update notification is available (in
-// seconds). Ideally we don't need this, but we do polling in case update
-// notification doesn't work. http://crbug.com/157080
-const int kSlowPollingIntervalInSec = 300;
-
 //================================ Helper functions ============================
 
 // The class to wait for the drive service to be ready to start operation.
 class OperationReadinessObserver : public google_apis::DriveServiceObserver {
  public:
   OperationReadinessObserver(google_apis::DriveServiceInterface* drive_service,
                              const base::Closure& callback)
       : drive_service_(drive_service),
         callback_(callback) {
     DCHECK(!callback_.is_null());
@@ skipping 152 matching lines @@
     DriveCache* cache,
     google_apis::DriveServiceInterface* drive_service,
     DriveWebAppsRegistry* webapps_registry,
     DriveResourceMetadata* resource_metadata,
     base::SequencedTaskRunner* blocking_task_runner)
     : profile_(profile),
       cache_(cache),
       drive_service_(drive_service),
       webapps_registry_(webapps_registry),
       resource_metadata_(resource_metadata),
-      update_timer_(true /* retain_user_task */, true /* is_repeating */),
       last_update_check_error_(DRIVE_FILE_OK),
       hide_hosted_docs_(false),
       blocking_task_runner_(blocking_task_runner),
       scheduler_(new DriveScheduler(profile, drive_service)),
-      polling_interval_sec_(kFastPollingIntervalInSec),
       push_notification_enabled_(false),
       ALLOW_THIS_IN_INITIALIZER_LIST(weak_ptr_factory_(this)) {
   // Should be created from the file browser extension API on UI thread.
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
 }
 
 void DriveFileSystem::Reload() {
   resource_metadata_->Reset(base::Bind(&DriveFileSystem::ReloadAfterReset,
                                        weak_ptr_factory_.GetWeakPtr()));
 }
@@ skipping 87 matching lines @@
     // OnReadyToPerformOperations() gets called.
     new OperationReadinessObserver(
         drive_service_,
         base::Bind(&DriveFileSystem::LoadIfNeeded,
                    weak_ptr_factory_.GetWeakPtr(),
                    DirectoryFetchInfo(),
                    base::Bind(&util::EmptyFileOperationCallback)));
   }
 }
 
-void DriveFileSystem::StartPolling() {
-  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
-
-  DCHECK(!update_timer_.IsRunning());
-  update_timer_.Start(FROM_HERE,
-                      base::TimeDelta::FromSeconds(polling_interval_sec_),
-                      base::Bind(&DriveFileSystem::CheckForUpdates,
-                                 weak_ptr_factory_.GetWeakPtr()));
-}
-
-void DriveFileSystem::StopPolling() {
-  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
-  if (update_timer_.IsRunning())
-    update_timer_.Stop();
-}
-
 void DriveFileSystem::SetPushNotificationEnabled(bool enabled) {
   push_notification_enabled_ = enabled;
-  polling_interval_sec_ = enabled ? kSlowPollingIntervalInSec :
-                          kFastPollingIntervalInSec;
 }
 
 void DriveFileSystem::GetEntryInfoByResourceId(
     const std::string& resource_id,
     const GetEntryInfoWithFilePathCallback& callback) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   DCHECK(!resource_id.empty());
   DCHECK(!callback.is_null());
 
   resource_metadata_->GetEntryInfoByResourceId(
@@ skipping 990 matching lines @@
     const GetFilesystemMetadataCallback& callback) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   DCHECK(!callback.is_null());
 
   DriveFileSystemMetadata metadata;
   metadata.loaded = change_list_loader_->loaded();
   metadata.refreshing = change_list_loader_->refreshing();
 
   // Metadata related to delta update.
   metadata.push_notification_enabled = push_notification_enabled_;
-  metadata.polling_interval_sec = polling_interval_sec_;
   metadata.last_update_check_time = last_update_check_time_;
   metadata.last_update_check_error = last_update_check_error_;
 
   resource_metadata_->GetLargestChangestamp(
       base::Bind(&OnGetLargestChangestamp, metadata, callback));
 }
 
 void DriveFileSystem::OnDisableDriveHostedFilesChanged() {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   PrefService* pref_service = profile_->GetPrefs();
@@ skipping 318 matching lines @@
     return;
   }
 
   PlatformFileInfoProto entry_file_info;
   util::ConvertPlatformFileInfoToProto(*file_info, &entry_file_info);
   *entry_proto->mutable_file_info() = entry_file_info;
   callback.Run(DRIVE_FILE_OK, entry_proto.Pass());
 }
 
 }  // namespace drive