Update security indicator icons on Android to match spec assets.

Update security indicator icons on Android to match spec assets.

BUG=666749
Committed: https://crrev.com/c087c64c1e08c1c3cc8b529d77fbb097dc9fc103
Cr-Commit-Position: refs/heads/master@{#433387}

[lgarron, 2016-11-18 19:26:07]

The CQ bit was checked by lgarron@chromium.org to run a CQ dry run

[lgarron, 2016-11-18 19:26:49]

lgarron@chromium.org changed reviewers:
+ tedchoc@chromium.org

[lgarron, 2016-11-18 19:26:49]

tedchoc@, could you review?

[commit-bot: I haz the power, 2016-11-18 19:26:52]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Ted C, 2016-11-18 20:04:24]

On 2016/11/18 19:26:49, lgarron wrote:
> tedchoc@, could you review?

lgtm w/ some potential homework

Did the image sizes change as a result of this?  Or is mdpi still 24x24.

Prior to this change, the icons were the same sizes per density and I'd
vote we try to keep the image size the same to avoid any unexpected quirks.

Also did you run these through the image compression steps from
chrome/app/theme/README?

=================

Please run src/tools/resources/optimize-png-files.sh on all new icons. For
example:
  tools/resources/optimize-png-files.sh -o2 new_pngs_dir

If this script does not work for some reason, at least pngcrush the files:
  mkdir crushed
  pngcrush -d crushed -brute -reduce -rem alla new/*.png

[lgarron, 2016-11-18 20:12:01]

On 2016/11/18 at 20:04:24, tedchoc wrote:
> On 2016/11/18 19:26:49, lgarron wrote:
> > tedchoc@, could you review?
> 
> lgtm w/ some potential homework
> 
> Did the image sizes change as a result of this?  Or is mdpi still 24x24.
> 
> Prior to this change, the icons were the same sizes per density and I'd
> vote we try to keep the image size the same to avoid any unexpected quirks.
> 
> Also did you run these through the image compression steps from
> chrome/app/theme/README?
> 
> =================
> 
> Please run src/tools/resources/optimize-png-files.sh on all new icons. For
example:
>   tools/resources/optimize-png-files.sh -o2 new_pngs_dir
> 
> If this script does not work for some reason, at least pngcrush the files:
>   mkdir crushed
>   pngcrush -d crushed -brute -reduce -rem alla new/*.png

Argh, sometimes I forget that the designers don't send us compressed assets.
I've compressed them in  patch 2.

The image sizes should all be the same; the icons have been sized and place
within the same dimensions.

[lgarron, 2016-11-19 02:35:09]

The CQ bit was checked by lgarron@chromium.org

[lgarron, 2016-11-19 02:35:09]

The patchset sent to the CQ was uploaded after l-g-t-m from tedchoc@chromium.org
Link to the patchset: https://codereview.chromium.org/2519533002/#ps20001
(title: "Optimize images.")

[commit-bot: I haz the power, 2016-11-19 02:35:30]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-19 03:22:00]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-19 03:23:51]

Description was changed from

==========
Update security indicator icons on Android to match spec assets.

BUG=666749
==========

to

==========
Update security indicator icons on Android to match spec assets.

BUG=666749

Committed: https://crrev.com/c087c64c1e08c1c3cc8b529d77fbb097dc9fc103
Cr-Commit-Position: refs/heads/master@{#433387}
==========

[commit-bot: I haz the power, 2016-11-19 03:23:52]

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