Add browser test for CustomizationWallpaperDownloader.

chrome/browser/chromeos/login/wallpaper_manager.cc

 // Copyright (c) 2013 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/login/wallpaper_manager.h"
 
 #include <numeric>
 #include <vector>
 
 #include "ash/ash_switches.h"
@@ skipping 418 matching lines @@
   if (!started_load_at_.is_null()) {
     const base::TimeDelta elapsed = base::Time::Now() - started_load_at_;
     manager->SaveLastLoadTime(elapsed);
   }
   if (manager->pending_inactive_ == this) {
     // ProcessRequest() was never executed.
     manager->pending_inactive_ = NULL;
   }
 
   // Destroy self.
-  DCHECK(manager->loading_.size() > 0);
-
-  for (WallpaperManager::PendingList::iterator i = manager->loading_.begin();
-       i != manager->loading_.end();
-       ++i)
-    if (i->get() == this) {
-      manager->loading_.erase(i);
-      break;
-    }
+  manager->RemovePendingWallpaperFromList(this);
 }
 
 // WallpaperManager, public: ---------------------------------------------------
 
 // TestApi. For testing purpose
 WallpaperManager::TestApi::TestApi(WallpaperManager* wallpaper_manager)
     : wallpaper_manager_(wallpaper_manager) {
 }
 
 WallpaperManager::TestApi::~TestApi() {
@@ skipping 1326 matching lines @@
   if (!pending_inactive_) {
     loading_.push_back(new WallpaperManager::PendingWallpaper(
         (delayed ? GetWallpaperLoadDelay()
                  : base::TimeDelta::FromMilliseconds(0)),
         user_id));
     pending_inactive_ = loading_.back();
   }
   return pending_inactive_;
 }
 
+void WallpaperManager::RemovePendingWallpaperFromList(
+    PendingWallpaper* pending) {
+  DCHECK(loading_.size() > 0);
+  for (WallpaperManager::PendingList::iterator i = loading_.begin();
+       i != loading_.end();
+       ++i) {
+    if (i->get() == pending) {
+      loading_.erase(i);
+      break;
+    }
+  }
+
+  if (loading_.empty())
+    FOR_EACH_OBSERVER(Observer, observers_, OnPendingListEmptyForTesting());
+}
+
 void WallpaperManager::SetCustomizedDefaultWallpaper(
     const GURL& wallpaper_url,
     const base::FilePath& downloaded_file,
     const base::FilePath& resized_directory) {
   // Should fail if this ever happens in tests.
   DCHECK(wallpaper_url.is_valid());
   if (!wallpaper_url.is_valid()) {
     if (!wallpaper_url.is_empty()) {
       LOG(WARNING) << "Invalid Customized Wallpaper URL '"
                    << wallpaper_url.spec() << "'";
@@ skipping 15 matching lines @@
                  weak_factory_.GetWeakPtr(),
                  wallpaper_url,
                  downloaded_file,
                  base::Passed(rescaled_files.Pass()));
   if (!BrowserThread::PostBlockingPoolTaskAndReply(
           FROM_HERE, check_file_exists, on_checked_closure)) {
     LOG(WARNING) << "Failed to start check CheckCustomizedWallpaperFilesExist.";
   }
 }
 
+size_t WallpaperManager::GetPendingListSizeForTesting() const {
+  return loading_.size();
+}
+
 void WallpaperManager::SetDefaultWallpaperPathsFromCommandLine(
     base::CommandLine* command_line) {
   default_small_wallpaper_file_ = command_line->GetSwitchValuePath(
       ash::switches::kAshDefaultWallpaperSmall);
   default_large_wallpaper_file_ = command_line->GetSwitchValuePath(
       ash::switches::kAshDefaultWallpaperLarge);
   guest_small_wallpaper_file_ =
       command_line->GetSwitchValuePath(ash::switches::kAshGuestWallpaperSmall);
   guest_large_wallpaper_file_ =
       command_line->GetSwitchValuePath(ash::switches::kAshGuestWallpaperLarge);
@@ skipping 72 matching lines @@
     }
   }
 
   if (need_update_screen) {
     DoSetDefaultWallpaper(std::string(),
                           MovableOnDestroyCallbackHolder().Pass());
   }
 }
 
 }  // namespace chromeos