clutter/stage: Remove redraw_pending and redraw_count

The redraw_pending boolean is used to schedule the first stage update
after starting Clutter. This flag is superfluous because we have the
pending_finish_queue_redraws flag which does the same.

While at it, also remove the redraw_count debug variable, since there
should be better ways to count the number of queued redraws nowadays,
for example Sysprof marks.

https://gitlab.gnome.org/GNOME/mutter/-/merge_requests/1527
This commit is contained in:
Jonas Dreßler 2020-10-19 12:48:39 +02:00
parent 10392359a9
commit 84dd6b4af5

View File

@ -128,10 +128,6 @@ struct _ClutterStagePrivate
ClutterPickStack *pick_stack;
ClutterPickMode cached_pick_mode;
#ifdef CLUTTER_ENABLE_DEBUG
gulong redraw_count;
#endif /* CLUTTER_ENABLE_DEBUG */
ClutterStageState current_state;
int update_freeze_count;
@ -139,7 +135,6 @@ struct _ClutterStagePrivate
gboolean needs_update_devices;
gboolean pending_finish_queue_redraws;
guint redraw_pending : 1;
guint throttle_motion_events : 1;
guint min_size_changed : 1;
guint motion_events_enabled : 1;
@ -2813,23 +2808,6 @@ _clutter_stage_queue_actor_redraw (ClutterStage *stage,
priv->pending_finish_queue_redraws = TRUE;
}
if (!priv->redraw_pending)
{
CLUTTER_NOTE (PAINT, "First redraw request");
clutter_stage_schedule_update (stage);
priv->redraw_pending = TRUE;
}
#ifdef CLUTTER_ENABLE_DEBUG
else
{
CLUTTER_NOTE (PAINT, "Redraw request number %lu",
priv->redraw_count + 1);
priv->redraw_count += 1;
}
#endif /* CLUTTER_ENABLE_DEBUG */
if (entry)
{
/* Ignore all requests to queue a redraw for an actor if a full