New issue
Advanced search Search tips

Issue 836699 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Apr 2018
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

Screenshot notification does not shown on UnifiedSystemTray

Project Member Reported by tetsui@chromium.org, Apr 25 2018

Issue description

Chrome Version: 68.0.3407.0
OS: Chrome OS

What steps will reproduce the problem?
(1) Enable chrome://flags/#enable-system-tray-unified
(2) Open system tray from the right bottom
(3) Ctrl-F4

What is the expected result?
Screenshot notification should be shown.

What happens instead?
Screenshot notification is not shown. When the collapse button on the top right of UnifiedSystemTray is clicked, it appears.
 

Comment 1 by tetsui@chromium.org, Apr 25 2018

Summary: Screenshot notification does not shown on UnifiedSystemTray (was: Screenshot notification does not shown on UnfiiedSystemTray)
Project Member

Comment 2 by bugdroid1@chromium.org, Apr 27 2018

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

commit c6695868ec2a4879e71066998543fe01f432faf0
Author: Tetsui Ohkubo <tetsui@chromium.org>
Date: Fri Apr 27 08:31:19 2018

Fix Screenshot notification in UnifiedSystemTray.

This CL fixes Screenshot notification to be shown on UnifiedSystemTray.
Screenshot notification is the easiest example to reproduce, but it
could happen on other notifications.

MessageListView scroller's Layout() should be explicitly called before
UnifiedMessageCenterView's PreferredSizeChanged is called, as the result
of the layout affects the preferred size of UnifiedMessageCenterView.

TEST=manual
BUG= 836699 

Change-Id: I1f6c519052f8bd0561a26a40943acaffdb6e586f
Reviewed-on: https://chromium-review.googlesource.com/1027392
Reviewed-by: Yoshiki Iguchi <yoshiki@chromium.org>
Commit-Queue: Tetsui Ohkubo <tetsui@chromium.org>
Cr-Commit-Position: refs/heads/master@{#554338}
[modify] https://crrev.com/c6695868ec2a4879e71066998543fe01f432faf0/ash/system/unified/unified_message_center_view.cc

Comment 3 by tetsui@chromium.org, Apr 27 2018

Status: Fixed (was: Started)

Sign in to add a comment