[Offline pages] Hooking up Archive Manager to Offline Page Model

components/offline_pages/archive_manager.cc

 // 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.
 
 #include "base/bind.h"
 #include "base/callback.h"
+#include "base/files/file_enumerator.h"
 #include "base/files/file_util.h"
 #include "base/location.h"
 #include "base/logging.h"
 #include "components/offline_pages/archive_manager.h"
 
 namespace offline_pages {
 
 namespace {
 
 void EnsureArchivesDirCreatedImpl(const base::FilePath& archives_dir) {
@@ skipping 11 matching lines @@
   for (const auto& file_path : file_paths) {
     // Make sure delete happens on the left of || so that it is always executed.
     *result = base::DeleteFile(file_path, false) || *result;
   }
 }
 
 void CompleteBooleanCallback(const base::Callback<void(bool)>& callback,
                              bool* exists) {
   callback.Run(*exists);
 }
+
+void GetAllArchivesImpl(const base::FilePath& archive_dir,
+                        std::set<base::FilePath>* archive_paths) {
+  DCHECK(archive_paths);
+  archive_paths->clear();
+  base::FileEnumerator file_enumerator(archive_dir, false,
+                                       base::FileEnumerator::FILES);
+  for (base::FilePath archive_path = file_enumerator.Next();
+       !archive_path.empty();
+       archive_path = file_enumerator.Next()) {
+    archive_paths->insert(archive_path);
+  }
+}
+
+void GetAllArchivesImplDone(
+    const base::Callback<void(const std::set<base::FilePath>&)>& callback,
+    std::set<base::FilePath>* archive_paths) {
+  callback.Run(*archive_paths);
+}
+
 }  // namespace
 
 ArchiveManager::ArchiveManager(
     const base::FilePath& archives_dir,
     const scoped_refptr<base::SequencedTaskRunner>& task_runner)
     : archives_dir_(archives_dir), task_runner_(task_runner) {}
 
 ArchiveManager::~ArchiveManager() {}
 
 void ArchiveManager::EnsureArchivesDirCreated(const base::Closure& callback) {
@@ skipping 18 matching lines @@
 
 void ArchiveManager::DeleteMultipleArchives(
     const std::vector<base::FilePath>& archive_paths,
     const base::Callback<void(bool)>& callback) {
   bool* result = new bool(false);
   task_runner_->PostTaskAndReply(
       FROM_HERE, base::Bind(DeleteArchivesImpl, archive_paths, result),
       base::Bind(CompleteBooleanCallback, callback, base::Owned(result)));
 }
 
+void ArchiveManager::GetAllArchives(
+    const base::Callback<void(const std::set<base::FilePath>&)>& callback)
+    const {
+  std::set<base::FilePath>* archive_paths = new std::set<base::FilePath>();
+  task_runner_->PostTaskAndReply(
+      FROM_HERE, base::Bind(GetAllArchivesImpl, archives_dir_, archive_paths),
+      base::Bind(GetAllArchivesImplDone, callback, base::Owned(archive_paths)));

[jianli, 2016/05/17 21:48:13]

I think you don't need GetAllArchivesImplDone.

[fgorski, 2016/05/17 22:50:29]

Changed per offline discussion.

+}
+
 }  // namespace offline_pages