New issue
Advanced search Search tips

Issue 780637 link

Starred by 1 user

Issue metadata

Status: Untriaged
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug

Blocking:
issue 693114



Sign in to add a comment

Mash: Implent CursorManagerTestApi

Project Member Reported by steve...@chromium.org, Nov 1 2017

Issue description

The following ash unittests rely on CursorManagerTestApi:

ash/display/mirror_window_controller_unittest.cc
ash/display/mouse_cursor_event_filter_unittest.cc
ash/display/window_tree_host_manager_unittest.cc
ash/wm/drag_window_resizer_unittest.cc
ash/wm/resize_shadow_and_cursor_unittest

It seems like disabling these tests is incorrect, we should probably modify CursorManagerTestApi to not rely on a global ::wm::CursorManager* in its constructor but instead construct and implement it for mash.


 
Components: Tests>Disabled
Labels: Test-Disabled
Some of the disabled tests (for searchability):

-WindowManagerTest.ActivateOnMouse
-WindowManagerTest.ActivateOnPointerWindowProperty
-WindowManagerTest.ActivateOnTouch
-WindowManagerTest.AdditionalFilters
-WindowManagerTest.Focus
-WindowManagerTest.MouseEventCursors
-WindowManagerTest.PanelActivation
-WindowManagerTest.TestCursorClientObserver
-WindowManagerTest.TransformActivate
-WindowManagerTest.UpdateCursorVisibility
-WindowManagerTest.UpdateCursorVisibilityAccelerator
-WindowManagerTest.UpdateCursorVisibilityOnKeyEvent
-WindowTreeHostManagerTest.DontUpdateInvisibleCursorLocationAfterDisplayChange
-WindowTreeHostManagerTest.UpdateMouseLocationAfterDisplayChange
-WindowTreeHostManagerTest.UpdateMouseLocationAfterDisplayChange_2ndOnLeft
-WindowTreeHostManagerTest.UpdateMouseLocationAfterDisplayChange_PrimaryDisconnected
-WindowTreeHostManagerTest.UpdateMouseLocationAfterDisplayChange_SwapPrimary
-WorkspaceWindowResizerTest.MouseMoveWithTouchDrag

Comment 4 by e...@chromium.org, Mar 9 2018

Cc: -e...@chromium.org
Un-cc-ing me from all bugs on my final day.
Components: Internals>Services>Ash
Labels: -Proj-Mustash-Mash

Sign in to add a comment