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}
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)
Description was changed from ========== Clean up media notification icons * Remove unused black notification ...
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
==========
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 :)
Description was changed from ========== Clean up media notification icons * Remove unused black notification ...
4 years, 1 month ago
(2016-10-28 11:37:35 UTC)
#14
Message was sent while issue was closed.
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
Committed patchset #2 (id:20001)
4 years, 1 month ago
(2016-10-28 11:37:36 UTC)
#15
Message was sent while issue was closed.
Committed patchset #2 (id:20001)
commit-bot: I haz the power
Description was changed from ========== Clean up media notification icons * Remove unused black notification ...
4 years, 1 month ago
(2016-10-28 11:39:05 UTC)
#16
Message was sent while issue was closed.
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
Patchset 2 (id:??) landed as https://crrev.com/b4bf56769b5a7516544f5f5e956e71f2581ded3a Cr-Commit-Position: refs/heads/master@{#428334}
4 years, 1 month ago
(2016-10-28 11:39:05 UTC)
#17
Issue 2449183005: Clean up media notification icons
(Closed)
Created 4 years, 1 month ago by Zhiqiang Zhang (Slow)
Modified 4 years, 1 month ago
Reviewers: mlamouri (slow - plz ping), Theresa, Bernhard Bauer
Base URL:
Comments: 0