Implement new bookmark app icon generation.

Implement new bookmark app icon generation.

This CL changes the generation of bookmark apps to create an icon that
consists of the favicon's dominant color as a background with the first
letter of the app's domain painted over it.

The letter is either black or white depending on the luminance of the
background.

This CL also turns on generation of 2x icons for 32 and 48 px icons.

BUG=315632
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=269745

[benwells, 2014-04-24 03:35:57]

lgtm

https://codereview.chromium.org/246333006/diff/40001/chrome/browser/extension...
File chrome/browser/extensions/bookmark_app_helper.cc (right):

https://codereview.chromium.org/246333006/diff/40001/chrome/browser/extension...
chrome/browser/extensions/bookmark_app_helper.cc:240: // Also generate the 2x
resource for this size.
This seems to be new to this patch. Can you call this out in the description?

[calamity, 2014-04-24 04:28:39]

https://codereview.chromium.org/246333006/diff/40001/chrome/browser/extension...
File chrome/browser/extensions/bookmark_app_helper.cc (right):

https://codereview.chromium.org/246333006/diff/40001/chrome/browser/extension...
chrome/browser/extensions/bookmark_app_helper.cc:240: // Also generate the 2x
resource for this size.
On 2014/04/24 03:35:58, benwells wrote:
> This seems to be new to this patch. Can you call this out in the description?

Done.

[calamity, 2014-04-24 04:32:32]

The CQ bit was checked by calamity@chromium.org

[commit-bot: I haz the power, 2014-04-24 05:02:10]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/calamity@chromium.org/246333006/40001

[commit-bot: I haz the power, 2014-04-24 08:24:57]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-04-24 08:24:57]

Try jobs failed on following builders:
  tryserver.chromium on mac_chromium_rel

[calamity, 2014-04-29 11:04:21]

*sigh* This is failing because Mac requires a valid font name or it crashes. I
think the way forward is to use the platform specific sans serif font for
non-ChromeOS and hardcode "Noto Sans" for ChromeOS.

[calamity, 2014-05-12 01:26:18]

The CQ bit was checked by calamity@chromium.org

[commit-bot: I haz the power, 2014-05-12 01:26:21]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/calamity@chromium.org/246333006/100001

[commit-bot: I haz the power, 2014-05-12 06:21:46]

Change committed as 269745