touchui: support XInput2 MT

base/message_pump_x.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/message_pump_x.h"
 
 #include <gdk/gdkx.h>
 #include <X11/extensions/XInput2.h>
 
 #include "base/basictypes.h"
@@ skipping 52 matching lines @@
     return;
 
   int event, err;
 
   if (!XQueryExtension(display, "XInputExtension", &xiopcode, &event, &err)) {
     VLOG(1) << "X Input extension not available.";
     xiopcode = -1;
     return;
   }
 
+#if defined(USE_XI2_MT)
+  int major = 2, minor = 1;

[sadrul, 2011/09/19 15:58:08]

Given that the official XInput2.1 will not have the MT support, I wonder if we
should use 2.2 here, and Ubuntu (and/or other distros) carries a patch that
enables it in 2.1, or if we should just use 2.1 as it is in this CL, and give it
a second thought if/when we need to. Daniel?

[Daniel Kurtz, 2011/09/20 05:12:27]

I agree with sadrul.  Please make 2.2 by default.
Perhaps we could pass in the minimum minor version supported via a build-time
flag... like the value of USE_XI2_MT_MINOR?

[ningxin.hu, 2011/09/20 14:28:59]

Thanks for comments. Will use USE_XI2_MT as minimum required XI2 minor version.

+#else
   int major = 2, minor = 0;
+#endif
   if (XIQueryVersion(display, &major, &minor) == BadRequest) {
     VLOG(1) << "XInput2 not supported in the server.";
     xiopcode = -1;
     return;
   }
 }
 
 }  // namespace
 
 namespace base {
@@ skipping 196 matching lines @@
 }
 
 MessagePumpObserver::EventStatus
     MessagePumpObserver::WillProcessXEvent(XEvent* xev) {
   return EVENT_CONTINUE;
 }
 
 COMPILE_ASSERT(XLASTEvent >= LASTEvent, XLASTEvent_too_small);
 
 }  // namespace base