[mutter] window: Treat CurrentTime as legal timestamp in activation

[Florian Müllner <fmuellner src gnome org>]

commit 5defe574d74eace55bfe9e7e79ffaaa40f50372f
Author: Florian Müllner <fmuellner gnome org>
Date:   Fri Apr 11 19:05:20 2014 +0200

    window: Treat CurrentTime as legal timestamp in activation
    
    Effectively we have been accepting CurrentTime timestamps for years,
    but still complained about "stupid pagers" when encountering them;
    just accept that we will never limit treating 0 timestamps as current
    time to pagers.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=728018

 src/core/window.c |   21 +++------------------
 1 files changed, 3 insertions(+), 18 deletions(-)
---
diff --git a/src/core/window.c b/src/core/window.c
index c9494be..ed09047 100644
--- a/src/core/window.c
+++ b/src/core/window.c
@@ -3703,23 +3703,13 @@ meta_window_activate_full (MetaWindow     *window,
                            MetaClientType  source_indication,
                            MetaWorkspace  *workspace)
 {
-  gboolean can_ignore_outdated_timestamps;
   meta_topic (META_DEBUG_FOCUS,
               "_NET_ACTIVE_WINDOW message sent for %s at time %u "
               "by client type %u.\n",
               window->desc, timestamp, source_indication);
 
-  /* Older EWMH spec didn't specify a timestamp; we decide to honor these only
-   * if the app specifies that it is a pager.
-   *
-   * Update: Unconditionally honor 0 timestamps for now; we'll fight
-   * that battle later.  Just remove the "FALSE &&" in order to only
-   * honor 0 timestamps for pagers.
-   */
-  can_ignore_outdated_timestamps =
-    (timestamp != 0 || (FALSE && source_indication != META_CLIENT_TYPE_PAGER));
-  if (XSERVER_TIME_IS_BEFORE (timestamp, window->display->last_user_time) &&
-      can_ignore_outdated_timestamps)
+  if (timestamp != 0 &&
+      XSERVER_TIME_IS_BEFORE (timestamp, window->display->last_user_time))
     {
       meta_topic (META_DEBUG_FOCUS,
                   "last_user_time (%u) is more recent; ignoring "
@@ -3729,13 +3719,8 @@ meta_window_activate_full (MetaWindow     *window,
       return;
     }
 
-  /* For those stupid pagers, get a valid timestamp and show a warning */
   if (timestamp == 0)
-    {
-      meta_warning ("meta_window_activate called by a pager with a 0 timestamp; "
-                    "the pager needs to be fixed.\n");
-      timestamp = meta_display_get_current_time_roundtrip (window->display);
-    }
+    timestamp = meta_display_get_current_time_roundtrip (window->display);
 
   meta_window_set_user_time (window, timestamp);