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

Issue 631405 link

Starred by 5 users

Issue metadata

Status: Fixed
Owner:
Closed: Aug 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Cursor not disappear / stay visible when playing video

Reported by jeremy9...@gmail.com, Jul 26 2016

Issue description

UserAgent: 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 !
 
Labels: Needs-Feedback
please make sure mouse cursor is not positioned in the video control bar position. move it to the center or top of the video. 
Please also provide a repro url.
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
Labels: -Needs-Feedback Needs-Bisect
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.

Comment 5 by ajha@chromium.org, Jul 27 2016

Cc: gov...@chromium.org e...@chromium.org ajha@chromium.org ligim...@chromium.org
Components: UI
Labels: -Type-Bug -Pri-2 -Needs-Bisect ReleaseBlock-Stable M-52 hasbisect Pri-1 Type-Bug-Regression
Owner: scottmg@chromium.org
Status: Assigned (was: Unconfirmed)
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.

Comment 6 by ajha@chromium.org, Jul 27 2016

>Works fine on Mac OS 10.11.5 and Windows-7 on chrome version: 52.0.2743.82.

Comment 7 by gov...@chromium.org, 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. 


Labels: -M-52 M-53
This is a regession in M53, tagging accordingly.
Labels: -M-53 -ReleaseBlock-Stable M-52
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.
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).

Comment 11 by cxp...@gmail.com, 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.
Labels: -hasbisect Needs-Bisect
Cc: sadrul@chromium.org sky@chromium.org
Labels: -Needs-Bisect hasbisect
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,
crrev.com/391660 does seem plausible (the !OS_CHROMEOS -> OS_WIN changes in r_w_h_v_a.cc look relevant).

Comment 15 by e...@chromium.org, 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.
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.
Labels: Merge-Request-52
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?

Comment 18 by e...@chromium.org, Aug 2 2016

SGTM

Comment 19 Deleted

Labels: -Merge-Approved-53 Merge-Approved-52
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.
Project Member

Comment 21 by bugdroid1@chromium.org, Aug 2 2016

Labels: -merge-approved-52 merge-merged-2743
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

Comment 22 by ajha@chromium.org, Aug 3 2016

Labels: TE-Verified-M52 TE-Verified-52.0.2743.116
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.
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 !
Status: Fixed (was: Assigned)
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 !

Comment 26 by cxp...@gmail.com, 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.
Firefox have the problem too so I guess, but not sure, that's a site problem.
Not reproduce the issue on youtube/vimeo/viki videos - Chrome 52.0.2743.116/8350.68.0 Peppy.





I confirm it's fixed with 52.0.2743.116. Thanks !

Sign in to add a comment