X11: Better timestamp handling for _NET_ACTIVE_WINDOW

ui/events/platform/x11/x11_event_source.cc

Index: ui/events/platform/x11/x11_event_source.cc
diff --git a/ui/events/platform/x11/x11_event_source.cc b/ui/events/platform/x11/x11_event_source.cc
index 0319da91b56843494f6c487d3876f316098142fd..9c239ea3344e598c48be431e66c73870bd1115ca 100644
--- a/ui/events/platform/x11/x11_event_source.cc
+++ b/ui/events/platform/x11/x11_event_source.cc
@@ -4,10 +4,12 @@
 
 #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/event_utils.h"
 #include "ui/events/platform/platform_event_dispatcher.h"
@@ -126,6 +128,42 @@ void X11EventSource::BlockUntilWindowMapped(XID window) {
   } while (event.type != MapNotify);
 }
 
+static Bool is_property_notify_for_timestamp(Display* display,
+                                             XEvent* event,
+                                             XPointer arg) {
+  return event->type == PropertyNotify &&
+      event->xproperty.window == DefaultRootWindow(display) &&
+      event->xproperty.atom == *(Atom*)arg;
+}
+
+Time X11EventSource::UpdateServerTime() {
+  base::TimeTicks start = base::TimeTicks::Now();
+
+  const char kAtomStr[] = __FILE__ " CHROMIUM_DUMMY_ATOM";
+
+  DCHECK(display_);
+  Window window = DefaultRootWindow(display_);
+  const unsigned char dummyData = 0;
+  Atom dummyAtom = XInternAtom(display_, kAtomStr, False);
+
+  XSelectInput(display_, window, PropertyChangeMask);
+
+  XChangeProperty(display_, window, dummyAtom, XA_INTEGER, 8, PropModeReplace,
+                  &dummyData, 1);
+
+  XEvent event;
+  XIfEvent(display_, &event, is_property_notify_for_timestamp,
+           (XPointer)&dummyAtom);
+
+  XDeleteProperty(display_, window, dummyAtom);

[Tom (Use chromium acct), 2016/05/06 20:42:00]

For some reason, XDeleteProperty doesn't actually delete the property in the
root window (but I can delete it using xprop -root -remove "xyz").  Anyone know
why this might be happening?

[piman, 2016/05/06 21:23:00]

Maybe need an XFlush ?

[Tom (Use chromium acct), 2016/05/06 22:09:43]

That worked! Done.

+
+  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