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

Issue 614654 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: May 2016
Cc:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression: Mouse pointer does not change to Zoom out on zooming in a picture.

Project Member Reported by radhakri...@techmahindra.com, May 25 2016

Issue description

Chrome Version       : 53.0.2748.0 (Official Build)
OS Version : Ubuntu 14.04

URL: http://vignette2.wikia.nocookie.net/austinally/images/8/80/Golden-Retriever-Puppies-Wallpaper-Desktop-Background.jpg/revision/latest?cb=20151113223713

What steps will reproduce the problem?
(1)Launch chrome and open the above url.
(2)Hover the mouse on the image(mouse pointer changes to Zoom in)
(3)Click on the image without moving the mouse and observe the pointer.

What is the expected result?
On clicking the image the mouse pointer should change to zoom out.

What happens instead?
The mouse pointer remains the same even though the image is zoomed in.Only on moving the mouse does the pointer change to zoom out.

This is a regression issue broken in M-52

This issue is not seen on windows.

 
actual.mp4
3.0 MB Download
expected.mp4
2.6 MB Download
Labels: hasbisect
Owner: e...@chromium.org
Status: Assigned (was: Unconfirmed)
Good Build:52.0.2724.0 
Bad Build :52.0.2725.0 

CHANGELOG URL:
  https://chromium.googlesource.com/chromium/src/+log/f086d731245c5a5c300a98fbfb3e410fbf3e6220..061eb2409c683efcd0c2b3f09db0197992ed37c3

Suspecting https://codereview.chromium.org/1945423002 from changelog.

@erg: Please re-assign if the issue is not related to your change.
Able to reproduce the issue on Ubuntu 14.04 using chrome latest Dev 53.0.2748.0 . Observed on clicking the image to zoom in the mouse pointer is not changing to zoom out.

Note: Issue is working fine on Mac OS as well.

Comment 3 by e...@chromium.org, May 31 2016

Status: Started (was: Assigned)
Project Member

Comment 4 by bugdroid1@chromium.org, May 31 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b9020bacdaff760ec0acc4fc5e9db9239438636d

commit b9020bacdaff760ec0acc4fc5e9db9239438636d
Author: erg <erg@chromium.org>
Date: Tue May 31 23:42:58 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 

Review-Url: https://codereview.chromium.org/2027693003
Cr-Commit-Position: refs/heads/master@{#396977}

[modify] https://crrev.com/b9020bacdaff760ec0acc4fc5e9db9239438636d/content/browser/renderer_host/render_widget_host_view_aura.cc

Comment 5 by e...@chromium.org, May 31 2016

Status: Fixed (was: Started)
Project Member

Comment 6 by bugdroid1@chromium.org, Jun 8 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/2161ef83c680c890b8f3beacac4984065eb2bd50

commit 2161ef83c680c890b8f3beacac4984065eb2bd50
Author: scottmg <scottmg@chromium.org>
Date: Wed Jun 08 01:03:32 2016

Revert of Revert cursor changes to RenderWidgetHostViewAura. (patchset #2 id:20001 of https://codereview.chromium.org/2027693003/ )

Reason for revert:
Speculative revert for very small likelihood of causing elevated crash rates as detailed in https://bugs.chromium.org/p/chromium/issues/detail?id=616399

Original issue's description:
> 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 
>
> Committed: https://crrev.com/b9020bacdaff760ec0acc4fc5e9db9239438636d
> Cr-Commit-Position: refs/heads/master@{#396977}

TBR=sadrul@chromium.org,erg@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG= 602727 , 614654 

Review-Url: https://codereview.chromium.org/2042213003
Cr-Commit-Position: refs/heads/master@{#398445}

[modify] https://crrev.com/2161ef83c680c890b8f3beacac4984065eb2bd50/content/browser/renderer_host/render_widget_host_view_aura.cc

Project Member

Comment 7 by bugdroid1@chromium.org, Jun 16 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/bf6f1be98c75b02ae4d4a6a0b5f330ff968e8b40

commit bf6f1be98c75b02ae4d4a6a0b5f330ff968e8b40
Author: scottmg <scottmg@chromium.org>
Date: Thu Jun 16 18:37:24 2016

Reland of cursor changes to RenderWidgetHostViewAura. (patchset #1 id:1 of https://codereview.chromium.org/2042213003/ )

Reason for revert:
Relanding, didn't seem to help.

Original issue's description:
> Revert of Revert cursor changes to RenderWidgetHostViewAura. (patchset #2 id:20001 of https://codereview.chromium.org/2027693003/ )
>
> Reason for revert:
> Speculative revert for very small likelihood of causing elevated crash rates as detailed in https://bugs.chromium.org/p/chromium/issues/detail?id=616399
>
> Original issue's description:
> > 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 
> >
> > Committed: https://crrev.com/b9020bacdaff760ec0acc4fc5e9db9239438636d
> > Cr-Commit-Position: refs/heads/master@{#396977}
>
> TBR=sadrul@chromium.org,erg@chromium.org
> # Not skipping CQ checks because original CL landed more than 1 days ago.
> BUG= 602727 , 614654 
>
> Committed: https://crrev.com/2161ef83c680c890b8f3beacac4984065eb2bd50
> Cr-Commit-Position: refs/heads/master@{#398445}

TBR=sadrul@chromium.org,erg@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG= 602727 , 614654 

Review-Url: https://codereview.chromium.org/2071903003
Cr-Commit-Position: refs/heads/master@{#400213}

[modify] https://crrev.com/bf6f1be98c75b02ae4d4a6a0b5f330ff968e8b40/content/browser/renderer_host/render_widget_host_view_aura.cc

Project Member

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

Labels: 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

Labels: TE-Verified-M52 TE-Verified-52.0.2743.116
Verified the issue on Ubuntu 14.04 using 52.0.2743.116 and its working fine.
Please find the attached screen cast for the same.
614654_Aug_3.ogv
5.4 MB View Download
Labels: M-52
Verified the issue on Chrome 52.0.2743.116/8350.68.0 - Peppy

Sign in to add a comment