New issue
Advanced search Search tips

Issue 689126 link

Starred by 0 users

Issue metadata

Status: Archived
Owner:
Closed: Feb 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

No mouse pointer on tip of tree

Project Member Reported by sky@chromium.org, Feb 6 2017

Issue description

I'm at a54850c488a394002d1221f3f3084eea1d59bb24 , 448263. When moving the mouse around I don't get the pointer at all, no matter where I move it.
 

Comment 1 by sky@chromium.org, Feb 6 2017

Cc: kylec...@chromium.org sadrul@chromium.org
I'll poke at this to see if we are at least trying to set the pointer.

Comment 2 by sky@chromium.org, Feb 7 2017

Owner: sky@chromium.org
Status: Started (was: Untriaged)
Project Member

Comment 3 by bugdroid1@chromium.org, Feb 7 2017

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

commit adb99b98e9271ed418fbc941852f5547b0d5d59c
Author: sky <sky@chromium.org>
Date: Tue Feb 07 19:22:22 2017

Fixes bugs in cursor handling

. CURSOR_NULL on ozone seems to mean leave the last cursor. So, this
  change makes sure we use POINTER rather than CURSOR_NULL.
. aura sets the cursor on the root window, which means mus needs to
  pick up the cursor from the root. I'm not particularly thrilled
  about this, but changing mus is a lot more straightforward than
  changing aura.

BUG= 689126 
TEST=covered by tests
R=erg@chromium.org

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

[modify] https://crrev.com/adb99b98e9271ed418fbc941852f5547b0d5d59c/services/ui/ws/cursor_unittest.cc
[modify] https://crrev.com/adb99b98e9271ed418fbc941852f5547b0d5d59c/services/ui/ws/event_dispatcher.cc
[modify] https://crrev.com/adb99b98e9271ed418fbc941852f5547b0d5d59c/services/ui/ws/event_dispatcher.h
[modify] https://crrev.com/adb99b98e9271ed418fbc941852f5547b0d5d59c/services/ui/ws/event_dispatcher_delegate.h
[modify] https://crrev.com/adb99b98e9271ed418fbc941852f5547b0d5d59c/services/ui/ws/server_window.cc
[modify] https://crrev.com/adb99b98e9271ed418fbc941852f5547b0d5d59c/services/ui/ws/test_utils.cc
[modify] https://crrev.com/adb99b98e9271ed418fbc941852f5547b0d5d59c/services/ui/ws/test_utils.h
[modify] https://crrev.com/adb99b98e9271ed418fbc941852f5547b0d5d59c/services/ui/ws/window_manager_state_unittest.cc
[modify] https://crrev.com/adb99b98e9271ed418fbc941852f5547b0d5d59c/services/ui/ws/window_server.cc
[modify] https://crrev.com/adb99b98e9271ed418fbc941852f5547b0d5d59c/services/ui/ws/window_server.h
[modify] https://crrev.com/adb99b98e9271ed418fbc941852f5547b0d5d59c/services/ui/ws/window_tree_unittest.cc

Comment 4 by sky@chromium.org, Feb 7 2017

Status: Fixed (was: Started)

Comment 5 by dchan@google.com, Apr 17 2017

Labels: VerifyIn-59

Comment 6 by dchan@google.com, May 30 2017

Labels: VerifyIn-60

Comment 7 by dchan@chromium.org, Aug 1 2017

Labels: VerifyIn-61

Comment 8 by dchan@chromium.org, Oct 14 2017

Status: Archived (was: Fixed)

Sign in to add a comment