Issue metadata
Sign in to add a comment
|
Cursor not disappear / stay visible when playing video
Reported by
jeremy9...@gmail.com,
Jul 26 2016
|
||||||||||||||||||||||
Issue descriptionUserAgent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/52.0.2743.82 Safari/537.36 Example URL: Steps to reproduce the problem: - Use Linux (Fedora 24 / Gnome Shell 3.20) - Play a Flash or HTML5 video What is the expected behavior? Cursor disappear What went wrong? Cursor not disappear / stay visible Did this work before? N/A Is it a problem with Flash or HTML5? Both Does this work in other browsers? Yes Chrome version: 52.0.2743.82 Channel: stable OS Version: Fedora 24 Flash Version: Shockwave Flash 22.0 r0 MPV (video player) manage to make the cursor disappear so not a Fedora / Gnome bug. Already partly reported years ago https://bugs.chromium.org/p/chromium/issues/detail?id=33763 Thanks !
,
Jul 26 2016
Of course I can confirm to you that the cursor is not positioned in the video control bar position. It stay visible wherever I put it. It happen with all videos but here is 2 exemple. https://www.youtube.com/watch?v=eJ2RNBAFLj0 http://youwatch.org/1uqjb2ur7c2s
,
Jul 26 2016
,
Jul 26 2016
I can add that Firefox make the cursor disappear except on this video I linked http://youwatch.org/1uqjb2ur7c2s I have 2 computers, both on Fedora 24, one desktop with Nvidia proprietary drivers and one laptop with Intel driver and the cursor is not disappearing on both computers.
,
Jul 27 2016
Able to reproduce this on the latest stable(52.0.2743.82) on Linux Ubuntu 14.04/ Fedora 22. Works fine on Mac OS 10.11.5 and Linux Ubuntu 14.04 with the same chrome version. Cursor is still seen on playing videos on youtube and going to full-screen on Linux. This is fixed in the latest M-54(54.0.2808.0) and the latest Dev(53.0.2785.30) on Linux. Reverse bisected to identify the CL that fixed this. Last bad build : 53.0.2769.0 First good build : 53.0.2770.0 Changelog:https://chromium.googlesource.com/chromium/src/+log/ecb0fd43ada77f1ff2c2c322348fa5fdb9fb4f3a..2063e083d9fed1165fe30a5e791de4fc7b960090 This was fixed by https://codereview.chromium.org/2071903003. scottmg@: If this is safe merge, Could you please get this merged to M-52 as well.
,
Jul 27 2016
>Works fine on Mac OS 10.11.5 and Windows-7 on chrome version: 52.0.2743.82.
,
Jul 27 2016
Is this a clear regression with M52? As per Bug Description (Already partly reported years ago https://bugs.chromium.org/p/chromium/issues/detail?id=33763) Also please note that M52 is already in Stable for Desktop and bar is VERY high. We're going to have M52 refresh Stable release next week and we can take this change in ONLY if it is critical, well baked/verified in Canary and safe to merge. Please confirm this. Thank you.
,
Jul 27 2016
This is a regession in M53, tagging accordingly.
,
Jul 27 2016
The issue do not exist in M53 and M54 and Linux specific, don't think its a Stable blocker, leaving it open just in case we need a merge.
,
Jul 28 2016
Did you tried with this video http://youwatch.org/1uqjb2ur7c2s ? This kind of video, I may be wrong but, seem to have an overlay on top of the video or something like that. For instance Firefox don't/can't make the cursor disappear on this one, at least for now (https://bugzilla.mozilla.org/show_bug.cgi?id=1289364).
,
Aug 1 2016
I can confirm this bug exists on google-chrome-stable as well on version 52.0.2743.82 running on GNOME 3.20.x. Firefox does not have the same issue. Just install any variant of Linux with GNOME 3.20.x with google-chrome-stable 52.0.2743.82 and run any YouTube video via fullscreen and you'll see this problem. It's easy to reproduce.
,
Aug 1 2016
,
Aug 2 2016
Able to reproduce the issue on Linux Ubuntu 14.04 on latest stable 52.0.2743.82. Cursor is still seen on playing videos on youtube and going to full-screen on Linux. This has regressed in M-52. Good Build:52.0.2724.0 Bad Build :52.0.2725.0 CHANGELOG URL: https://chromium.googlesource.com/chromium/src/+log/f086d731245c5a5c300a98fbfb3e410fbf3e6220..061eb2409c683efcd0c2b3f09db0197992ed37c3 From above CL, suspecting below: https://chromium.googlesource.com/chromium/src/+/19ddf3a6399fbee33681ea74d41529b4915e9c03 ccing 'erg, sadrul, sky' from reviewers list. Could anyone please confirm if this safe merging to M-52 as well. Thanks,
,
Aug 2 2016
crrev.com/391660 does seem plausible (the !OS_CHROMEOS -> OS_WIN changes in r_w_h_v_a.cc look relevant).
,
Aug 2 2016
+scottmg crrev.com/391660 was the original breakage. It was fixed with crrev.com/396977. scottmg@ reverted the revert (crrev.com/398445), and then reverted his revert of the revert (crrev.com/400213). I don't know why scottmg@ didn't merge the revert of the revert though, or the state of the M-52 branch, but crrev.com/396977 should be minimal enough and low impact enough to merge in.
,
Aug 2 2016
Did we branch between my revert (398445)/reland (400213)? Or it just that 396977 was never merged? I don't really know what this bug is about, but I can merge that CL to 52 if that's all we need to do.
,
Aug 2 2016
Answering my own questions: [5ca3397...]d:\src\cr3\src>git find-releases 19ddf3a6399fbee33681ea74d41529b4915e9c03 commit 19ddf3a6399fbee33681ea74d41529b4915e9c03 was: initially in 52.0.2725.0 No merges found. If this seems wrong, be sure that you did: git fetch origin && gclient sync --with_branch_heads [5ca3397...]d:\src\cr3\src>git find-releases b9020bacdaff760ec0acc4fc5e9db9239438636d commit b9020bacdaff760ec0acc4fc5e9db9239438636d was: initially in 53.0.2754.0 No merges found. If this seems wrong, be sure that you did: git fetch origin && gclient sync --with_branch_heads [5ca3397...]d:\src\cr3\src>git find-releases 2161ef83c680c890b8f3beacac4984065eb2bd50 commit 2161ef83c680c890b8f3beacac4984065eb2bd50 was: initially in 53.0.2763.0 No merges found. If this seems wrong, be sure that you did: git fetch origin && gclient sync --with_branch_heads [5ca3397...]d:\src\cr3\src>git find-releases bf6f1be98c75b02ae4d4a6a0b5f330ff968e8b40 commit bf6f1be98c75b02ae4d4a6a0b5f330ff968e8b40 was: initially in 53.0.2770.0 No merges found. If this seems wrong, be sure that you did: git fetch origin && gclient sync --with_branch_heads So, 391660 was 52, hence it being broken. All the rest (the fix and my fiddling) were on 53, so it is fine. erg, do you want me to merge your crrev.com/396977 to 52 then?
,
Aug 2 2016
SGTM
,
Aug 2 2016
M52 is already in Stable for Desktop and bar is VERY high. We can crrev.com/396977 only if extremely safe to merge (per comment #15 and 17 it seems to be safe and fixes actual issue). So approving merge to M52 branch 2743. Please merge asap as we're cutting M52 stable RC soon today. Thank you.
,
Aug 2 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/6b251bef1179aa523da6afba9047b3874f760b9a commit 6b251bef1179aa523da6afba9047b3874f760b9a Author: Scott Graham <scottmg@chromium.org> Date: Tue Aug 02 19:29:27 2016 Revert cursor changes to RenderWidgetHostViewAura. This is a selective revert of one file from https://codereview.chromium.org/1945423002, which was breaking cursor changes on zoom in/zoom out on the Linux version of chrome. BUG= 602727 , 614654 , 631405 Review-Url: https://codereview.chromium.org/2027693003 Cr-Commit-Position: refs/heads/master@{#396977} (cherry picked from commit b9020bacdaff760ec0acc4fc5e9db9239438636d) Review URL: https://codereview.chromium.org/2204883003 . Cr-Commit-Position: refs/branch-heads/2743@{#726} Cr-Branched-From: 2b3ae3b8090361f8af5a611712fc1a5ab2de53cb-refs/heads/master@{#394939} [modify] https://crrev.com/6b251bef1179aa523da6afba9047b3874f760b9a/content/browser/renderer_host/render_widget_host_view_aura.cc
,
Aug 3 2016
Verified the merge on the latest M-52(52.0.2743.116/ 2743@{#728}) on Linux Ubuntu 14.04, this is working as intended. Cursor disappears after sometime when the video is playing in full-screen.
,
Aug 3 2016
Can you say me if it's the website that prevent to hide the cursor with this kind of videos please ? youwatch.org/1uqjb2ur7c2s Thanks !
,
Aug 3 2016
,
Aug 3 2016
A little disappointed that nobody can say me if it's the website that prevent to hide the cursor with this kind of videos youwatch.org/1uqjb2ur7c2s But thank you for the quick fix !
,
Aug 3 2016
Hi, It's easy. Just try different browsers. If all of them have the same issue, it's likely the site itself. You can inspect element to find out more. If only Chrome has the issue for example.. Then it's the browser. Like in this case.
,
Aug 3 2016
Firefox have the problem too so I guess, but not sure, that's a site problem.
,
Aug 3 2016
Not reproduce the issue on youtube/vimeo/viki videos - Chrome 52.0.2743.116/8350.68.0 Peppy.
,
Aug 3 2016
I confirm it's fixed with 52.0.2743.116. Thanks ! |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by yini...@chromium.org
, Jul 26 2016