Regression : Mouse pointer Flickers on hovering mouse on the message center |
|||||||||||||
Issue descriptionChrome 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.
,
Feb 22 2018
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
,
Feb 22 2018
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.
,
Mar 7 2018
,
Mar 7 2018
JFYI: "original issue" mentioned in the comment #3 was Issue 798056.
,
Mar 9 2018
,
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
,
Mar 15 2018
,
Mar 15 2018
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
,
Mar 16 2018
,
Mar 16 2018
https://chromium-review.googlesource.com/c/chromium/src/+/903223 was backported to M65 too, does this need M65 backport too?
,
Mar 16 2018
not sure if this is critical, but adding 65 as well.
,
Mar 19 2018
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.
,
Mar 19 2018
+josafat,bhthompson for M65
,
Mar 19 2018
Lets see how this does in 66 before merging to 65, we can probably still catch a second stable here.
,
Mar 20 2018
I'm still waiting for approval to merge to 66, in case it's not obvious.
,
Mar 20 2018
,
Mar 20 2018
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
,
Mar 22 2018
,
Mar 23 2018
,
Mar 23 2018
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
,
Mar 29 2018
Issue 808538 has been merged into this issue. |
|||||||||||||
►
Sign in to add a comment |
|||||||||||||
Comment 1 by yoshiki@chromium.org
, Feb 22 20186.3 MB
6.3 MB View Download