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

Issue 814600 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2018
Cc:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

Regression : Mouse pointer Flickers on hovering mouse on the message center

Project Member Reported by yoshiki@chromium.org, Feb 22 2018

Issue description

Chrome Version: 66.0.3344 (yesterday's canary) on Eve.

What steps will reproduce the problem?
(1) Show the message center with several notifications
(2) Move the mouse cursor hardly for 10-20sec on the message center

What is the expected result?
The mouse cursor moves normally.

What happens instead?
After moving hardly for 10-20sec, the mouse cursor gets flickering, and the cursor movement gets delayed

Please use labels and text to provide additional information.

This happens on both ARC and non-ARC notifications (see the video, the download notification is non-ARC)

The attached video is 240fps (4x-slower) slo-mo.
 
IMG_132861439.MOV
15.7 MB Download
I'm attaching another video which shows the delayed cursor issue.

The mouse cursor keeps moving even after I get my finger off of the touch-pad.
IMG_71064698.MOV
6.3 MB View Download
I confirms this is still happening on the latest canary.

Google Chrome	66.0.3350.3 (Official Build) canary (64-bit)
Platform	10424.0.0 (Official Build) canary-channel eve

Comment 3 by osh...@chromium.org, Feb 22 2018

Labels: -Pri-1 Pri-2
Status: Assigned (was: Available)
Yes, this is different issue than original one. 

I don't think this is P1 as this requires me to continuously moving the cursor on notifications for several seconds though. I have some idea and I'll look into this when I have time.
Cc: tetsui@chromium.org
JFYI: "original issue" mentioned in the comment #3 was Issue 798056.
Status: Started (was: Assigned)
Project Member

Comment 7 by bugdroid1@chromium.org, Mar 10 2018

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

commit 46d3870fc70873949e34f263f6b71df7b3f3aaac
Author: Mitsuru Oshima <oshima@chromium.org>
Date: Sat Mar 10 06:21:15 2018

Use the cursor stored in Widget for regular surface as well.

The original CL crrev.com/c/903223 changed the GetCurosr logic
only for notification surface. Regular exo surface should do the same.

This also removes obsolete path.

Bug:  814600 
Test: manual. See bug for repro step.
Change-Id: Icbdf000db30efa96aa056c1b7fe15d38822b7610
Reviewed-on: https://chromium-review.googlesource.com/956103
Reviewed-by: David Reveman <reveman@chromium.org>
Commit-Queue: Mitsuru Oshima <oshima@chromium.org>
Cr-Commit-Position: refs/heads/master@{#542347}
[modify] https://crrev.com/46d3870fc70873949e34f263f6b71df7b3f3aaac/components/exo/pointer.cc
[modify] https://crrev.com/46d3870fc70873949e34f263f6b71df7b3f3aaac/components/exo/pointer.h
[modify] https://crrev.com/46d3870fc70873949e34f263f6b71df7b3f3aaac/components/exo/surface.cc
[modify] https://crrev.com/46d3870fc70873949e34f263f6b71df7b3f3aaac/components/exo/surface.h

Comment 8 by osh...@chromium.org, Mar 15 2018

Labels: Merge-Request-66
Project Member

Comment 9 by sheriffbot@chromium.org, Mar 15 2018

Labels: -Merge-Request-66 Merge-Review-66 Hotlist-Merge-Review
This bug requires manual review: M66 has already been promoted to the beta branch, so this requires manual review
Please contact the milestone owner if you have questions.
Owners: cmasso@(Android), cmasso@(iOS), josafat@(ChromeOS), abdulsyed@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -Pri-2 Pri-1
https://chromium-review.googlesource.com/c/chromium/src/+/903223 was backported to M65 too, does this need M65 backport too?
Labels: Merge-Request-65
not sure if this is critical, but adding 65 as well.
It looks pretty horrible and extremely easy to trigger.
Do you feel the fix is too risky? Ideally we should give user a PE experience.

Cc: bhthompson@chromium.org josa...@chromium.org
+josafat,bhthompson for M65
Lets see how this does in 66 before merging to 65, we  can probably still catch a second stable here. 
I'm still waiting for approval to merge to 66, in case it's not obvious.
Labels: -Merge-Review-66 Merge-Approved-66 M-66
Project Member

Comment 18 by bugdroid1@chromium.org, Mar 20 2018

Labels: -merge-approved-66 merge-merged-3359
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/1d8162838bbc7ac94d12447144c765a2dc9dd621

commit 1d8162838bbc7ac94d12447144c765a2dc9dd621
Author: Mitsuru Oshima <oshima@chromium.org>
Date: Tue Mar 20 17:01:21 2018

Use the cursor stored in Widget for regular surface as well.

The original CL crrev.com/c/903223 changed the GetCurosr logic
only for notification surface. Regular exo surface should do the same.

This also removes obsolete path.

Bug:  814600 
Test: manual. See bug for repro step.
Change-Id: Icbdf000db30efa96aa056c1b7fe15d38822b7610
Reviewed-on: https://chromium-review.googlesource.com/956103
Reviewed-by: David Reveman <reveman@chromium.org>
Commit-Queue: Mitsuru Oshima <oshima@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#542347}(cherry picked from commit 46d3870fc70873949e34f263f6b71df7b3f3aaac)
Reviewed-on: https://chromium-review.googlesource.com/971242
Reviewed-by: Mitsuru Oshima <oshima@chromium.org>
Cr-Commit-Position: refs/branch-heads/3359@{#345}
Cr-Branched-From: 66afc5e5d10127546cc4b98b9117aff588b5e66b-refs/heads/master@{#540276}
[modify] https://crrev.com/1d8162838bbc7ac94d12447144c765a2dc9dd621/components/exo/pointer.cc
[modify] https://crrev.com/1d8162838bbc7ac94d12447144c765a2dc9dd621/components/exo/pointer.h
[modify] https://crrev.com/1d8162838bbc7ac94d12447144c765a2dc9dd621/components/exo/surface.cc
[modify] https://crrev.com/1d8162838bbc7ac94d12447144c765a2dc9dd621/components/exo/surface.h

Labels: -Hotlist-Merge-Review -Merge-Request-65 Merge-Approved-65
Status: Fixed (was: Started)
https://crrev.com/c/977433
Project Member

Comment 21 by bugdroid1@chromium.org, Mar 23 2018

Labels: -merge-approved-65 merge-merged-3325
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/66325a1f31c169c473ee1ab054045e264fd7b31d

commit 66325a1f31c169c473ee1ab054045e264fd7b31d
Author: Mitsuru Oshima <oshima@chromium.org>
Date: Fri Mar 23 04:10:37 2018

Use the cursor stored in Widget for regular surface as well.

The original CL crrev.com/c/903223 changed the GetCurosr logic
only for notification surface. Regular exo surface should do the same.

This also removes obsolete path.

Bug:  814600 
Test: manual. See bug for repro step.
Change-Id: Icbdf000db30efa96aa056c1b7fe15d38822b7610
Reviewed-on: https://chromium-review.googlesource.com/956103
Reviewed-by: David Reveman <reveman@chromium.org>
Commit-Queue: Mitsuru Oshima <oshima@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#542347}(cherry picked from commit 46d3870fc70873949e34f263f6b71df7b3f3aaac)
Reviewed-on: https://chromium-review.googlesource.com/977433
Reviewed-by: Mitsuru Oshima <oshima@chromium.org>
Cr-Commit-Position: refs/branch-heads/3325@{#734}
Cr-Branched-From: bc084a8b5afa3744a74927344e304c02ae54189f-refs/heads/master@{#530369}
[modify] https://crrev.com/66325a1f31c169c473ee1ab054045e264fd7b31d/components/exo/pointer.cc
[modify] https://crrev.com/66325a1f31c169c473ee1ab054045e264fd7b31d/components/exo/pointer.h
[modify] https://crrev.com/66325a1f31c169c473ee1ab054045e264fd7b31d/components/exo/surface.cc
[modify] https://crrev.com/66325a1f31c169c473ee1ab054045e264fd7b31d/components/exo/surface.h

 Issue 808538  has been merged into this issue.

Sign in to add a comment