AppBanner: add requirement for an 144x144 image/png icon.

AppBanner: add requirement for an 144x144 image/png icon.

This requirement is only a requirement on the Manifest content. It is
then up to the ManifestIconSelector to pick the best icon which might
be a smaller one.

BUG=457399
Committed: https://crrev.com/6c608b2ef8e522462355b2dadc721c608985b322
Cr-Commit-Position: refs/heads/master@{#317362}

[mlamouri (slow - plz ping), 2015-02-20 14:02:27]

mlamouri@chromium.org changed reviewers:
+ dfalcantara@chromium.org

[mlamouri (slow - plz ping), 2015-02-20 14:02:27]

I will work on some tests later. Landing this sounds a tad more urgent ;)

[gone, 2015-02-20 17:36:41]

lgtm

https://chromiumcodereview.appspot.com/946573002/diff/1/chrome/browser/androi...
File chrome/browser/android/banners/app_banner_manager.cc (right):

https://chromiumcodereview.appspot.com/946573002/diff/1/chrome/browser/androi...
chrome/browser/android/banners/app_banner_manager.cc:61: if (size.width() >= 144
&& size.height() >= 144)
Make 144 a constant up above.

[mlamouri (slow - plz ping), 2015-02-20 18:41:28]

New patchsets have been uploaded after l-g-t-m from dfalcantara@chromium.org

[mlamouri (slow - plz ping), 2015-02-20 18:41:35]

The CQ bit was checked by mlamouri@chromium.org

[commit-bot: I haz the power, 2015-02-20 18:42:54]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/946573002/20001

[commit-bot: I haz the power, 2015-02-20 19:08:14]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-02-20 19:09:02]

Patchset 2 (id:??) landed as
https://crrev.com/6c608b2ef8e522462355b2dadc721c608985b322
Cr-Commit-Position: refs/heads/master@{#317362}

[mlamouri (slow - plz ping), 2015-02-24 12:28:43]

+cc benwells@ fyi