wrong icons parameter value in manifest.json for chrome main repo fix

This fix is related to current ticket: https://issues.adblockplus.org/ticket/156

Relative review:
http://codereview.adblockplus.org/5406295150559232

[Wladimir Palant, 2014-03-18 17:18:28]

http://codereview.adblockplus.org/5676337998069760/diff/5629499534213120/meta...
File metadata.chrome (right):

http://codereview.adblockplus.org/5676337998069760/diff/5629499534213120/meta...
metadata.chrome:17: icons = icons/abp-19.png icons/abp-48.png icons/abp-128.png
Please see http://developer.chrome.com/extensions/manifest/icons, the icon sizes
required for Chrome are 16x16, 48x48 and 128x128. So we need to have
icons/abp-16.png in Chrome and use it here.

[saroyanm, 2014-03-19 09:07:37]

New patch uploaded.

http://codereview.adblockplus.org/5676337998069760/diff/5629499534213120/meta...
File metadata.chrome (right):

http://codereview.adblockplus.org/5676337998069760/diff/5629499534213120/meta...
metadata.chrome:17: icons = icons/abp-19.png icons/abp-48.png icons/abp-128.png
On 2014/03/18 17:18:29, Wladimir Palant wrote:
> Please see http://developer.chrome.com/extensions/manifest/icons, the icon
sizes
> required for Chrome are 16x16, 48x48 and 128x128. So we need to have
> icons/abp-16.png in Chrome and use it here.

How I could be that inattentive, Thanks on pointing that.
Fixed.

[saroyanm, 2014-03-19 09:21:20]

On 2014/03/19 09:07:37, saroyanm wrote:
> New patch uploaded.
> 
>
http://codereview.adblockplus.org/5676337998069760/diff/5629499534213120/meta...
> File metadata.chrome (right):
> 
>
http://codereview.adblockplus.org/5676337998069760/diff/5629499534213120/meta...
> metadata.chrome:17: icons = icons/abp-19.png icons/abp-48.png
icons/abp-128.png
> On 2014/03/18 17:18:29, Wladimir Palant wrote:
> > Please see http://developer.chrome.com/extensions/manifest/icons, the icon
> sizes
> > required for Chrome are 16x16, 48x48 and 128x128. So we need to have
> > icons/abp-16.png in Chrome and use it here.
> 
> How I could be that inattentive, Thanks on pointing that.
> Fixed.

Sorry for being quick, 
I've uploaded another patch while I've noticed that abp-16.png now are used by
both chrome and safari builds, 
So moved to root/icons folder. To have it in one place.

[Wladimir Palant, 2014-03-19 12:20:08]

LGTM with my comment below addressed.

http://codereview.adblockplus.org/5676337998069760/diff/5717271485874176/meta...
File metadata.chrome (right):

http://codereview.adblockplus.org/5676337998069760/diff/5717271485874176/meta...
metadata.chrome:17: icons = icons/abp-16.png icons/abp-48.png icons/abp-128.png
I was about to ask whether we need abp-32.png in the icons folder. However, from
https://code.google.com/p/chromium/issues/detail?id=94301 it seems that this
icon size is also being used even though the documentation doesn't mention it.
So please add it here as well.

[saroyanm, 2014-03-20 07:05:04]

On 2014/03/19 12:20:08, Wladimir Palant wrote:
> LGTM with my comment below addressed.
> 
>
http://codereview.adblockplus.org/5676337998069760/diff/5717271485874176/meta...
> File metadata.chrome (right):
> 
>
http://codereview.adblockplus.org/5676337998069760/diff/5717271485874176/meta...
> metadata.chrome:17: icons = icons/abp-16.png icons/abp-48.png
icons/abp-128.png
> I was about to ask whether we need abp-32.png in the icons folder. However,
from
> https://code.google.com/p/chromium/issues/detail?id=94301 it seems that this
> icon size is also being used even though the documentation doesn't mention it.
> So please add it here as well.

That's quite interesting.. Thanks.
pushed.