[Offline pages] Moving disk size related calls to Archive Manager, reorganizing UMA

components/offline_pages/archive_manager_unittest.cc

Index: components/offline_pages/archive_manager_unittest.cc
diff --git a/components/offline_pages/archive_manager_unittest.cc b/components/offline_pages/archive_manager_unittest.cc
index 15dba80971ba52adc98cefef9d4273758dba8569..7013033c2a62c415645587ac3f4951d7d0d2a764 100644
--- a/components/offline_pages/archive_manager_unittest.cc
+++ b/components/offline_pages/archive_manager_unittest.cc
@@ -39,6 +39,7 @@ class ArchiveManagerTest : public testing::Test {
   void ResetManager(const base::FilePath& file_path);
   void Callback(bool result);
   void GetAllArchivesCallback(const std::set<base::FilePath>& archive_paths);
+  void GetStorageSizesCallback(ArchiveManager::StorageSizes storage_sizes);
 
   ArchiveManager* manager() { return manager_.get(); }
   const base::FilePath& temp_path() const { return temp_dir_.path(); }
@@ -46,6 +47,9 @@ class ArchiveManagerTest : public testing::Test {
   const std::set<base::FilePath>& last_archvie_paths() const {
     return last_archvie_paths_;
   }
+  ArchiveManager::StorageSizes last_storage_sizes() const {
+    return last_storage_sizes_;
+  }
 
  private:
   scoped_refptr<base::TestSimpleTaskRunner> task_runner_;
@@ -55,12 +59,14 @@ class ArchiveManagerTest : public testing::Test {
   std::unique_ptr<ArchiveManager> manager_;
   CallbackStatus callback_status_;
   std::set<base::FilePath> last_archvie_paths_;
+  ArchiveManager::StorageSizes last_storage_sizes_;
 };
 
 ArchiveManagerTest::ArchiveManagerTest()
     : task_runner_(new base::TestSimpleTaskRunner),
       task_runner_handle_(task_runner_),
-      callback_status_(CallbackStatus::NOT_CALLED) {}
+      callback_status_(CallbackStatus::NOT_CALLED),
+      last_storage_sizes_({0, 0}) {}
 
 void ArchiveManagerTest::SetUp() {
   ASSERT_TRUE(temp_dir_.CreateUniqueTempDir());
@@ -91,6 +97,11 @@ void ArchiveManagerTest::GetAllArchivesCallback(
   last_archvie_paths_ = archive_paths;
 }
 
+void ArchiveManagerTest::GetStorageSizesCallback(
+    ArchiveManager::StorageSizes storage_sizes) {
+  last_storage_sizes_ = storage_sizes;
+}
+
 TEST_F(ArchiveManagerTest, EnsureArchivesDirCreated) {
   base::FilePath archive_dir =
       temp_path().Append(FILE_PATH_LITERAL("test_path"));
@@ -235,4 +246,18 @@ TEST_F(ArchiveManagerTest, GetAllArchives) {
   EXPECT_EQ(1UL, last_archvie_paths().count(archive_path_3));
 }
 
+TEST_F(ArchiveManagerTest, GetStorageSizes) {
+  base::FilePath archive_path_1;
+  EXPECT_TRUE(base::CreateTemporaryFileInDir(temp_path(), &archive_path_1));
+  base::FilePath archive_path_2;
+  EXPECT_TRUE(base::CreateTemporaryFileInDir(temp_path(), &archive_path_2));
+
+  manager()->GetStorageSizes(base::Bind(
+      &ArchiveManagerTest::GetStorageSizesCallback, base::Unretained(this)));
+  PumpLoop();
+  EXPECT_GT(last_storage_sizes().free_disk_space, 0);
+  EXPECT_EQ(last_storage_sizes().total_archives_size,
+            base::ComputeDirectorySize(temp_path()));
+}
+
 }  // namespace offline_pages