New issue
Advanced search Search tips

Issue 709225 link

Starred by 1 user

Issue metadata

Status: Archived
Owner:
Closed: Apr 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

mushrome: Fix exo WMHelperAsh usage of ui::DeviceDataManager

Project Member Reported by jamescook@chromium.org, Apr 6 2017

Issue description

Under mushrome there's no ui::DeviceDataManager but WMHelperAsh uses one to detect keyboard configuration changes.

I'm not familiar enough with DDM to know what the right solution is here.

 
Cc: -penghuang@chromium.org
Owner: penghuang@chromium.org
Status: Assigned (was: Untriaged)
Project Member

Comment 2 by bugdroid1@chromium.org, Apr 26 2017

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

commit 807942831018b22765df0b45bbd0d0d7deec27c1
Author: penghuang <penghuang@chromium.org>
Date: Wed Apr 26 21:49:37 2017

Mushrome: Use ui::InputDeviceManager in exo.

The ui::InputDeviceManager implementation ui::DeviceDataManager is not
available in Mushrome, but another implementation ui::InputDeviceClient
is avaliable in Mushrome. So we should use
ui::InputDeviceManager::GetInstance() which will return correct
implementation for Cash and Mushrome instead of using
ui::DeviceDataManager::GetInstance().

BUG= 709225 

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

[modify] https://crrev.com/807942831018b22765df0b45bbd0d0d7deec27c1/components/exo/wm_helper_ash.cc

Status: Fixed (was: Assigned)

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

Labels: VerifyIn-61

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

Status: Archived (was: Fixed)

Sign in to add a comment