Hook up custom wallpaper code path to new wallpaper picker.

chrome/browser/chromeos/extensions/wallpaper_private_api.h

 // 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.
 
 #ifndef CHROME_BROWSER_CHROMEOS_EXTENSIONS_WALLPAPER_PRIVATE_API_H_
 #define CHROME_BROWSER_CHROMEOS_EXTENSIONS_WALLPAPER_PRIVATE_API_H_
 
 #include "ash/desktop_background/desktop_background_resources.h"
 #include "chrome/browser/extensions/extension_function.h"
 #include "net/url_request/url_fetcher_delegate.h"
 #include "ui/gfx/image/image_skia.h"
 
 // Wallpaper manager strings.
 class WallpaperStringsFunction : public SyncExtensionFunction {
  public:
   DECLARE_EXTENSION_FUNCTION_NAME("wallpaperPrivate.getStrings");
 
  protected:
   virtual ~WallpaperStringsFunction() {}
 
   // SyncExtensionFunction overrides.
   virtual bool RunImpl() OVERRIDE;
 };
 
-class WallpaperSetWallpaperFunction : public AsyncExtensionFunction {
+class WallpaperFunctionBase : public AsyncExtensionFunction {
+ public:
+  WallpaperFunctionBase();
+
+ protected:
+  virtual ~WallpaperFunctionBase();
+
+  // AsyncExtensionFunction overrides.
+  virtual bool RunImpl() OVERRIDE;
+
+  class WallpaperDecoder;

[Ivan Korotkov, 2012/08/30 14:07:36]

Forward declarations go after includes.

[bshe, 2012/08/30 19:22:40]

Looks like this is a nested class. Not sure if I should forward declare it after
includes.

On 2012/08/30 14:07:36, Ivan Korotkov wrote:

+
+  // Holds an instance of WallpaperDecoder.
+  static WallpaperDecoder* wallpaper_decoder_;
+
+ private:
+  virtual void OnWallpaperDecoded(const gfx::ImageSkia& wallpaper) = 0;
+  virtual void OnFail() = 0;

[Ivan Korotkov, 2012/08/30 14:07:36]

OnFailure?

[bshe, 2012/08/30 19:22:40]

Done.

+};
+
+class WallpaperSetWallpaperFunction : public WallpaperFunctionBase {
  public:
   DECLARE_EXTENSION_FUNCTION_NAME("wallpaperPrivate.setWallpaper");
 
   WallpaperSetWallpaperFunction();
 
  protected:
   virtual ~WallpaperSetWallpaperFunction();
 
   // AsyncExtensionFunction overrides.
   virtual bool RunImpl() OVERRIDE;
 
  private:
-  class WallpaperDecoder;
-
-  void OnWallpaperDecoded(const gfx::ImageSkia& wallpaper);
-  void OnFail();
+  virtual void OnWallpaperDecoded(const gfx::ImageSkia& wallpaper) OVERRIDE;
+  virtual void OnFail() OVERRIDE;
 
   // Saves the image data to a file.
   void SaveToFile();
 
   // Sets wallpaper to the decoded image.
   void SetDecodedWallpaper();
 
   // Layout of the downloaded wallpaper.
   ash::WallpaperLayout layout_;
 
   // The decoded wallpaper.
   gfx::ImageSkia wallpaper_;
 
   // Email address of logged in user.
   std::string email_;
 
   // File name extracts from URL.
   std::string file_name_;
 
   // String representation of downloaded wallpaper.
   std::string image_data_;
+};
 
-  // Holds an instance of WallpaperDecoder.
-  static WallpaperDecoder* wallpaper_decoder_;
+class WallpaperSetCustomWallpaperFunction : public WallpaperFunctionBase {
+ public:
+  DECLARE_EXTENSION_FUNCTION_NAME("wallpaperPrivate.setCustomWallpaper");
+
+  WallpaperSetCustomWallpaperFunction();
+
+ protected:
+  virtual ~WallpaperSetCustomWallpaperFunction();
+
+  // AsyncExtensionFunction overrides.
+  virtual bool RunImpl() OVERRIDE;
+
+ private:
+  virtual void OnWallpaperDecoded(const gfx::ImageSkia& wallpaper) OVERRIDE;
+  virtual void OnFail() OVERRIDE;
+
+  // Layout of the downloaded wallpaper.
+  ash::WallpaperLayout layout_;
+
+  // Email address of logged in user.
+  std::string email_;
+
+  // String representation of downloaded wallpaper.
+  std::string image_data_;
 };
 
 #endif  // CHROME_BROWSER_CHROMEOS_EXTENSIONS_WALLPAPER_PRIVATE_API_H_