Fix the crash in DesktopBackgroundView::OnPaint

chrome/browser/chromeos/login/wallpaper_manager.cc

 // Copyright (c) 2012 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 <vector>
 
 #include "ash/desktop_background/desktop_background_controller.h"
 #include "ash/desktop_background/desktop_background_resources.h"
@@ skipping 449 matching lines @@
   // If decoded wallpaper is empty, we are probably failed to decode the file.
   // Use default wallpaper in this case.
   if (wallpaper.image().isNull()) {
     ash::Shell::GetInstance()->desktop_background_controller()->
         SetDefaultWallpaper(ash::GetDefaultWallpaperIndex());
     return;
   }
 
   bool is_persistent = ShouldPersistDataForUser(username);
 
+  wallpaper.image().EnsureRepsForSupportedScaleFactors();
+  scoped_ptr<gfx::ImageSkia> deep_copy(wallpaper.image().DeepCopy());
+
+  WallpaperInfo wallpaper_info = {
+      wallpaper_path,
+      layout,
+      type,
+      // Date field is not used.
+      base::Time::Now().LocalMidnight()
+  };
+  // TODO(bshe): This may break if RawImage becomes RefCountedMemory.
   BrowserThread::PostTask(
-      BrowserThread::FILE,
-      FROM_HERE,
-      base::Bind(&WallpaperManager::GenerateUserWallpaperThumbnail,
-                 base::Unretained(this), username, type, delegate,
-                 wallpaper.image()));
-
-  if (is_persistent) {
-    BrowserThread::PostTask(
         BrowserThread::FILE,
         FROM_HERE,
-        base::Bind(&WallpaperManager::SaveCustomWallpaper,
+        base::Bind(&WallpaperManager::ProcessCustomWallpaper,
                    base::Unretained(this),
                    username,
-                   FilePath(wallpaper_path),
-                   layout,
-                   wallpaper));
-  }
-
+                   is_persistent,
+                   wallpaper_info,
+                   delegate,
+                   base::Passed(&deep_copy),
+                   wallpaper.raw_image()));
   ash::Shell::GetInstance()->desktop_background_controller()->
       SetCustomWallpaper(wallpaper.image(), layout);
 
   // User's custom wallpaper path is determined by username/email and the
   // appropriate wallpaper resolution in GetCustomWallpaperInternal. So use
   // DUMMY as file name here.
   WallpaperInfo info = {
       "DUMMY",
       layout,
       User::CUSTOMIZED,
@@ skipping 598 matching lines @@
   // Only cache user wallpaper at login screen.
   if (!UserManager::Get()->IsUserLoggedIn()) {
     wallpaper_cache_.insert(std::make_pair(email, wallpaper.image()));
   }
   if (update_wallpaper) {
     ash::Shell::GetInstance()->desktop_background_controller()->
         SetCustomWallpaper(wallpaper.image(), layout);
   }
 }
 
+void WallpaperManager::ProcessCustomWallpaper(
+    const std::string& email,
+    bool persistent,
+    const WallpaperInfo& info,
+    base::WeakPtr<WallpaperDelegate> delegate,
+    scoped_ptr<gfx::ImageSkia> image,
+    const UserImage::RawImage& raw_image) {
+  UserImage wallpaper(*image.get(), raw_image);
+  GenerateUserWallpaperThumbnail(email, info.type, delegate, wallpaper.image());
+  if (persistent)
+    SaveCustomWallpaper(email, FilePath(info.file), info.layout, wallpaper);
+}
+
 void WallpaperManager::OnWallpaperEncoded(const FilePath& path,
     scoped_refptr<base::RefCountedBytes> data) {
   SaveWallpaperInternal(path,
                         reinterpret_cast<const char*>(data->front()),
                         data->size());
 }
 
 void WallpaperManager::SaveCustomWallpaper(const std::string& email,
                                            const FilePath& path,
                                            ash::WallpaperLayout layout,
@@ skipping 90 matching lines @@
                           FROM_HERE,
                           base::Bind(&WallpaperManager::StartLoad,
                                      base::Unretained(this),
                                      email,
                                      info,
                                      update_wallpaper,
                                      valid_path));
 }
 
 }  // chromeos