Refactor SelectFaviconFrames to make it easy to use SelectFaviconFrameson objects other than bitmaps

Refactor SelectFaviconFrames to make it easy to use SelectFaviconFrames on objects other than SkBitmaps.
Changed the selection and score calculation code to operate on std::vector<gfx::Size> instead of an std::vector<SkBitmap>.

Bug=10802066
Test=SelectFaviconFramesTest.* pass
TBR=thakis

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

[pkotwicz, 2012-08-27 20:36:50]

Refactor SelectFaviconFrames to make it easy to use SelectFaviconFrames on
objects other than SkBitmaps.
Changed the selection and score calculation code to operate on
std::vector<gfx::Size> instead of std::vector<SkBitmap>.

My goal in this patch is to allow the history service to use
SelectFaviconFrames, but on non bitmap data it receives from the thumbnail
database. Take a look at http://codereview.chromium.org/10802066/ to see how I
use it.

I am hoping to move select_favicon_frames to history/ but will do that in a
separate patch.

[pkotwicz, 2012-08-27 20:38:38]

Codereview moved from https://chromiumcodereview.appspot.com/10883056/

[commit-bot: I haz the power, 2012-08-27 20:40:05]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/pkotwicz@chromium.org/10872093/1

[commit-bot: I haz the power, 2012-08-27 21:10:35]

Try job failure for 10872093-1 (retry) on android for steps "compile, build"
(clobber build).
It's a second try, previously, steps "compile, build" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android&nu...

[commit-bot: I haz the power, 2012-08-27 21:49:32]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/pkotwicz@chromium.org/10872093/1003

[commit-bot: I haz the power, 2012-08-28 00:11:17]

Change committed as 153590