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

Issue 671127 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

Regression: Weird default mouse pointer is seen instead of mouse drag hand pointer in http://media.tojicode.com/q3bsp/

Project Member Reported by sc00335...@techmahindra.com, Dec 5 2016

Issue description

Chrome Version: 57.0.2942.0 dev
OS: Ubuntu 14.04

What steps will reproduce the problem?
(1)Launch chrome and go to http://media.tojicode.com/q3bsp/ >> Hover mouse and observe mouse pointer.

Expected: Mouse drag pointer should be seen
Actual: Instead default mouse pointer is seen with some dot at bottom.

This is a regression issue broken in M57. Will provide bisect info and windows behaviour soon.


 
Actual_cursor.ogv
544 KB View Download
Expected_cursor.ogv
526 KB View Download
This issue is working fine in Windows.
Labels: ReleaseBlock-Stable
Manual Bisect Info:
====================
Good build : 56.0.2924.0
Bad build : 57.0.2926.0
Labels: -Needs-Bisect hasbisect-per-revision
Owner: thomasanderson@chromium.org
Status: Assigned (was: Unconfirmed)
Using the per-revision bisect providing the bisect results,
Good build: 56.0.2924.0 (Revision: 433059).
Bad build: 57.0.2926.0 (Revision: 433437).

You are probably looking for a change made after 433149 (known good), but no later than 433150 (first known bad).
CHANGELOG URL:
  https://chromium.googlesource.com/chromium/src/+log/863f2af2768cddc229e8822c622fa7dd539de290..16c9f899aa654b5cb868242f2a19ab375b8397f9

@thomasanderson -- Could you please look into the issue, pardon me if it has nothing to do with your changes and if possible please assign it to concern owner.

Thank You.
Components: -Blink>Input -Blink Blink>DataTransfer
Components: -Blink>DataTransfer Blink>Input
Status: Started (was: Assigned)
msrchandra@ thanks, this is related to my change.  I'll investigate.

OP, which cursor theme are you using (DMZ-White?)
Status: WontFix (was: Started)
This is correct behavior.  Please see https://developer.mozilla.org/en-US/docs/Web/CSS/cursor

The app is using the 'move' cursor, which is meant for drag-n-drop, but what it wants is 'all-scroll'.
Status: Assigned (was: WontFix)
In responce to comment#7: Please go to https://developer.mozilla.org/en-US/docs/Web/CSS/cursor and hover on both "move" cursor and "all-scroll" cursor and observe those cursors. Respective symbols should be seen. But in latest for both 'move' cursor and 'all-scroll' default cursor with small dot at bottom is seen.

Attaching both M55 stable behavior and latest M57 behavior videos. Please observe

@thomasanderson: Re-opening issue please revert if any.  
Issue 671127.ogv
4.6 MB View Download
Status: WontFix (was: Assigned)
Once again, this is correct behavior.  Your cursor theme chose to make the move cursor look like that.

There are 3 types of dnd action cursors: alias, copy, and move.  If you look at alias and copy, you can see it is a normal cursor with small icon at the bottom right to indicate which type of dnd operation it is.  The makers of your cursor theme decided that dnd-move should be the same: a left pointer with an icon at the bottom right.  If you don't like that, please file a bug externally for your cursor theme.

If many Linux cursor themes use a bad icon, I would consider changing 'move' to use the 'all-scroll' icon.
Status: Assigned (was: WontFix)
Sorry I didn't actually take a look at your video.  We shouldn't be using the same cursor for move and all-scroll.  Will fix soon
Project Member

Comment 11 by bugdroid1@chromium.org, Dec 8 2016

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

commit f8614e087b47efbc3f359234b461cd2d88833f8a
Author: thomasanderson <thomasanderson@google.com>
Date: Thu Dec 08 02:24:57 2016

Linux aura: use all-scroll cursor in place of move

R=sadrul@chromium.org
BUG= 671127 

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

[modify] https://crrev.com/f8614e087b47efbc3f359234b461cd2d88833f8a/ui/base/cursor/cursor_loader_x11.cc

Status: Fixed (was: Assigned)

Sign in to add a comment