clutter: Remove CLUTTER_CULL_RESULT_PARTIAL

Culling paint volumes don't give this level of detail anymore, and in
fact knowing whether it was partially or fully in was only being used
in a debug path. For the purposes of culling, it doesn't matter if a
given actor is partially or completely inside the frustum; either way,
it must be painted.

https://gitlab.gnome.org/GNOME/mutter/-/merge_requests/1489
This commit is contained in:
Georges Basile Stavracas Neto 2020-10-09 19:24:34 -03:00
parent 80bd44cba0
commit 496aea51af
2 changed files with 1 additions and 5 deletions

View File

@ -3372,7 +3372,6 @@ _clutter_actor_paint_cull_result (ClutterActor *self,
case CLUTTER_CULL_RESULT_OUT: case CLUTTER_CULL_RESULT_OUT:
clutter_color_init (&color, 0, 0, 255, 255); clutter_color_init (&color, 0, 0, 255, 255);
break; break;
case CLUTTER_CULL_RESULT_PARTIAL:
default: default:
clutter_color_init (&color, 0, 255, 255, 255); clutter_color_init (&color, 0, 255, 255, 255);
break; break;
@ -3503,10 +3502,8 @@ cull_actor (ClutterActor *self,
switch (overlap) switch (overlap)
{ {
case CAIRO_REGION_OVERLAP_IN: case CAIRO_REGION_OVERLAP_IN:
*result_out = CLUTTER_CULL_RESULT_IN;
break;
case CAIRO_REGION_OVERLAP_PART: case CAIRO_REGION_OVERLAP_PART:
*result_out = CLUTTER_CULL_RESULT_PARTIAL; *result_out = CLUTTER_CULL_RESULT_IN;
break; break;
case CAIRO_REGION_OVERLAP_OUT: case CAIRO_REGION_OVERLAP_OUT:
*result_out = CLUTTER_CULL_RESULT_OUT; *result_out = CLUTTER_CULL_RESULT_OUT;

View File

@ -250,7 +250,6 @@ typedef enum _ClutterCullResult
CLUTTER_CULL_RESULT_UNKNOWN, CLUTTER_CULL_RESULT_UNKNOWN,
CLUTTER_CULL_RESULT_IN, CLUTTER_CULL_RESULT_IN,
CLUTTER_CULL_RESULT_OUT, CLUTTER_CULL_RESULT_OUT,
CLUTTER_CULL_RESULT_PARTIAL
} ClutterCullResult; } ClutterCullResult;
gboolean _clutter_has_progress_function (GType gtype); gboolean _clutter_has_progress_function (GType gtype);