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

Unified Diff: ui/ozone/platform/x11/ozone_platform_x11.cc

Issue 2629983002: Fix the MessageLoop type in case more than one ozone platform is built (Closed)
Patch Set: addressed moar sadrul/kylechar's feedback Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ui/ozone/platform/x11/ozone_platform_x11.cc
diff --git a/ui/ozone/platform/x11/ozone_platform_x11.cc b/ui/ozone/platform/x11/ozone_platform_x11.cc
index 6db5b67b1b99ea898677884c378241b7311c4086..e3e085563c8bb2be303194d5aa88be1e808634e4 100644
--- a/ui/ozone/platform/x11/ozone_platform_x11.cc
+++ b/ui/ozone/platform/x11/ozone_platform_x11.cc
@@ -104,6 +104,12 @@ class OzonePlatformX11 : public OzonePlatform {
surface_factory_ozone_.reset(new X11SurfaceFactory());
}
+ base::MessageLoop::Type GetMessageLoopTypeForGpu() override {
+ // If we are be running Ozone X11 we need a UI loop to grab Expose events.
+ // See GLSurfaceGLX and https://crbug.com/326995.
fwang 2017/02/07 17:03:46 "we are running" grab or Expose?
+ return base::MessageLoop::TYPE_UI;
+ }
+
private:
// Objects in the browser process.
std::unique_ptr<X11WindowManagerOzone> window_manager_;

Powered by Google App Engine
This is Rietveld 408576698