Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4585)

Unified Diff: chrome/browser/chromeos/login/wallpaper_manager_browsertest.cc

Issue 11415015: Remove use of index in wallpaper picker code and some refactor (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Please review this patch Created 8 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/chromeos/login/wallpaper_manager_browsertest.cc
diff --git a/chrome/browser/chromeos/login/wallpaper_manager_browsertest.cc b/chrome/browser/chromeos/login/wallpaper_manager_browsertest.cc
index c6c508e0831622e490d559b8e2161f0aacd45aae..c89dc68377b142cc84a39cd66c567830f25a5e96 100644
--- a/chrome/browser/chromeos/login/wallpaper_manager_browsertest.cc
+++ b/chrome/browser/chromeos/login/wallpaper_manager_browsertest.cc
@@ -4,7 +4,6 @@
#include "chrome/browser/chromeos/login/wallpaper_manager.h"
-#include "ash/desktop_background/desktop_background_resources.h"
#include "ash/desktop_background/desktop_background_controller.h"
#include "ash/desktop_background/desktop_background_controller_observer.h"
#include "ash/display/display_manager.h"
@@ -44,7 +43,6 @@ int kExpectedLargeWallpaperWidth = 256;
int kExpectedLargeWallpaperHeight = ash::kLargeWallpaperMaxHeight;
#endif
-
} // namespace
class WallpaperManagerBrowserTest : public CrosInProcessBrowserTest,
@@ -171,15 +169,14 @@ IN_PROC_BROWSER_TEST_F(WallpaperManagerBrowserTest,
FilePath large_wallpaper_path =
wallpaper_manager->GetWallpaperPathForUser(kTestUser1, false);
- int index = ash::GetDefaultWallpaperIndex();
// Saves the small/large resolution wallpapers to small/large custom
// wallpaper paths.
SaveUserWallpaperData(kTestUser1,
small_wallpaper_path,
- GetWallpaperViewInfo(index, SMALL).id);
+ ash::kDefaultSmallWallpaper.idr);
SaveUserWallpaperData(kTestUser1,
large_wallpaper_path,
- GetWallpaperViewInfo(index, LARGE).id);
+ ash::kDefaultLargeWallpaper.idr);
// Saves wallpaper info to local state for user |kTestUser1|.
WallpaperInfo info = {
@@ -229,10 +226,9 @@ IN_PROC_BROWSER_TEST_F(WallpaperManagerBrowserTest,
FilePath small_wallpaper_path =
wallpaper_manager->GetWallpaperPathForUser(kTestUser1, true);
- int index = ash::GetDefaultWallpaperIndex();
SaveUserWallpaperData(kTestUser1,
small_wallpaper_path,
- GetWallpaperViewInfo(index, SMALL).id);
+ ash::kDefaultSmallWallpaper.idr);
// Saves wallpaper info to local state for user |kTestUser1|.
WallpaperInfo info = {

Powered by Google App Engine
This is Rietveld 408576698