lib: consolidate change notification

if the daemon sends out multiple change notifications for a user
because several properties changed in quick succession, try to
batch them up, and only update info at the end.

https://bugs.freedesktop.org/show_bug.cgi?id=103488

1 files changed, 19 insertions, 1 deletions

diff --git a/src/libaccountsservice/act-user.c b/src/libaccountsservice/act-user.c
index 300118a..d26b46a 100644
--- a/src/libaccountsservice/act-user.c
+++ b/src/libaccountsservice/act-user.c
@@ -140,6 +140,8 @@ struct _ActUser {
         guint           system_account : 1;
         guint           local_account : 1;
         guint           nonexistent : 1;
+
+        guint           update_info_timeout_id;
 };
 
 struct _ActUserClass
@@ -600,6 +602,10 @@ act_user_finalize (GObject *object)
                 g_object_unref (user->connection);
         }
 
+        if (user->update_info_timeout_id != 0) {
+                g_source_remove (user->update_info_timeout_id);
+        }
+
         if (G_OBJECT_CLASS (act_user_parent_class)->finalize)
                 (*G_OBJECT_CLASS (act_user_parent_class)->finalize) (object);
 }
@@ -1366,13 +1372,25 @@ update_info (ActUser *user)
                            user);
 }
 
+static gboolean
+on_timeout_update_info (ActUser *user)
+{
+        update_info (user);
+        user->update_info_timeout_id = 0;
+
+        return G_SOURCE_REMOVE;
+}
+
 static void
 changed_handler (AccountsUser *object,
                  gpointer   *data)
 {
         ActUser *user = ACT_USER (data);
 
-        update_info (user);
+        if (user->update_info_timeout_id != 0)
+                return;
+
+        user->update_info_timeout_id = g_timeout_add (250, (GSourceFunc) on_timeout_update_info, user);
 }
 
 /**