Issue metadata
Sign in to add a comment
|
Setting mouse cursor style often doesn't work
Reported by
real...@gmail.com,
Jun 13 2016
|
||||||||||||||||||||
Issue descriptionUserAgent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/53.0.2763.0 Safari/537.36 Steps to reproduce the problem: 1. Use Javascript to change a canvas element's cursor style property. 2. 3. What is the expected behavior? The mouse pointer should change to the requested style. What went wrong? The cursor style seems to not usually change. In one part of my app (which changes the cursor between "default" and "pointer" in mousemove events) it works correctly, but at other times, during click event and requestAnimationFrame handlers, the change seems to be completely ignored, or I occasionally see it flash very briefly to the requested style before changing back. Did this work before? Yes Until about a week ago I think, but I'm not sure in which version it broke. It works correctly in Chromium 51.0.2704.79. Chrome version: 53.0.2763.0 Channel: dev OS Version: Debian testing Flash Version: Shockwave Flash 22.0 r0
,
Jun 14 2016
This attachment distils what I'm trying to do in my app. The cursor should disappear after a second, and temporarily reappear if the mouse is moved. I'm using requestAnimationFrame and using my own timekeeping rather than using setTimeout, because my main app uses requestAnimationFrame anyway, so it was simpler.
,
Jun 14 2016
Thank you for providing more feedback. Adding requester "ashejole@chromium.org" for another review and adding "Needs-Review" label for tracking. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jun 16 2016
@realh69: Thanks for the sample file, much appreciated. Able to reproduce the above issue on Ubuntu 14.04 only, not seen on windows & Mac. Regression issue broken in M52, below is the bisect details. Narrow bisect ------------ CHANGELOG URL: https://chromium.googlesource.com/chromium/src/+log/f086d731245c5a5c300a98fbfb3e410fbf3e6220..061eb2409c683efcd0c2b3f09db0197992ed37c3 Based on the above changelog, plausible offending CL: https://codereview.chromium.org/1945423002 ? @erg: Hey, would you mind checking the above issue and see if it's related to your change ? Feel free to re-assign to concern dev if that is not the case. Marking the above issue as RB-Stable, feel free to remove if not required. I really appreciate your help. Thank you!
,
Jun 16 2016
The original patch (https://codereview.chromium.org/1945423002) had a partial revert because of said bug (First revert: https://codereview.chromium.org/2027693003). scottmg@ than speculatively reverted my revert (https://codereview.chromium.org/2042213003) to fix an unrelated bug, and 53.0.2763.0 was then cut. So this is currently broken. scottmg@, can we back out the revert yet?
,
Jun 16 2016
@erg: Thanks for the update :)
,
Jun 16 2016
Yes, I think it'd didn't help, so I'll reland.
,
Jun 16 2016
Revert of revert of revert landed: https://chromium.googlesource.com/chromium/src.git/+/bf6f1be98c75b02ae4d4a6a0b5f330ff968e8b40
,
Jun 17 2016
@erg/scottmg: Thanks a lot for the quick turnaround. Thank you! |
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by ashej...@chromium.org
, Jun 14 2016Labels: Needs-Feedback