Cleanup thumbnail database part #2

Pass in favicon sizes parameter to ThumbnailDatabase::AddFavicon().
Fix ThumbnailDatabase::GetIconMappingsForPageURL to have functionality described by the comment.

Bug=10896006
Test=Try bots

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

[pkotwicz, 2012-09-07 17:47:23]

Scott, can you please take a look?

Subpart of http://codereview.chromium.org/10917041

[sky, 2012-09-07 18:16:47]

LGTM

http://codereview.chromium.org/10915146/diff/2001/chrome/browser/history/hist...
File chrome/browser/history/history_backend_unittest.cc (right):

http://codereview.chromium.org/10915146/diff/2001/chrome/browser/history/hist...
chrome/browser/history/history_backend_unittest.cc:213:
CR_DEFINE_STATIC_LOCAL(std::vector<gfx::Size>, kSizesSmallAndLarge, ());
This is a test, there's no point in forcing a static here. Return a new vector
each time.

[commit-bot: I haz the power, 2012-09-07 22:10:56]

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

[commit-bot: I haz the power, 2012-09-08 00:29:14]

Change committed as 155540