Chromium Code Reviews
Help | Chromium Project | Gerrit Changes | Sign in

Issue 2313353003: Ozone/Wayland: Move to newer APIs and improve GPU initialization (Closed)

1 year, 4 months ago by fwang
1 year, 4 months ago
chromium-reviews, kalyank,, spang, kylechar, not to use - tonikitoo
Target Ref:


Ozone/Wayland: Move to newer APIs and improve GPU initialization The Wayland platform for Ozone currently implements the following deprecated API from SurfaceOzoneCanvas: * CreateViewGLSurface * CreateOffscreenGLSurface * GetNativeDisplay * LoadEGLGLES2Bindings That code is moved into a separate GLOzoneEGLWayland helper class implementing GLOzoneEGL. This also allows to better handle the fact that the support is restricted to the kGLImplementationEGLGLES2 implementation. Also OzonePlatformWayland is updated to prepare support for the newer variants of InitializeForUI and InitializeGPU with initialization parameters, even if every callers do not set these parameters correctly at the moment. Currently, InitializeGPU checks whether surface_factory_ has been initialized and otherwise creates a WaylandSurfaceFactory with a null Wayland connection. Some comments are added to clarify these two workarounds. In particular, the verification could be made clearer with args.single_process when the migration to the new InitializeGPU variant is complete. Additional changes better protect the part of WaylandSurfaceFactory that assumes a non-null Wayland connection and ensure that we always fullfill this assumption inside the WaylandSurfaceFactory and WaylandCanvasSurface helper classes. BUG=none Committed: Cr-Commit-Position: refs/heads/master@{#417528}

Patch Set 1 #


Total messages: 12 (7 generated)
Thanks. lgtm.
1 year, 4 months ago (2016-09-08 22:47:14 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at
1 year, 4 months ago (2016-09-09 07:02:06 UTC) #6
commit-bot: I haz the power
Committed patchset #1 (id:1)
1 year, 4 months ago (2016-09-09 07:40:15 UTC) #9
commit-bot: I haz the power
Patchset 1 (id:??) landed as Cr-Commit-Position: refs/heads/master@{#417528}
1 year, 4 months ago (2016-09-09 07:41:42 UTC) #11
1 year, 4 months ago (2016-09-09 07:48:47 UTC) #12
Message was sent while issue was closed.
For the record, the GLOzone interface interface was introduced in

Powered by Google App Engine
This is Rietveld 0eb02b776