Refactor the wallpaper encoding to another file/class.

Refactor the wallpaper encoding to another file/class.
Encoding the wallpaper to png from a skia is something we'll need to do from at least one more place (the screensaver images source) so it makes sense to refactor this code to pull out the encoder so it's usable from other places too.

R=bshe@chromium.org,csilv@chromium.org
BUG=None.
TEST=Tested with the lock screen to verify that the background set for the user is pulled and displayed.


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=148272

[rkc, 2012-07-19 22:54:25]

On 2012/07/19 22:53:55, Rahul Chaturvedi wrote:

Reviews request,
bshe - main review
csilv - owners review

[bshe, 2012-07-20 17:56:40]

On 2012/07/19 22:54:25, Rahul Chaturvedi wrote:
> On 2012/07/19 22:53:55, Rahul Chaturvedi wrote:
> 
> Reviews request,
> bshe - main review
> csilv - owners review


lgtm

this is great. wallpaper_thumbnail_source.cc also contains an encoder. We may
use the class you refactored as well.

[csilv, 2012-07-23 21:35:18]

lgtm w/ nit.

http://codereview.chromium.org/10790084/diff/1/chrome/browser/ui/webui/option...
File chrome/browser/ui/webui/options2/chromeos/wallpaper_source.cc (right):

http://codereview.chromium.org/10790084/diff/1/chrome/browser/ui/webui/option...
chrome/browser/ui/webui/options2/chromeos/wallpaper_source.cc:17: extern "C" {
Looks like we don't need to include zlib.h anymore.

[rkc, 2012-07-24 23:01:44]

http://codereview.chromium.org/10790084/diff/1/chrome/browser/ui/webui/option...
File chrome/browser/ui/webui/options2/chromeos/wallpaper_source.cc (right):

http://codereview.chromium.org/10790084/diff/1/chrome/browser/ui/webui/option...
chrome/browser/ui/webui/options2/chromeos/wallpaper_source.cc:17: extern "C" {
On 2012/07/23 21:35:18, csilv wrote:
> Looks like we don't need to include zlib.h anymore.

Done.

[commit-bot: I haz the power, 2012-07-24 23:01:53]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rkc@chromium.org/10790084/5001

[commit-bot: I haz the power, 2012-07-25 01:24:10]

Change committed as 148272