Issue metadata
Sign in to add a comment
|
Regression: Notification count is seen but Notification is missing in OOBE screen |
||||||||||||||||||||||
Issue descriptionChrome Version: 72.0.3612.0/11285.0.0 dev channel Blaze, Cyan & Gnawty OS: Chrome OS What steps will reproduce the problem? (1)Recover build>> In Connect to Network overlay, click on any WiFi network and give some random long password and click on 'Connect', observe Error notification is seen with Notification count in ubertray. (2)Now click on on the Error notification and close it>> Observe Notification count is seen and notification is missing Actual: Notification count is seen but Notification is missing in OOBE screen Expected: Both Notification count and Notification should be seen This is a Regression issue as same is working fine in M71 beta Attaching screencasts for reference..
,
Dec 1
,
Dec 4
Current behavior of UnifiedMessageCenterView should be correct. To match with the old NotificationTray behavior (~M69), we should hide the counter on the shelf in OOBE.
,
Dec 5
,
Dec 5
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/d9d05c40612fa2b2ffb6395891a8ff3866fdb926 commit d9d05c40612fa2b2ffb6395891a8ff3866fdb926 Author: Tetsui Ohkubo <tetsui@chromium.org> Date: Wed Dec 05 06:20:20 2018 Unified: Hide notification counter in OOBE. We should hide the notification counter in OOBE, because UnifiedMessageCenterView is hidden in such cases, and it's confusing to have the counter still visible. TEST=manual BUG= 907817 Change-Id: Ic81ab346fd993ced9f0ef10220abbc170dde0bbd Reviewed-on: https://chromium-review.googlesource.com/c/1362693 Reviewed-by: Yoshiki Iguchi <yoshiki@chromium.org> Commit-Queue: Tetsui Ohkubo <tetsui@chromium.org> Cr-Commit-Position: refs/heads/master@{#613883} [modify] https://crrev.com/d9d05c40612fa2b2ffb6395891a8ff3866fdb926/ash/system/unified/notification_counter_view.cc
,
Dec 5
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by yoshiki@google.com
, Nov 22