Promote Notification action icons to stable.

Promote Notification action icons to stable.

Intent to implement and ship: https://goo.gl/GXb7lo

BUG=581336
Committed: https://crrev.com/96390dca62e14a0faf254f226dc9b012e7a2e735
Cr-Commit-Position: refs/heads/master@{#376990}

[Michael van Ouwerkerk, 2016-02-19 14:56:44]

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

[commit-bot: I haz the power, 2016-02-19 14:58:30]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1714903002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1714903002/1

[Michael van Ouwerkerk, 2016-02-19 14:59:42]

mvanouwerkerk@chromium.org changed reviewers:
+ dglazkov@chromium.org, peter@chromium.org

[Michael van Ouwerkerk, 2016-02-19 14:59:43]

Peter, could you take a look as feature reviewer?

Dimitri, could you take a look as owner?

[Peter Beverloo, 2016-02-19 15:03:30]

lgtm! \o/

Two questions:
  (1) Why status=stable instead of removing the flag? Do you think there's a
risk that we might have to revert post-branch?
  (2) Can we drop the --enable-notification-action-icons command line flag?

[Michael van Ouwerkerk, 2016-02-19 15:07:35]

On 2016/02/19 15:03:30, Peter Beverloo wrote:
> lgtm! \o/
> 
> Two questions:
>   (1) Why status=stable instead of removing the flag? Do you think there's a
> risk that we might have to revert post-branch?
>   (2) Can we drop the --enable-notification-action-icons command line flag?

My plan was to delete all the crufty flag plumbing after launch.

[commit-bot: I haz the power, 2016-02-19 16:23:41]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-02-19 16:23:42]

Dry run: This issue passed the CQ dry run.

[Michael van Ouwerkerk, 2016-02-23 10:51:56]

mvanouwerkerk@chromium.org changed reviewers:
+ jochen@chromium.org
- dglazkov@chromium.org

[Michael van Ouwerkerk, 2016-02-23 10:51:57]

Jochen, could you take a look please as owner? Dimitri is OOO.

[jochen (gone - plz use gerrit), 2016-02-23 14:33:51]

lgtm

[Michael van Ouwerkerk, 2016-02-23 14:36:10]

The CQ bit was checked by mvanouwerkerk@chromium.org

[commit-bot: I haz the power, 2016-02-23 14:36:31]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1714903002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1714903002/1

[commit-bot: I haz the power, 2016-02-23 15:42:38]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-02-23 15:43:24]

Description was changed from

==========
Promote Notification action icons to stable.

Intent to implement and ship: https://goo.gl/GXb7lo

BUG=581336
==========

to

==========
Promote Notification action icons to stable.

Intent to implement and ship: https://goo.gl/GXb7lo

BUG=581336

Committed: https://crrev.com/96390dca62e14a0faf254f226dc9b012e7a2e735
Cr-Commit-Position: refs/heads/master@{#376990}
==========

[commit-bot: I haz the power, 2016-02-23 15:43:24]

Patchset 1 (id:??) landed as
https://crrev.com/96390dca62e14a0faf254f226dc9b012e7a2e735
Cr-Commit-Position: refs/heads/master@{#376990}