Issue metadata
Sign in to add a comment
|
Invisible cursor impossible
Reported by
xilef...@gmail.com,
Mar 7 2017
|
||||||||||||||||||||||
Issue descriptionUserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/56.0.2924.87 Safari/537.36 Steps to reproduce the problem: 1. Add cursor: none; anywhere 2. Prevent contextmenu from opening 3. Right-click What is the expected behavior? Mouse cursor stays invisible What went wrong? Mouse cursor appears for a few ms, then gets invisible again Did this work before? N/A Does this work in other browsers? No Firefox nightly seems fine Chrome version: 56.0.2924.87 Channel: stable OS Version: 10.0 Flash Version: Same behaviour with cursor images
,
Mar 8 2017
@xilefmai-- Could you please provide us the sample test url/test case / test steps to reproduce the issue and possible help us by providing the expected result screenshot. Thanks!
,
Mar 8 2017
I can't capture a screenshot, the cursor gets visible and flickers 1 or 2 times until it gets invisible again. Test case attached. I'm using windows 10 on a dell xps 15 9550
,
Mar 8 2017
Thank you for providing more feedback. Adding requester "hdodda@chromium.org" to the cc list and removing "Needs-Feedback" label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Mar 8 2017
,
Mar 9 2017
Tested on windows 10 & 7 , ubuntu 14.04 , mac os 10.12.2 using chrome M56 #56.0.2924.87 and M59 #59.0.3035.0 issue is reproduced. Cursor is visible on the given html page and goes invisible when moved inside the page .Attached screencast for reference. Issue is seen from M30 #30.0.1549.0 and is a non-regression issue. Issue is also seen in Mac and linux os also. Marking it as untraiged for further inputs on this. Thanks!
,
Mar 9 2017
,
Mar 9 2017
Setting this back to unconfirmed as the repro doesn't match the original bug description. I.e., the original bug states that the expected behavior is that the cursor remains invisible, and the bug is that it doesn't remain invisible on a right click. The repro states that the cursor remains invisible. I can't repro this on Ubuntu 14.04.5 or Mac 10.12.3 on stable version 56.0.2924.87 or canary version 59.0.3035.0
,
Mar 15 2017
Able to reproduce the issue on Windows 10 using latest canary #59.0.3041.0 but unable to reproduce the issue using chrome reported version #56.0.2924.87. This is a win-OS specific issue. Issue is not seen in Linux and Mac. Bisect Information: ===================== Good build: 58.0.2994.0 Revision(446546) Bad Build : 58.0.2995.0 Revision(446864) Change Log URL: https://chromium.googlesource.com/chromium/src/+log/c4ac85eec321202ca4ecd56f0364dd49d9fb8236..a05a1702da44842fed11d76a18a1452c06421473 From the above change log suspecting below change Review url: https://codereview.chromium.org/2637403012 dtapuska@ - Could you please check whether this is caused with respect to your change, if not please help us in assigning it to the right owner. Note: This seems to be a recent regression. Hence, adding label ReleaseBlock-Stable. Thanks...!!
,
Mar 16 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/7e40fe40d19b2c766df8d860169b11e2cacd9771 commit 7e40fe40d19b2c766df8d860169b11e2cacd9771 Author: dtapuska <dtapuska@chromium.org> Date: Thu Mar 16 13:37:02 2017 Remove setting the pointer cursor when the context menu event is sent. The cursor would flash when the context menu is sent if the event is preventDefaulted. With a change https://codereview.chromium.org/2637403012 this occurred more frequently because the move event would get suppressed. Ultimately I found when this code was activated in https://codereview.chromium.org/7270 and it seems it was mis-guided in being added. I've check on Windows removing this code makes everything work fine. BUG= 699232 Review-Url: https://codereview.chromium.org/2755783002 Cr-Commit-Position: refs/heads/master@{#457418} [modify] https://crrev.com/7e40fe40d19b2c766df8d860169b11e2cacd9771/third_party/WebKit/Source/web/WebViewImpl.cpp
,
Mar 16 2017
Removing RB-Stable. This issue existed for a long time (in all versions of Chrome); it was made a little worse by my change but I don't think it necessitates a RB-Stable label.
,
Mar 16 2017
,
Mar 17 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/05e316e960818110e3c1378c75cedb6372fd2792 commit 05e316e960818110e3c1378c75cedb6372fd2792 Author: dtapuska <dtapuska@chromium.org> Date: Fri Mar 17 00:41:04 2017 Remove setting the pointer cursor when the context menu event is sent. I thought I checked WebFrameWidgetImpl as well but it appears I hadn't. Remove the useless code there as well. WebViewImpl change was originally done in https://chromium.googlesource.com/chromium/src.git/+/7e40fe40d19b2c766df8d860169b11e2cacd9771 BUG= 699232 R=bokan@chromium.org Review-Url: https://codereview.chromium.org/2754003004 Cr-Commit-Position: refs/heads/master@{#457629} [modify] https://crrev.com/05e316e960818110e3c1378c75cedb6372fd2792/third_party/WebKit/Source/web/WebFrameWidgetImpl.cpp
,
Mar 17 2017
Tested the issue on windows 10 using chrome dev version #59.0.3043.0 as per comment #0 and #3. Observed that after opening the cursor_bug.html in chrome browser and applying right click the cursor appeared for a few ms, then became invisible again. This doesn't seem to work as expected. Attaching screen cast for reference. dtapuska@ - Could you please verify the screen cast and please confirm the behavior. Thanks...!!
,
Mar 17 2017
It seems that fix missed chrome version #59.0.3043.0 and is available in chrome version #59.0.3044.0 revision(457648). Observed that the fix is working as expected on windows 10 using chrome version #59.0.3044.0. Attaching screen cast for reference. Hence, adding the verified labels. Thanks...!!
,
Mar 17 2017
,
Mar 17 2017
,
Mar 18 2017
Your change meets the bar and is auto-approved for M58. Please go ahead and merge the CL to branch 3029 manually. Please contact milestone owner if you have questions. Owners: amineer@(Android), cmasso@(iOS), bhthompson@(ChromeOS), govind@(Desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Mar 20 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/7826aa234283f17ebd376da3dda80ea1c31fe080 commit 7826aa234283f17ebd376da3dda80ea1c31fe080 Author: Dave Tapuska <dtapuska@chromium.org> Date: Mon Mar 20 13:12:09 2017 Remove setting the pointer cursor when the context menu event is sent. The cursor would flash when the context menu is sent if the event is preventDefaulted. With a change https://codereview.chromium.org/2637403012 this occurred more frequently because the move event would get suppressed. Ultimately I found when this code was activated in https://codereview.chromium.org/7270 and it seems it was mis-guided in being added. I've check on Windows removing this code makes everything work fine. BUG= 699232 Review-Url: https://codereview.chromium.org/2755783002 Cr-Commit-Position: refs/heads/master@{#457418} (cherry picked from commit 7e40fe40d19b2c766df8d860169b11e2cacd9771) Review-Url: https://codereview.chromium.org/2757243002 . Cr-Commit-Position: refs/branch-heads/3029@{#295} Cr-Branched-From: 939b32ee5ba05c396eef3fd992822fcca9a2e262-refs/heads/master@{#454471} [modify] https://crrev.com/7826aa234283f17ebd376da3dda80ea1c31fe080/third_party/WebKit/Source/web/WebViewImpl.cpp
,
Mar 20 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/814b1ba05c324df5e8c1499c4b22eb397b9864c8 commit 814b1ba05c324df5e8c1499c4b22eb397b9864c8 Author: Dave Tapuska <dtapuska@chromium.org> Date: Mon Mar 20 13:14:54 2017 Remove setting the pointer cursor when the context menu event is sent. I thought I checked WebFrameWidgetImpl as well but it appears I hadn't. Remove the useless code there as well. WebViewImpl change was originally done in https://chromium.googlesource.com/chromium/src.git/+/7e40fe40d19b2c766df8d860169b11e2cacd9771 BUG= 699232 R=bokan@chromium.org Review-Url: https://codereview.chromium.org/2754003004 Cr-Commit-Position: refs/heads/master@{#457629} (cherry picked from commit 05e316e960818110e3c1378c75cedb6372fd2792) Review-Url: https://codereview.chromium.org/2763583002 . Cr-Commit-Position: refs/branch-heads/3029@{#296} Cr-Branched-From: 939b32ee5ba05c396eef3fd992822fcca9a2e262-refs/heads/master@{#454471} [modify] https://crrev.com/814b1ba05c324df5e8c1499c4b22eb397b9864c8/third_party/WebKit/Source/web/WebFrameWidgetImpl.cpp
,
Mar 22 2017
Observed that the fix is working as expected on windows 7 & 10 using chrome version #58.0.3029.33. Attaching screen cast for reference. Hence, adding the verified labels. Thanks...!!
,
Jun 9 2017
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by manoranj...@chromium.org
, Mar 7 2017