New issue
Advanced search Search tips

Issue 706474 link

Starred by 2 users

Issue metadata

Status: Archived
Owner:
Closed: Jun 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug

Blocked on:
issue 718860

Blocking:
issue 714150



Sign in to add a comment

Evaluate if MouseCursorEventFilter needs to be made to work for mash

Project Member Reported by sky@chromium.org, Mar 29 2017

Issue description

It's used for unified and extend desktop modes, and may need to move to mus.
 
Labels: mustash-display

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

Blocking: 714150
Project Member

Comment 3 by bugdroid1@chromium.org, May 5 2017

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

commit a5888e81e68aff997094e675cccfe09c559d5f4b
Author: sky <sky@chromium.org>
Date: Fri May 05 17:56:31 2017

chromeos: enable MouseCursorEventFilter for mushrome

Now that mushrome creates a WindowTreeTreeHostManager it can create a
MouseCursorEventFilter. Also updates test expectation as a bunch of
tests were relying on this.

BUG= 714150 , 706474 
TEST=covered by tests
R=jamescook@chromium.org

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

[modify] https://crrev.com/a5888e81e68aff997094e675cccfe09c559d5f4b/ash/shell.cc
[modify] https://crrev.com/a5888e81e68aff997094e675cccfe09c559d5f4b/testing/buildbot/filters/ash_mus_unittests.filter

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

Summary: Evaluate if MouseCursorEventFilter needs to be made to work for mash (was: Evaluate if MouseCursorEventFilter needs to be made to work for mus/mash)
MouseCursorEventFilter is now enabled in mushrome, so I'm updating the title to reflect we need to figure this out for mash now.
I suspect we'll need MouseCursorEventFilter for mash as well, but that will require some restructuring, and is blocked on getting mash to support simplified display management.

Comment 5 by sky@chromium.org, May 5 2017

Blockedon: 718860

Comment 6 by sky@chromium.org, Jun 21 2017

Owner: sky@chromium.org
Status: Fixed (was: Untriaged)
Simplified display management means this is now enabled.

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

Status: Archived (was: Fixed)

Sign in to add a comment