[meta] Eliminate (or greatly simplify) components/mus/surfaces |
||||||||
Issue descriptionThere's a lot of code in mus that can likely be dedup'ed with Chrome with a little bit of effort as part of the display_compositor refactor: 1. TopLevelDisplayClient => OnscreenDisplayClient 2. DirectOutputSurface/DirectOutputSurfaceOzone => SurfaceDisplayOutputSurface. 3. SurfacesContextProvider => InProcessContextProvider
,
May 14 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/4ae9b136bbf91f081db79870a0ceee7d88b78464 commit 4ae9b136bbf91f081db79870a0ceee7d88b78464 Author: fsamuel <fsamuel@chromium.org> Date: Sat May 14 03:00:07 2016 Pull parts of TopLevelDisplayClient into DisplayCompositor This CL starts to expose the beginnings of the API DisplayCompositor should have: SubmitCompositorFrame RequestCopyOfOutput TODO: Hittest TopLevelDisplayClient becomes a very thin implementation of cc::DisplayClient and starts to resemble OnscreenDisplayClient. BUG= 610937 , 610935 Review-Url: https://codereview.chromium.org/1976663003 Cr-Commit-Position: refs/heads/master@{#393731} [modify] https://crrev.com/4ae9b136bbf91f081db79870a0ceee7d88b78464/components/mus/surfaces/BUILD.gn [add] https://crrev.com/4ae9b136bbf91f081db79870a0ceee7d88b78464/components/mus/surfaces/display_compositor.cc [add] https://crrev.com/4ae9b136bbf91f081db79870a0ceee7d88b78464/components/mus/surfaces/display_compositor.h [modify] https://crrev.com/4ae9b136bbf91f081db79870a0ceee7d88b78464/components/mus/surfaces/top_level_display_client.cc [modify] https://crrev.com/4ae9b136bbf91f081db79870a0ceee7d88b78464/components/mus/surfaces/top_level_display_client.h [modify] https://crrev.com/4ae9b136bbf91f081db79870a0ceee7d88b78464/components/mus/ws/platform_display.cc [modify] https://crrev.com/4ae9b136bbf91f081db79870a0ceee7d88b78464/components/mus/ws/platform_display.h [modify] https://crrev.com/4ae9b136bbf91f081db79870a0ceee7d88b78464/components/mus/ws/server_window_surface_manager.cc
,
May 17 2016
fsamuel@ finished?
,
May 18 2016
No, not even close :P
,
Oct 4 2016
,
Dec 1 2016
,
Dec 3 2016
I think it's fair to call this bug finished at this point. Mus uses InProcessContextProvider now, and Mus has a lot less Mus-only code. There's a lot more code sharing now.
,
Jun 13 2017
,
Feb 26 2018
,
Feb 26 2018
|
||||||||
►
Sign in to add a comment |
||||||||
Comment 1 by fsam...@chromium.org
, May 12 2016