1
0
Fork 0

clutter/stage: Always queue events in _clutter_stage_queue_event()

We've been sending all events to clients immediately for quite some time
now, so this is only really impacting the Clutter scene graph, not
clients anymore.

That makes this behavior a somewhat unnecessary optimization (it was
useful at the time it was added, but it's not anymore), which will only
make our lives harder when we actually expect an event to be queued
(eg. in tests), so remove it.

Part-of: <https://gitlab.gnome.org/GNOME/mutter/-/merge_requests/2697>
This commit is contained in:
Jonas Dreßler 2022-11-16 14:32:39 +01:00 committed by Marge Bot
parent 0d0a6ece32
commit fe01e423a3

View file

@ -611,23 +611,8 @@ _clutter_stage_queue_event (ClutterStage *stage,
first_event = priv->event_queue->length == 0;
if (copy_event)
event = clutter_event_copy (event);
if (first_event)
{
gboolean compressible = event->type == CLUTTER_MOTION ||
event->type == CLUTTER_TOUCH_UPDATE;
if (!compressible)
{
_clutter_process_event (event);
clutter_event_free (event);
return;
}
}
g_queue_push_tail (priv->event_queue, event);
g_queue_push_tail (priv->event_queue,
copy_event ? clutter_event_copy (event) : event);
if (first_event)
clutter_stage_schedule_update (stage);