[Offline Pages] Two-step expiration in storage manager.

components/offline_pages/archive_manager.h

 // Copyright 2016 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 COMPONENTS_OFFLINE_PAGES_ARCHIVE_MANAGER_H_
 #define COMPONENTS_OFFLINE_PAGES_ARCHIVE_MANAGER_H_
 
 #include <set>
 #include <vector>
 
@@ skipping 29 matching lines @@
                              const base::Callback<void(bool)>& callback);
 
   // Deletes an archive with specified |archive_path|.
   // It is considered successful to attempt to delete a file that does not
   // exist.
   virtual void DeleteArchive(const base::FilePath& archive_path,
                              const base::Callback<void(bool)>& callback);
 
   // Deletes multiple archives that are specified in |archive_paths|.
   // It is considered successful to attempt to delete a file that does not
-  // exist.
+  // exist, or |archive_path| is empty.

[jianli, 2016/05/26 21:59:31]

Adding this behavior seems to be confusing.

[romax, 2016/05/27 00:41:37]

removed

   virtual void DeleteMultipleArchives(
       const std::vector<base::FilePath>& archive_paths,
       const base::Callback<void(bool)>& callback);
 
   // Lists all archive files in the archive directory.
   virtual void GetAllArchives(
       const base::Callback<void(const std::set<base::FilePath>&)>& callback)
       const;
 
   // Gets stats about archive storage, i.e. total archive sizes and free disk
   // space.
   virtual void GetStorageStats(
       const base::Callback<void(const StorageStats& storage_sizes)>& callback)
       const;
 
  protected:
   ArchiveManager();
 
  private:
   // Path under which all of the managed archives should be stored.
   base::FilePath archives_dir_;
   // Task runner for running file operations.
   scoped_refptr<base::SequencedTaskRunner> task_runner_;
 };
 
 }  // namespace offline_pages
 
 #endif  // COMPONENTS_OFFLINE_PAGES_ARCHIVE_MANAGER_H_