Show small notification icons in notification tray (reland)

Show small notification icons in notification tray (reland)

This patch is a reland of crrev.com/411019 with fixing the memory leak by moving the place of AddChild(count_label_view).

Original Description:

This patch shows small notification icons (up to 3) of the
current notifications in the notification tray.

This also makes the counter show the count of all the
notification excluding the visible small icons (previously
it was the unread count).

See the issue for screenshots.

BUG=633399
TEST=manual test (with and without MD mode)
R=xiyuan@chromuim.org
TBR=oshima@chromium.org

Committed: https://crrev.com/79c70981ab68a0e4667493ab05a411d4c1f0dd24
Cr-Commit-Position: refs/heads/master@{#411418}

[yoshiki, 2016-08-11 18:20:01]

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

[commit-bot: I haz the power, 2016-08-11 18:20:30]

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

[yoshiki, 2016-08-11 18:26:03]

Description was changed from

==========
.

BUG=
==========

to

==========
Show small notification icons in notification tray (reland)

This patch is a reland of crrev.com/411019 with fixing the memory leak by moving
the place of AddChild(count_label_view).

Original Description:

This patch shows small notification icons (up to 3) of the
current notifications in the notification tray.

This also makes the counter show the count of all the
notification excluding the visible small icons (previously
it was the unread count).

See the issue for screenshots.

BUG=633399
TEST=manual test (with and without MD mode)
R=xiyuan@chromuim.org
TBR=oshima@chromium.org
==========

[yoshiki, 2016-08-11 18:26:03]

yoshiki@chromium.org changed reviewers:
+ xiyuan@chromium.org

[yoshiki, 2016-08-11 18:27:04]

Xiyuan, PTAL. The difference from the original patch is only the place of
AddChild(counter_label_view). I confirmed that this passes the asan bot.

[commit-bot: I haz the power, 2016-08-11 19:04:02]

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

[commit-bot: I haz the power, 2016-08-11 19:04:03]

Dry run: This issue passed the CQ dry run.

[xiyuan, 2016-08-11 20:02:32]

lgtm

[yoshiki, 2016-08-11 20:53:45]

The CQ bit was checked by yoshiki@chromium.org

[commit-bot: I haz the power, 2016-08-11 20:54:31]

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-08-11 20:59:56]

Description was changed from

==========
Show small notification icons in notification tray (reland)

This patch is a reland of crrev.com/411019 with fixing the memory leak by moving
the place of AddChild(count_label_view).

Original Description:

This patch shows small notification icons (up to 3) of the
current notifications in the notification tray.

This also makes the counter show the count of all the
notification excluding the visible small icons (previously
it was the unread count).

See the issue for screenshots.

BUG=633399
TEST=manual test (with and without MD mode)
R=xiyuan@chromuim.org
TBR=oshima@chromium.org
==========

to

==========
Show small notification icons in notification tray (reland)

This patch is a reland of crrev.com/411019 with fixing the memory leak by moving
the place of AddChild(count_label_view).

Original Description:

This patch shows small notification icons (up to 3) of the
current notifications in the notification tray.

This also makes the counter show the count of all the
notification excluding the visible small icons (previously
it was the unread count).

See the issue for screenshots.

BUG=633399
TEST=manual test (with and without MD mode)
R=xiyuan@chromuim.org
TBR=oshima@chromium.org
==========

[commit-bot: I haz the power, 2016-08-11 20:59:57]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-11 21:01:22]

Description was changed from

==========
Show small notification icons in notification tray (reland)

This patch is a reland of crrev.com/411019 with fixing the memory leak by moving
the place of AddChild(count_label_view).

Original Description:

This patch shows small notification icons (up to 3) of the
current notifications in the notification tray.

This also makes the counter show the count of all the
notification excluding the visible small icons (previously
it was the unread count).

See the issue for screenshots.

BUG=633399
TEST=manual test (with and without MD mode)
R=xiyuan@chromuim.org
TBR=oshima@chromium.org
==========

to

==========
Show small notification icons in notification tray (reland)

This patch is a reland of crrev.com/411019 with fixing the memory leak by moving
the place of AddChild(count_label_view).

Original Description:

This patch shows small notification icons (up to 3) of the
current notifications in the notification tray.

This also makes the counter show the count of all the
notification excluding the visible small icons (previously
it was the unread count).

See the issue for screenshots.

BUG=633399
TEST=manual test (with and without MD mode)
R=xiyuan@chromuim.org
TBR=oshima@chromium.org

Committed: https://crrev.com/79c70981ab68a0e4667493ab05a411d4c1f0dd24
Cr-Commit-Position: refs/heads/master@{#411418}
==========

[commit-bot: I haz the power, 2016-08-11 21:01:23]

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