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

Issue 820400 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

Non-Regression: Notification and notification popup both are seen at a time when shelf position is changed

Project Member Reported by kebalaji@chromium.org, Mar 9 2018

Issue description

Chrome Version:67.0.3365.0/10472.0.0 dev channel Minnie,Gnawty,Celes
OS:Chrome OS

What steps will reproduce the problem?
(1)Sign-in to user>> Perform some operation to get Notification Pop-up (Eg: Screen rotation)
(2)Now change the shelf position to Left or right and click on Notifications icon and observe

Actual: Notification and notification popup both are seen( it's seen only for first time)
Expected: When Notification is opened notification popup should disappear

This is a Non-Regression issue seen from M65

NOTE: Issue is seen in  M66 also
      In M64, When we click on notification icon, notification pop up gets closed automatically

@Yoshiki: Please confirm the issue
 
Expected2Notifications.webm
13.9 MB View Download
Actual2Notifications.webm
18.8 MB Download
Labels: Not-Touch-Friendly-Launcher
Project Member

Comment 3 by bugdroid1@chromium.org, Mar 14 2018

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

commit 9ef10a9b61d4f68d4dfb5fbaec5f057de218df03
Author: yoshiki iguchi <yoshiki@chromium.org>
Date: Wed Mar 14 12:36:53 2018

Rebuilt popup bubbles correctly after the shelf alignment is changed

Previously popup bubbles remain with insufficient state after the shelf
position is changed. This CL adds the code to explicitly recreate bubbles
so that it works correctly even after the shelf position is updated.

Bug:  820400 
Test: manual test (checked the repro steps in the issue)
Change-Id: I7c196614c2a148eda53650cae145b544af9bbded
Reviewed-on: https://chromium-review.googlesource.com/960681
Reviewed-by: Tetsui Ohkubo <tetsui@chromium.org>
Commit-Queue: Yoshiki Iguchi <yoshiki@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543062}
[modify] https://crrev.com/9ef10a9b61d4f68d4dfb5fbaec5f057de218df03/ash/system/web_notification/web_notification_tray.cc

Status: Started (was: Assigned)
Status: Fixed (was: Started)

Sign in to add a comment