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

Issue 2389053003: [ozone/wayland] Use FakeDisplayDelegate in order to make "off device" launch work (Closed)

Created:
4 years, 2 months ago by tonikitoo
Modified:
4 years, 2 months ago
Reviewers:
rjkroege, kylechar, sadrul, spang
CC:
chromium-reviews, kalyank, ozone-reviews_chromium.org, fwang
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

[ozone/wayland] Use FakeDisplayDelegate in order to make "off device" launch work [1] introduced the "fake display" concept, updating OzonePlatformHeadless and OzonePlatformX11 to use it, but not OzonePlatformWayland. As a result, "chrome --mash --ozone-platform=wayland" fails to launch. This CL fixes it by returning a FakeDisplayDelegate instance in OzonePlatformWayland::CreateNativeDisplayDelegate, similarly to other off-device capable ozone platforms. [1] https://codereview.chromium.org/2324163002 BUG=611475 Committed: https://crrev.com/170de2963d3e977eb340fc5e496b3624047018e9 Cr-Commit-Position: refs/heads/master@{#423254}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -2 lines) Patch
M ui/ozone/platform/wayland/ozone_platform_wayland.cc View 2 chunks +2 lines, -2 lines 0 comments Download

Messages

Total messages: 20 (9 generated)
tonikitoo
PTAL. CL updates OzonePlatformWayland::CreateNativeDisplayDelegate similarly to its OzonePlatformX11 and OzonePlatformHeadless counterparts, in order to support ...
4 years, 2 months ago (2016-10-04 04:27:27 UTC) #3
fwang
informal l g t m Also confirming that it allows to launch chrome --mash on ...
4 years, 2 months ago (2016-10-04 12:55:48 UTC) #4
kylechar
lgtm
4 years, 2 months ago (2016-10-04 13:51:36 UTC) #5
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/2389053003/1
4 years, 2 months ago (2016-10-04 13:54:14 UTC) #7
kylechar
You'll need rjkroege for OWNERS still, I don't have it for top level Ozone.
4 years, 2 months ago (2016-10-04 13:55:33 UTC) #8
tonikitoo
On 2016/10/04 13:55:33, kylechar wrote: > You'll need rjkroege for OWNERS still, I don't have ...
4 years, 2 months ago (2016-10-04 13:57:03 UTC) #10
sadrul
stamp lgtm
4 years, 2 months ago (2016-10-05 19:16:23 UTC) #13
rjkroege
lgtm
4 years, 2 months ago (2016-10-05 19:24:58 UTC) #14
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/2389053003/1
4 years, 2 months ago (2016-10-05 19:26:34 UTC) #16
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 2 months ago (2016-10-05 19:40:04 UTC) #18
commit-bot: I haz the power
4 years, 2 months ago (2016-10-05 19:42:32 UTC) #20
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/170de2963d3e977eb340fc5e496b3624047018e9
Cr-Commit-Position: refs/heads/master@{#423254}

Powered by Google App Engine
This is Rietveld 408576698