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

Issue 615861 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: Dec 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug
mus

Blocked on:
issue 639039



Sign in to add a comment

Cursor positioning seem a little off

Project Member Reported by sadrul@chromium.org, May 30 2016

Issue description

Cursor positioning seems a little off in mash-chrome. For example, if you click on http://rbyers.net/paint.html, you will notice the cursor shows up a few pixels to the left of where the event is actually going (i.e. where the cursor is supposed to be).
 

Comment 1 by staraz@chromium.org, Aug 15 2016

Owner: staraz@chromium.org

Comment 2 by staraz@chromium.org, Aug 24 2016

Blockedon: 639039
Status: Started (was: Available)
Components: Internals>MUS
Labels: Proj-Mustash
Project Member

Comment 4 by bugdroid1@chromium.org, Dec 16 2016

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

commit 41725e3dd88eadb07f8a1297c0f8f6ecb9b30c79
Author: staraz <staraz@chromium.org>
Date: Fri Dec 16 18:26:27 2016

Replaced CursorLoader in PlatformDisplay with a ImageCursor

ImageCursor->SetDisplay() is called in Init() so that all cursors are loaded at
launch. This fixes the bug where the cursor image is positioned off to the left
in Chrome mash.

BUG= 615861 

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

[modify] https://crrev.com/41725e3dd88eadb07f8a1297c0f8f6ecb9b30c79/services/ui/ws/display.cc
[modify] https://crrev.com/41725e3dd88eadb07f8a1297c0f8f6ecb9b30c79/services/ui/ws/display.h
[modify] https://crrev.com/41725e3dd88eadb07f8a1297c0f8f6ecb9b30c79/services/ui/ws/platform_display_default.cc
[modify] https://crrev.com/41725e3dd88eadb07f8a1297c0f8f6ecb9b30c79/services/ui/ws/platform_display_default.h
[modify] https://crrev.com/41725e3dd88eadb07f8a1297c0f8f6ecb9b30c79/services/ui/ws/platform_display_delegate.h

Comment 5 by staraz@chromium.org, Dec 16 2016

Status: Fixed (was: Started)

Comment 6 by dchan@google.com, Mar 4 2017

Labels: VerifyIn-58

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

Labels: VerifyIn-59

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

Labels: VerifyIn-60

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

Labels: VerifyIn-61
Status: Verified (was: Fixed)
Verified on build 9817.0.0
Components: -Internals>MUS Internals>Services>WindowService
Components: -MUS

Sign in to add a comment