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

Issue 847732 link

Starred by 1 user
Status: Fixed
Owner:
tetsui@chromium.org
Closed: Jun 2018
Cc:
omrilio@chromium.org
yamaguchi@chromium.org
█ yoshiki@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Opening the unified system tray causes the Chrome's top bar to flicker

Project Member Reported by satorux@chromium.org, May 30 2018 
Chrome Version       : 69.0.3442.0
OS Version: 10726.0.0

What steps will reproduce the problem?
1. Enable the unified system tray
2. Click on the unified system tray

What is the expected result?

The unified system tray is opened.


What happens instead of that?

The unified system tray is opened, and the Chrome's top bar flickers, as shown in the attached video.


Please provide any additional information below. Attach a screenshot if
possible.
flickering.webm
3.0 MB View Download

Comment 1 by satorux@chromium.org, May 30 2018

Cc: yamaguchi@chromium.org
Still reproducible with a newer canary image:

Chrome Version       : 69.0.3443.0
OS Version: 10729.0.0

Comment 2 by tetsui@chromium.org, May 31 2018

Status: Started (was: Assigned)
Project Member

Comment 3 by bugdroid1@chromium.org, May 31 2018 

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

commit d5d446edbf3ab83b025a32ec86985fdfd3cc619d
Author: Tetsui Ohkubo <tetsui@chromium.org>
Date: Thu May 31 15:31:08 2018

Do not activate UnifiedSystemTray by default.

This CL stops activating UnifiedSystemTray bubble by default, which is
the same behavior as old SystemTray. At the beginning of the
implementation it took that approach for simplicity, but it was
planned to stop this. Activating system tray by default also leads to
some artifact such as flickering.

TEST=manual
BUG= 847732 

Change-Id: I4112492bc8b8783667ce111220aab36bbd7eeac2
Reviewed-on: https://chromium-review.googlesource.com/1080476
Reviewed-by: James Cook <jamescook@chromium.org>
Commit-Queue: Tetsui Ohkubo <tetsui@chromium.org>
Cr-Commit-Position: refs/heads/master@{#563246}
[modify] https://crrev.com/d5d446edbf3ab83b025a32ec86985fdfd3cc619d/ash/accelerators/accelerator_controller.cc
[modify] https://crrev.com/d5d446edbf3ab83b025a32ec86985fdfd3cc619d/ash/system/unified/unified_system_tray.cc
[modify] https://crrev.com/d5d446edbf3ab83b025a32ec86985fdfd3cc619d/ash/system/unified/unified_system_tray.h
[modify] https://crrev.com/d5d446edbf3ab83b025a32ec86985fdfd3cc619d/ash/system/unified/unified_system_tray_bubble.cc
[modify] https://crrev.com/d5d446edbf3ab83b025a32ec86985fdfd3cc619d/ash/system/unified/unified_system_tray_bubble.h

Comment 4 by tetsui@chromium.org, Jun 1 2018

Status: Fixed (was: Started)
►

Sign in to add a comment