Non-Regression : Screenshot Notification is still seen at 'Notifications' even after clicking on screenshot |
|||||
Issue descriptionChrome Version: 64.0.3282.144/10176.68.0 Stable channel Daisy,Reks and Kip OS: Chrome What steps will reproduce the problem? (1)Sign into user -> Press 'Ctrl+F5(Overview Mode Button)' for screenshot ->Notification is seen -> wait until Notification vanishes (2)Now click on Notifications Icon -> Again Notification is seen and expand it (3)Click on screenshot -> Files App gets opened -> observe Screenshot Notification is still seen at 'Notifications' even on clicking it (Please refer Video) Note: 1.Save any Image(or Download any video or Save any Page) -> click on Notifications Icon -> Notification is seen and expand it -> Click on Image and FilesApp gets opened -> observe Notification is not seen at 'Notifications' after clicking on it (Please refer 'Expected_NotificationBehavior' for Video) 2. Issue is seen on M-65 and on latest M-66 also Expected: Screenshot Notification should not be seen at 'Notifications' after clicking on it Actual: Instead Screenshot Notification is still seen at 'Notifications' even on clicking it This is Non-Regression issue seen from M-62 @yoshiki : Please confirm the Issue
,
Feb 6 2018
,
Feb 8 2018
,
Feb 8 2018
,
Feb 13 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/f3ed6ebc63ac8a78efc02fca37459ba99e44fa1b commit f3ed6ebc63ac8a78efc02fca37459ba99e44fa1b Author: Megumi Hattori <megumihattori@google.com> Date: Tue Feb 13 06:23:01 2018 Close screenshot notification after it is clicked. When the screenshot notification was clicked, a folder which had the screenshot item got opened, but the notification was still seen. This CL closes the screenshot notification after it is clicked and the folder got open. BUG= 808974 Change-Id: Ie285b368e14b8e2809113a48bf9f54524a842c43 Reviewed-on: https://chromium-review.googlesource.com/910476 Commit-Queue: Megumi Hattori <megumihattori@google.com> Commit-Queue: Tetsui Ohkubo <tetsui@chromium.org> Reviewed-by: Steven Bennetts <stevenjb@chromium.org> Reviewed-by: Tetsui Ohkubo <tetsui@chromium.org> Cr-Commit-Position: refs/heads/master@{#536269} [modify] https://crrev.com/f3ed6ebc63ac8a78efc02fca37459ba99e44fa1b/chrome/browser/ui/ash/chrome_screenshot_grabber.cc
,
Feb 13 2018
|
|||||
►
Sign in to add a comment |
|||||
Comment 1 by mmanchala@chromium.org
, Feb 5 2018