New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 769665 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Aug 8
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

Make message center not load notifier list every time a user opens it

Project Member Reported by tetsui@chromium.org, Sep 28 2017

Issue description

MessageCenterView creates NotifierSettingsView every time it is initialized, which means notifier list loading is triggered every time when user opens message center.

Notifier list loading is heavy operation which on some devices takes several hundreds of milliseconds.

I remember this is first found by yoshiki@. (Please let me know if this is no longer the case.)
 

Comment 1 by tetsui@chromium.org, Oct 26 2017

Labels: -M-63 M-64

Comment 2 by tetsui@chromium.org, Dec 20 2017

Status: WontFix (was: Assigned)
IIRC this is now done in an asynchronous way so no longer a problem.

https://cs.chromium.org/chromium/src/ash/message_center/notifier_settings_view.cc?l=544&rcl=25bbd5e7ab85c8c615ea9d2232549b01f932004f

Comment 3 by tetsui@chromium.org, Dec 21 2017

Status: Started (was: WontFix)
#2 was my misunderstanding. NotifierButton::Layout consumes significant time when there are may notifier sources, so I think we have to delay this until notifier settings is opened.
Status: WontFix (was: Started)
UnifiedSystemTray does not have this issue

Sign in to add a comment