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

Issue 693340 link

Starred by 2 users

Issue metadata

Status: Archived
Owner:
Last visit > 30 days ago
Closed: Jun 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Implement WindowTreeHostMus::MoveCursorToScreenLocationInPixels()

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

Issue description

This will need to bounce to the server.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Feb 17 2017

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

commit 1a9a22346bc6a98adcf83734b97a6ca88f493606
Author: sky <sky@chromium.org>
Date: Fri Feb 17 21:12:43 2017

chromeos: add TODO for implementing MoveCursorToScreenLocationInPixels

In WindowTreeHostMus. This appeases some tests, but really needs to go
to the server.

BUG= 693340 
TEST=none
R=msw@chromium.org

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

[modify] https://crrev.com/1a9a22346bc6a98adcf83734b97a6ca88f493606/ui/aura/mus/window_tree_host_mus.cc
[modify] https://crrev.com/1a9a22346bc6a98adcf83734b97a6ca88f493606/ui/aura/mus/window_tree_host_mus.h

Owner: sky@chromium.org
Status: Assigned (was: Untriaged)

Comment 3 by sky@chromium.org, Feb 21 2017

Owner: ----
Status: Available (was: Assigned)
I'm removing myself as an owner as someone else on the team may get to it before myself.

Comment 4 by sky@chromium.org, May 8 2017

Owner: e...@chromium.org
Status: Assigned (was: Available)
Project Member

Comment 5 by bugdroid1@chromium.org, Jun 2 2017

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

commit 29e1c03f3e5d909b2bd2a784a78bf3580bed4ccd
Author: erg <erg@chromium.org>
Date: Fri Jun 02 20:01:15 2017

Implement a MoveCursorToScreenLocation for just the window manager.

While we don't want to expose this to any mus app (which could be
hostile), we still need a way for window management tasks to move the
cursor to specific locations.

BUG= 693340 

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

[modify] https://crrev.com/29e1c03f3e5d909b2bd2a784a78bf3580bed4ccd/services/ui/public/interfaces/window_manager.mojom
[modify] https://crrev.com/29e1c03f3e5d909b2bd2a784a78bf3580bed4ccd/services/ui/ws/platform_display.h
[modify] https://crrev.com/29e1c03f3e5d909b2bd2a784a78bf3580bed4ccd/services/ui/ws/platform_display_default.cc
[modify] https://crrev.com/29e1c03f3e5d909b2bd2a784a78bf3580bed4ccd/services/ui/ws/platform_display_default.h
[modify] https://crrev.com/29e1c03f3e5d909b2bd2a784a78bf3580bed4ccd/services/ui/ws/test_utils.cc
[modify] https://crrev.com/29e1c03f3e5d909b2bd2a784a78bf3580bed4ccd/services/ui/ws/window_manager_state.cc
[modify] https://crrev.com/29e1c03f3e5d909b2bd2a784a78bf3580bed4ccd/services/ui/ws/window_manager_state.h
[modify] https://crrev.com/29e1c03f3e5d909b2bd2a784a78bf3580bed4ccd/services/ui/ws/window_tree.cc
[modify] https://crrev.com/29e1c03f3e5d909b2bd2a784a78bf3580bed4ccd/services/ui/ws/window_tree.h
[modify] https://crrev.com/29e1c03f3e5d909b2bd2a784a78bf3580bed4ccd/services/ui/ws/window_tree_unittest.cc
[modify] https://crrev.com/29e1c03f3e5d909b2bd2a784a78bf3580bed4ccd/ui/aura/mus/window_tree_client.cc
[modify] https://crrev.com/29e1c03f3e5d909b2bd2a784a78bf3580bed4ccd/ui/aura/mus/window_tree_client.h
[modify] https://crrev.com/29e1c03f3e5d909b2bd2a784a78bf3580bed4ccd/ui/aura/mus/window_tree_host_mus.cc
[modify] https://crrev.com/29e1c03f3e5d909b2bd2a784a78bf3580bed4ccd/ui/aura/mus/window_tree_host_mus_delegate.h
[modify] https://crrev.com/29e1c03f3e5d909b2bd2a784a78bf3580bed4ccd/ui/aura/test/mus/test_window_manager_client.cc
[modify] https://crrev.com/29e1c03f3e5d909b2bd2a784a78bf3580bed4ccd/ui/aura/test/mus/test_window_manager_client.h

Comment 6 by e...@chromium.org, Jun 16 2017

Status: Fixed (was: Assigned)
Fixed in the window manager case (and deliberately left unimplemented for unprivileged clients, as we don't want them to muck with the cursor position).

Comment 7 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Sign in to add a comment