New issue
Advanced search Search tips

Issue 600811 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Apr 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug
mus



Sign in to add a comment

no cursor displays in "pixel" mus

Project Member Reported by rjkroege@chromium.org, Apr 5 2016

Issue description

mus "pixel" doesn't display cursor images. some event data is being distributed

 
Did a bit more digging using the oxygen build. The cursor is still set to 0 (kCursorNull) as well, however on X11 that is equivalent to XUndefineCursor which sets the cursor to the default cursor. This seems to be the same behavior on Windows.

Perhaps the cursor should be initialized to kCursorNone, then setting the cursor to kCursorNull may show a cursor, though I'd have to test and see how the cursor bitmap is defined for kCursorNull.
Cc: -dnicoara@chromium.org kylec...@chromium.org
Owner: dnicoara@chromium.org
I'm taking this since I've got 2 changes that should fix this issue.
Project Member

Comment 4 by bugdroid1@chromium.org, Apr 7 2016

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

commit c82ddf12fd204b7ad9d684049f32ee42952a66f7
Author: dnicoara <dnicoara@chromium.org>
Date: Thu Apr 07 18:15:44 2016

Set last cursor default to kCursorNone

This makes the last set cursor as hidden.

Since MUS is controlling the cursor (and managing the cursor
resources), starting with a hidden cursor is better as MUS can
control the cursor interaction end-to-end.

Also, when running MUS as the display manager, MUS is in charge
of setting the default cursor, so there is no underlying
window manager to set the cursor initially (the initial cursor
state is invisible).

On multi-display configurations the expected behavior is to
start with a hidden cursor on new displays.

BUG= 600811 

Review URL: https://codereview.chromium.org/1869463002

Cr-Commit-Position: refs/heads/master@{#385809}

[modify] https://crrev.com/c82ddf12fd204b7ad9d684049f32ee42952a66f7/components/mus/ws/display.cc

Status: Fixed (was: Assigned)
Labels: VerifyIn-51
Status: Verified (was: Fixed)
Bulk verified
Components: Internals>Services>WindowService
Components: -MUS>Phase>1

Sign in to add a comment