Clean up media notification icons

Clean up media notification icons

* Remove unused black notification icon assets for SDK v21,
  since all clients use NotificationCompat.
* Rename file names from ic_vidcontrol_* to ic_media_control_*,
  since "vidcontrol" stands for "video_control" but we also
  have audio.

BUG=656574
TEST=Manual verified on SDK v21

Committed: https://crrev.com/b4bf56769b5a7516544f5f5e956e71f2581ded3a
Cr-Commit-Position: refs/heads/master@{#428334}

[Zhiqiang Zhang (Slow), 2016-10-27 18:22:51]

Description was changed from

==========
Clean up media notification icons

* Remove unused black notification icon assets for SDK v21.
* Rename file names from ic_vidcontrol_* to ic_media_control_*

BUG=656574
==========

to

==========
Clean up media notification icons

* Remove unused black notification icon assets for SDK v21,
  since all clients use NotificationCompat.
* Rename file names from ic_vidcontrol_* to ic_media_control_*,
  since "vidcontrol" stands for "video_control" but we also
  have audio.

BUG=656574
==========

[Zhiqiang Zhang (Slow), 2016-10-27 18:23:46]

Description was changed from

==========
Clean up media notification icons

* Remove unused black notification icon assets for SDK v21,
  since all clients use NotificationCompat.
* Rename file names from ic_vidcontrol_* to ic_media_control_*,
  since "vidcontrol" stands for "video_control" but we also
  have audio.

BUG=656574
==========

to

==========
Clean up media notification icons

* Remove unused black notification icon assets for SDK v21,
  since all clients use NotificationCompat.
* Rename file names from ic_vidcontrol_* to ic_media_control_*,
  since "vidcontrol" stands for "video_control" but we also
  have audio.

BUG=656574
TEST=Manual verified on SDK v21
==========

[Zhiqiang Zhang (Slow), 2016-10-27 18:34:30]

zqzhang@chromium.org changed reviewers:
+ mlamouri@chromium.org, twellington@chromium.org

[Zhiqiang Zhang (Slow), 2016-10-27 18:34:31]

PTAL.

Tested on a SDK v21 device removing the black icons. We should be good to go.

[Theresa, 2016-10-27 19:40:38]

lgtm

[mlamouri (slow - plz ping), 2016-10-28 09:42:35]

lgtm. If you didn't, can you run ./tools/resources/optimize-png-files.sh on the
files you are adding?

[Zhiqiang Zhang (Slow), 2016-10-28 10:40:26]

On 2016/10/28 09:42:35, mlamouri wrote:
> lgtm. If you didn't, can you run ./tools/resources/optimize-png-files.sh on
the
> files you are adding?

Already done :)

[Zhiqiang Zhang (Slow), 2016-10-28 10:41:26]

zqzhang@chromium.org changed reviewers:
+ bauerb@chromium.org

[Zhiqiang Zhang (Slow), 2016-10-28 10:41:26]

+bauerb to rummberstamp:
MediaCaptureNotificationService.java
DownloadNotificationService.java

[Bernhard Bauer, 2016-10-28 10:50:48]

lgtm

[Zhiqiang Zhang (Slow), 2016-10-28 11:08:27]

The CQ bit was checked by zqzhang@chromium.org

[Zhiqiang Zhang (Slow), 2016-10-28 11:08:27]

The patchset sent to the CQ was uploaded after l-g-t-m from
twellington@chromium.org, mlamouri@chromium.org
Link to the patchset: https://codereview.chromium.org/2449183005/#ps20001
(title: "using standard asset set")

[commit-bot: I haz the power, 2016-10-28 11:08:44]

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-10-28 11:37:35]

Description was changed from

==========
Clean up media notification icons

* Remove unused black notification icon assets for SDK v21,
  since all clients use NotificationCompat.
* Rename file names from ic_vidcontrol_* to ic_media_control_*,
  since "vidcontrol" stands for "video_control" but we also
  have audio.

BUG=656574
TEST=Manual verified on SDK v21
==========

to

==========
Clean up media notification icons

* Remove unused black notification icon assets for SDK v21,
  since all clients use NotificationCompat.
* Rename file names from ic_vidcontrol_* to ic_media_control_*,
  since "vidcontrol" stands for "video_control" but we also
  have audio.

BUG=656574
TEST=Manual verified on SDK v21
==========

[commit-bot: I haz the power, 2016-10-28 11:37:36]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-28 11:39:05]

Description was changed from

==========
Clean up media notification icons

* Remove unused black notification icon assets for SDK v21,
  since all clients use NotificationCompat.
* Rename file names from ic_vidcontrol_* to ic_media_control_*,
  since "vidcontrol" stands for "video_control" but we also
  have audio.

BUG=656574
TEST=Manual verified on SDK v21
==========

to

==========
Clean up media notification icons

* Remove unused black notification icon assets for SDK v21,
  since all clients use NotificationCompat.
* Rename file names from ic_vidcontrol_* to ic_media_control_*,
  since "vidcontrol" stands for "video_control" but we also
  have audio.

BUG=656574
TEST=Manual verified on SDK v21

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

[commit-bot: I haz the power, 2016-10-28 11:39:05]

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