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

Issue 2773503003: Change Ozone X11 PlatformEventSource initialization. (Closed)

Created:
3 years, 9 months ago by kylechar
Modified:
3 years, 9 months ago
Reviewers:
sadrul
CC:
chromium-reviews, kalyank, ozone-reviews_chromium.org
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

Change Ozone X11 PlatformEventSource initialization. The GPU process PlatformEventSource is only there to listen for Expose XEvents. This isn't important for tests so we don't need to initialize the PlatfromEventSource in single process mode at all. BUG=664570 Review-Url: https://codereview.chromium.org/2773503003 Cr-Commit-Position: refs/heads/master@{#459500} Committed: https://chromium.googlesource.com/chromium/src/+/11d88d96ccf5f46266cddb5241528be6923f1f41

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+21 lines, -7 lines) Patch
M ui/ozone/platform/x11/ozone_platform_x11.cc View 3 chunks +21 lines, -7 lines 0 comments Download

Messages

Total messages: 8 (4 generated)
kylechar
3 years, 9 months ago (2017-03-23 14:39:49 UTC) #2
sadrul
lgtm
3 years, 9 months ago (2017-03-24 17:52:42 UTC) #3
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/2773503003/1
3 years, 9 months ago (2017-03-24 18:01:21 UTC) #5
commit-bot: I haz the power
3 years, 9 months ago (2017-03-24 18:58:24 UTC) #8
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/11d88d96ccf5f46266cddb524152...

Powered by Google App Engine
This is Rietveld 408576698