X11: Better timestamp handling for _NET_ACTIVE_WINDOW

ui/events/platform/x11/x11_event_source.cc

 // Copyright 2014 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 "ui/events/platform/x11/x11_event_source.h"
 
+#include <X11/Xatom.h>
 #include <X11/XKBlib.h>
 #include <X11/Xlib.h>
 
 #include "base/logging.h"
+#include "base/metrics/histogram_macros.h"
 #include "ui/events/devices/x11/device_data_manager_x11.h"
 #include "ui/events/devices/x11/touch_factory_x11.h"
 #include "ui/events/event_utils.h"
 #include "ui/events/platform/platform_event_dispatcher.h"
 #include "ui/events/platform/x11/x11_hotplug_event_handler.h"
 
 namespace ui {
 
 namespace {
 
@@ skipping 50 matching lines @@
   return CurrentTime;
 }
 
 void UpdateDeviceList() {
   XDisplay* display = gfx::GetXDisplay();
   DeviceListCacheX11::GetInstance()->UpdateDeviceList(display);
   TouchFactory::GetInstance()->UpdateDeviceList(display);
   DeviceDataManagerX11::GetInstance()->UpdateDeviceList(display);
 }
 
+Bool IsPropertyNotifyForTimestamp(Display* display,
+                                  XEvent* event,
+                                  XPointer arg) {
+  return event->type == PropertyNotify &&
+         event->xproperty.window == DefaultRootWindow(display) &&
+         event->xproperty.atom == *(Atom*)arg;
+}
+
 }  // namespace
 
 X11EventSource* X11EventSource::instance_ = nullptr;
 
 X11EventSource::X11EventSource(X11EventSourceDelegate* delegate,
                                XDisplay* display)
     : delegate_(delegate),
       display_(display),
       last_seen_server_time_(CurrentTime),
       continue_stream_(true) {
@@ skipping 36 matching lines @@
 void X11EventSource::BlockUntilWindowMapped(XID window) {
   XEvent event;
   do {
     // Block until there's a message of |event_mask| type on |w|. Then remove
     // it from the queue and stuff it in |event|.
     XWindowEvent(display_, window, StructureNotifyMask, &event);
     ExtractCookieDataDispatchEvent(&event);
   } while (event.type != MapNotify);
 }
 
+Time X11EventSource::UpdateLastSeenServerTime() {
+  base::TimeTicks start = base::TimeTicks::Now();
+
+  const char kAtomStr[] = "X11_EVENT_SOURCE_UPDATE_SERVER_TIME_DUMMY_ATOM";
+
+  DCHECK(display_);
+  Window window = DefaultRootWindow(display_);
+  const unsigned char dummy_data = 0;
+  Atom dummy_atom = XInternAtom(display_, kAtomStr, False);
+
+  XWindowAttributes attributes;
+  XGetWindowAttributes(display_, window, &attributes);

[danakj, 2016/05/12 21:32:50]

Round trips :(

Can you just use some arbitrary never-mapped window to do this timestamp
generation instead of the root?

+  long old_event_mask = attributes.your_event_mask;
+
+  // Make a no-op property change on the root window.
+  XSelectInput(display_, window, old_event_mask | PropertyChangeMask);
+  XChangeProperty(display_, window, dummy_atom, XA_INTEGER, 8, PropModeReplace,
+                  &dummy_data, 1);
+
+
+  // Observe the resulting PropertyChange to obtain the timestamp.
+  XEvent event;
+  XIfEvent(display_, &event, IsPropertyNotifyForTimestamp,
+           (XPointer)&dummy_atom);
+
+  // Clean up: don't leave a dummy property in the root window.
+  XSelectInput(display_, window, old_event_mask);
+  XDeleteProperty(display_, window, dummy_atom);
+  XFlush(display_);
+
+  last_seen_server_time_ = event.xproperty.time;
+
+  UMA_HISTOGRAM_TIMES("Event.Latency.X11EventSource.UpdateServerTime",
+                      base::TimeTicks::Now() - start);
+  return last_seen_server_time_;
+}
+
 ////////////////////////////////////////////////////////////////////////////////
 // X11EventSource, protected
 
 void X11EventSource::ExtractCookieDataDispatchEvent(XEvent* xevent) {
   bool have_cookie = false;
   if (xevent->type == GenericEvent &&
       XGetEventData(xevent->xgeneric.display, &xevent->xcookie)) {
     have_cookie = true;
   }
   Time event_time = ExtractTimeFromXEvent(*xevent);
@@ skipping 36 matching lines @@
 
 void X11EventSource::OnDispatcherListChanged() {
   if (!hotplug_event_handler_) {
     hotplug_event_handler_.reset(new X11HotplugEventHandler());
     // Force the initial device query to have an update list of active devices.
     hotplug_event_handler_->OnHotplugEvent();
   }
 }
 
 }  // namespace ui