Fix the MessageLoop type in case more than one ozone platform is built

ui/ozone/public/ozone_platform.cc

Index: ui/ozone/public/ozone_platform.cc
diff --git a/ui/ozone/public/ozone_platform.cc b/ui/ozone/public/ozone_platform.cc
index 6f25a1b12ead6e06b2230a137bee70503b79d28c..c04cf2f5f4fc39c5a51375e28230a70cc52856de 100644
--- a/ui/ozone/public/ozone_platform.cc
+++ b/ui/ozone/public/ozone_platform.cc
@@ -104,4 +104,12 @@ IPC::MessageFilter* OzonePlatform::GetGpuMessageFilter() {
 void OzonePlatform::AddInterfaces(
     service_manager::InterfaceRegistry* registry) {}
 
+base::MessageLoop::Type OzonePlatform::GetMessageLoopType() {
+  // If we are be running Ozone X11 we need a UI loop to grab Expose events.
+  // See GLSurfaceGLX and https://crbug.com/326995.
+  return (!strcmp(GetOzonePlatformName(), "x11"))

[spang, 2017/01/19 16:31:55]

Don't use the platform name. Use a virtual function.

[kylechar, 2017/01/19 16:59:50]

Right, that's a better idea. We can't run OzonePlatform::InitializeForGPU()
before the main MessageLoop is created. Something will try and use the
MessageLoop then crash.

However, we can create the OzonePlatform instance without doing any
initialization if OzonePlatform::CreateInstance() is made public. It will get
still get initialized later and GetMessageLoopType() can be virtual.

[tonikitoo, 2017/02/07 16:18:37]

Done.

+             ? base::MessageLoop::TYPE_UI
+             : base::MessageLoop::TYPE_DEFAULT;
+}
+
 }  // namespace ui