ARC Notification shows blank in message center after popup opens |
||||||||
Issue descriptionChrome Version: (copy from chrome://version) OS: (e.g. Win7, OSX 10.9.5, etc...) What steps will reproduce the problem? (1) See a popup of ARC notification (2) While the popup is visible, open the message center What is the expected result? See the content in the notification in the message center What happens instead? See the blank in the notification in the message center 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.
,
Aug 2 2017
This is a regression of https://chromium.googlesource.com/chromium/src/+/32fc87a3e1ba26f1a4038c9583401e14d88e1b49.
,
Aug 3 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/e0d6b809e63bdc2ff0e06a679cd2b37b02ab1d19 commit e0d6b809e63bdc2ff0e06a679cd2b37b02ab1d19 Author: yoshiki iguchi <yoshiki@chromium.org> Date: Thu Aug 03 02:12:38 2017 Detach surface only when the surface is attaching to the current view Previously, a notification content view wrongly detached the surface which is attaching to another view. That caused crbug.com/751355 . This CL prevents that. Bug: 751355 Change-Id: Ibaec4d7fd88f96e864634258d16e45fac9d2a945 Reviewed-on: https://chromium-review.googlesource.com/595312 Commit-Queue: Yoshiki Iguchi <yoshiki@chromium.org> Reviewed-by: Yuichiro Hanada <yhanada@chromium.org> Cr-Commit-Position: refs/heads/master@{#491613} [modify] https://crrev.com/e0d6b809e63bdc2ff0e06a679cd2b37b02ab1d19/ui/arc/notification/arc_notification_content_view.cc [modify] https://crrev.com/e0d6b809e63bdc2ff0e06a679cd2b37b02ab1d19/ui/arc/notification/arc_notification_content_view_unittest.cc
,
Aug 3 2017
,
Aug 10 2017
I forgot merging. I confirmed it's working on 62.0.3174.0 canary
,
Aug 10 2017
This bug requires manual review: M61 has already been promoted to the beta branch, so this requires manual review Please contact the milestone owner if you have questions. Owners: amineer@(Android), cmasso@(iOS), ketakid@(ChromeOS), govind@(Desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Aug 10 2017
Approving merge to M61 Chrome OS.
,
Aug 10 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/1765a2a95967802c4ad45f3129deeaaec957f66d commit 1765a2a95967802c4ad45f3129deeaaec957f66d Author: yoshiki iguchi <yoshiki@chromium.org> Date: Thu Aug 10 17:23:27 2017 Detach surface only when the surface is attaching to the current view Previously, a notification content view wrongly detached the surface which is attaching to another view. That caused crbug.com/751355 . This CL prevents that. TBR=yoshiki@chromium.org (cherry picked from commit e0d6b809e63bdc2ff0e06a679cd2b37b02ab1d19) Bug: 751355 Change-Id: Ibaec4d7fd88f96e864634258d16e45fac9d2a945 Reviewed-on: https://chromium-review.googlesource.com/595312 Commit-Queue: Yoshiki Iguchi <yoshiki@chromium.org> Reviewed-by: Yuichiro Hanada <yhanada@chromium.org> Cr-Original-Commit-Position: refs/heads/master@{#491613} Reviewed-on: https://chromium-review.googlesource.com/610641 Reviewed-by: Yoshiki Iguchi <yoshiki@chromium.org> Cr-Commit-Position: refs/branch-heads/3163@{#435} Cr-Branched-From: ff259bab28b35d242e10186cd63af7ed404fae0d-refs/heads/master@{#488528} [modify] https://crrev.com/1765a2a95967802c4ad45f3129deeaaec957f66d/ui/arc/notification/arc_notification_content_view.cc [modify] https://crrev.com/1765a2a95967802c4ad45f3129deeaaec957f66d/ui/arc/notification/arc_notification_content_view_unittest.cc
,
Aug 10 2017
,
Jan 22 2018
|
||||||||
►
Sign in to add a comment |
||||||||
Comment 1 by yoshiki@chromium.org
, Aug 2 2017