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

Issue 773903 link

Starred by 1 user

Issue metadata

Status: Started
Owner:
Last visit > 30 days ago
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Opening message center takes much time

Project Member Reported by yoshiki@chromium.org, Oct 12 2017

Issue description

Chrome Version: (copy from chrome://version)
OS: (e.g. Win7, OSX 10.9.5, etc...)

What steps will reproduce the problem?
(1) Install many apps (eg. my env has 40+ apps).
(2) Show the message center by clicking the tray

What is the expected result?
Message center should open instantly.

What happens instead?
Opening message center takes 900ms.

Please use labels and text to provide additional information.


For graphics-related bugs, please copy/paste the contents of the about:gpu
page at the end of this report.

 
Screenshot 2017-10-12 at 08.56.58.png
87.6 KB View Download

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

Is this related to  https://crbug.com/769665  ? I have a fix https://crrev.com/c/691616 but pending because I don't have an environment to measure the actual number.

Comment 2 by tetsui@chromium.org, Oct 12 2017

My fix is straightforward so feel free to take the item. Thank you for doing this!
Project Member

Comment 3 by bugdroid1@chromium.org, Oct 12 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/a359f4181b68c0aaa61d572da58617021d44ea2a

commit a359f4181b68c0aaa61d572da58617021d44ea2a
Author: yoshiki iguchi <yoshiki@chromium.org>
Date: Thu Oct 12 09:51:16 2017

EntryView::CalculatePreferredSize() returns the fixed value

This method returns always the same value and we don't need to calculate.

Bug: 773903
Change-Id: Icd8c4d7f4673bfdfc3cb38ef459a6bf7ce7a73e6
Reviewed-on: https://chromium-review.googlesource.com/714497
Reviewed-by: Naoki Fukino <fukino@chromium.org>
Commit-Queue: Yoshiki Iguchi <yoshiki@chromium.org>
Cr-Commit-Position: refs/heads/master@{#508291}
[modify] https://crrev.com/a359f4181b68c0aaa61d572da58617021d44ea2a/ash/message_center/notifier_settings_view.cc

Components: UI>Shell>Notifications

Sign in to add a comment