Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(75)

Issue 2750173006: exo: Explicitly initialize Ozone in wayland clients. (Closed)

Created:
3 years, 9 months ago by Daniele Castagna
Modified:
3 years, 9 months ago
Reviewers:
kylechar, reveman
CC:
chromium-reviews
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

exo: Explicitly initialize Ozone in wayland clients. crrev.com/2749873002 moved Ozone GPU initialization away from gl init and it now need to be explcitly called before initializing GL. This CL explicitly initialize Ozone platform in exo wayland clients. TBR=reveman@chromium.org BUG= Review-Url: https://codereview.chromium.org/2750173006 Cr-Commit-Position: refs/heads/master@{#457792} Committed: https://chromium.googlesource.com/chromium/src/+/f26905d38e61b7ec3ea082814e3ef497f0450aaf

Patch Set 1 #

Patch Set 2 : Add client base ozone dep. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+9 lines, -1 line) Patch
M components/exo/wayland/BUILD.gn View 1 1 chunk +4 lines, -1 line 0 comments Download
M components/exo/wayland/clients/client_base.cc View 2 chunks +5 lines, -0 lines 0 comments Download

Messages

Total messages: 18 (13 generated)
Daniele Castagna
3 years, 9 months ago (2017-03-17 03:17:10 UTC) #4
kylechar
non-OWNERS lgtm.
3 years, 9 months ago (2017-03-17 14:59:20 UTC) #11
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2750173006/20001
3 years, 9 months ago (2017-03-17 16:14:46 UTC) #14
commit-bot: I haz the power
Committed patchset #2 (id:20001) as https://chromium.googlesource.com/chromium/src/+/f26905d38e61b7ec3ea082814e3ef497f0450aaf
3 years, 9 months ago (2017-03-17 16:33:29 UTC) #17
reveman
3 years, 9 months ago (2017-03-17 17:02:35 UTC) #18
Message was sent while issue was closed.
lgtm

Powered by Google App Engine
This is Rietveld 408576698