Implement a single interface for ozone-gbm clients to configure displays |
|||||||
Issue descriptionChrome OS Version: Trunk In theory, applications with Ozone in Chromium can choose its backends such as X11, DRM/GBM, Wayland, or headless at runtime. However, if we build content_shell with ozone-gbm(DRM/GBM) backend, it does not work well because it does not implement a display configurator to configure displays, but also needs to manage multiple displays. Those display configuration/management features are tightly coupled with ChromeOS so we need to separate those features from ChromeOS so that any ozone-gbm clients could use them more easily. First, we need to implement a single Interface for Display Configuration and Management so ui::DisplayManager can be the single interface for ozone-gbm clients by moving DisplayConfigurator and DisplayChangeObserver under DisplayManager, which can simplify the relationship between CrOS shell and ui/Display/Manager. Ozone-gbm clients can easily use display configuration/management features by only accessing DisplayManager. In addition, this change will allow ui::DisplayManager to directly use ui::DisplayConfigurator without ash::WindowTreeHostManager. For more details, see the design document at http://bit.ly/2FBXL2c
,
Jan 15
,
Jan 16
(6 days ago)
,
Jan 16
(6 days ago)
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/74b3398b9b828fa9517c79bee0df9fe0680183eb commit 74b3398b9b828fa9517c79bee0df9fe0680183eb Author: Joone Hur <joone.hur@intel.com> Date: Wed Jan 16 20:57:48 2019 Remove DisplayConfigurator dependency from TouchTransformController DisplayConfigurator dependency should be removed since it is not used in TouchTransformController. Bug: 922182 Change-Id: I49818325a3fd8601271dbb53e3cdcb0ddd5bc39b Reviewed-on: https://chromium-review.googlesource.com/c/1412837 Reviewed-by: Mitsuru Oshima <oshima@chromium.org> Commit-Queue: Joone Hur <joone.hur@intel.com> Cr-Commit-Position: refs/heads/master@{#623351} [modify] https://crrev.com/74b3398b9b828fa9517c79bee0df9fe0680183eb/ash/shell.cc [modify] https://crrev.com/74b3398b9b828fa9517c79bee0df9fe0680183eb/ash/touch/ash_touch_transform_controller.cc [modify] https://crrev.com/74b3398b9b828fa9517c79bee0df9fe0680183eb/ash/touch/ash_touch_transform_controller.h [modify] https://crrev.com/74b3398b9b828fa9517c79bee0df9fe0680183eb/ui/display/manager/touch_transform_controller.cc [modify] https://crrev.com/74b3398b9b828fa9517c79bee0df9fe0680183eb/ui/display/manager/touch_transform_controller.h [modify] https://crrev.com/74b3398b9b828fa9517c79bee0df9fe0680183eb/ui/display/manager/touch_transform_controller_unittest.cc
,
Jan 16
(6 days ago)
,
Jan 17
(5 days ago)
,
Jan 19
(3 days ago)
,
Jan 19
(3 days ago)
|
|||||||
►
Sign in to add a comment |
|||||||
Comment 1 by joone....@intel.com
, Jan 15