Evict drive cache with file download from website if necessary.

chrome/browser/chromeos/drive/download_handler.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_DOWNLOAD_HANDLER_H_
 #define CHROME_BROWSER_CHROMEOS_DRIVE_DOWNLOAD_HANDLER_H_
 
 #include "base/callback_forward.h"
 #include "base/memory/weak_ptr.h"
 #include "chrome/browser/download/all_download_item_notifier.h"
@@ skipping 55 matching lines @@
   base::FilePath GetCacheFilePath(const content::DownloadItem* download);
 
   // Checks if there is a Drive upload associated with |download|
   bool IsDriveDownload(const content::DownloadItem* download);
 
   // Checks a file corresponding to the download item exists in Drive.
   void CheckForFileExistence(
       const content::DownloadItem* download,
       const content::CheckForFileExistenceCallback& callback);
 
+  // Checks available storage space and free disk space if necessary for
+  // |download|. If |immediate| is false, execution is delayed and rate limited.
+  void FreeDiskSpaceIfNeeded(const content::DownloadItem* download,
+                             bool immediate);

[hashimoto, 2015/10/28 09:06:16]

Generally, boolean parameters make the code less readable.
(cf. "nullptr/NULL, true/false, 1, 2, 3..." section of our style guide
https://google-styleguide.googlecode.com/svn/trunk/cppguide.html#Implementati...)

How about passing base::TimeDelta instead?

[yawano, 2015/10/29 02:52:43]

Split a method into two methods, FreeDiskSpaceIfNeeded and
FreeDiskSpaceIfNeededImmediately.

+
+  // Sets free disk space delay for testing.
+  void SetFreeDiskSpaceDelayForTesting(const base::TimeDelta& delay);
+
  private:
   // AllDownloadItemNotifier::Observer overrides:
   void OnDownloadCreated(content::DownloadManager* manager,
                          content::DownloadItem* download) override;
   void OnDownloadUpdated(content::DownloadManager* manager,
                          content::DownloadItem* download) override;
 
   // Removes the download.
   void RemoveDownload(void* manager_id, int id);
 
   // Callback for FileSystem::CreateDirectory().
   // Used to implement SubstituteDriveDownloadPath().
   void OnCreateDirectory(const SubstituteDriveDownloadPathCallback& callback,
                          FileError error);
 
   // Starts the upload of a downloaded/downloading file.
   void UploadDownloadItem(content::DownloadManager* manager,
                           content::DownloadItem* download);
 
   // Sets |cache_file_path| as user data of the download item specified by |id|.
   void SetCacheFilePath(void* manager_id,
                         int id,
                         const base::FilePath* cache_file_path,
                         FileError error);
 
+  void ExecuteFreeDiskSpaceIfNeededFor();
+
   // Gets a download manager, given a |manager_id| casted from the pointer to
   // the manager. This is used to validate the manager that may be deleted while
   // asynchronous task posting. Returns NULL if the manager is already gone.
   content::DownloadManager* GetDownloadManager(void* manager_id);
 
   FileSystemInterface* file_system_;  // Owned by DriveIntegrationService.
+
   // Observe the DownloadManager for new downloads.
   scoped_ptr<AllDownloadItemNotifier> notifier_;
   scoped_ptr<AllDownloadItemNotifier> notifier_incognito_;
 
   // Temporary download location directory.
   base::FilePath drive_tmp_download_path_;
 
+  // Pending requested space. 0 is set if there is no request.
+  int64 pending_request_space_;
+
+  base::TimeDelta free_disk_space_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<DownloadHandler> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(DownloadHandler);
 };
 
 }  // namespace drive
 
 #endif  // CHROME_BROWSER_CHROMEOS_DRIVE_DOWNLOAD_HANDLER_H_