renderer: Remove offload argument from render()

It turns out it's not needed, because all relevant information for
subsurfaces is already available by querying the subsurface itself.
This commit is contained in:
Benjamin Otte
2023-11-14 21:45:47 +01:00
parent f6b6176ec0
commit 1b08fda93e
8 changed files with 11 additions and 23 deletions

View File

@@ -919,8 +919,7 @@ gsk_broadway_renderer_add_node (GskRenderer *renderer,
static void
gsk_broadway_renderer_render (GskRenderer *renderer,
GskRenderNode *root,
const cairo_region_t *update_area,
GskOffload *offload)
const cairo_region_t *update_area)
{
GskBroadwayRenderer *self = GSK_BROADWAY_RENDERER (renderer);

View File

@@ -275,8 +275,7 @@ update_area_requires_clear (GdkSurface *surface,
static void
gsk_gl_renderer_render (GskRenderer *renderer,
GskRenderNode *root,
const cairo_region_t *update_area,
GskOffload *offload)
const cairo_region_t *update_area)
{
GskGLRenderer *self = (GskGLRenderer *)renderer;
cairo_region_t *render_region;
@@ -314,7 +313,7 @@ gsk_gl_renderer_render (GskRenderer *renderer,
clear_framebuffer = update_area_requires_clear (surface, render_region);
gsk_gl_driver_begin_frame (self->driver, self->command_queue);
job = gsk_gl_render_job_new (self->driver, &viewport, scale, render_region, 0, clear_framebuffer, offload);
job = gsk_gl_render_job_new (self->driver, &viewport, scale, render_region, 0, clear_framebuffer);
if (GSK_RENDERER_DEBUG_CHECK (GSK_RENDERER (self), FALLBACK))
gsk_gl_render_job_set_debug_fallback (job, TRUE);
gsk_gl_render_job_render (job, root);
@@ -400,7 +399,7 @@ gsk_gl_renderer_render_texture (GskRenderer *renderer,
&render_target))
{
gsk_gl_driver_begin_frame (self->driver, self->command_queue);
job = gsk_gl_render_job_new (self->driver, viewport, 1, NULL, render_target->framebuffer_id, TRUE, NULL);
job = gsk_gl_render_job_new (self->driver, viewport, 1, NULL, render_target->framebuffer_id, TRUE);
if (GSK_RENDERER_DEBUG_CHECK (GSK_RENDERER (self), FALLBACK))
gsk_gl_render_job_set_debug_fallback (job, TRUE);
gsk_gl_render_job_render_flipped (job, root);

View File

@@ -37,7 +37,6 @@
#include <gsk/gskroundedrectprivate.h>
#include <gsk/gskrectprivate.h>
#include <gsk/gskrendererprivate.h>
#include <gsk/gskoffloadprivate.h>
#include <math.h>
#include <string.h>
@@ -180,8 +179,6 @@ struct _GskGLRenderJob
* looking at the format of the framebuffer we are rendering on.
*/
int target_format;
GskOffload *offload;
};
typedef struct _GskGLRenderOffscreen
@@ -4622,8 +4619,7 @@ gsk_gl_render_job_new (GskGLDriver *driver,
float scale,
const cairo_region_t *region,
guint framebuffer,
gboolean clear_framebuffer,
GskOffload *offload)
gboolean clear_framebuffer)
{
const graphene_rect_t *clip_rect = viewport;
graphene_rect_t transformed_extents;
@@ -4659,7 +4655,6 @@ gsk_gl_render_job_new (GskGLDriver *driver,
job->scale_y = scale;
job->viewport = *viewport;
job->target_format = get_framebuffer_format (job->command_queue->context, framebuffer);
job->offload = offload;
gsk_gl_render_job_set_alpha (job, 1.0f);
gsk_gl_render_job_set_projection_from_rect (job, viewport, NULL);

View File

@@ -27,8 +27,7 @@ GskGLRenderJob *gsk_gl_render_job_new (GskGLDriver *dri
float scale,
const cairo_region_t *region,
guint framebuffer,
gboolean clear_framebuffer,
GskOffload *offload);
gboolean clear_framebuffer);
void gsk_gl_render_job_free (GskGLRenderJob *job);
void gsk_gl_render_job_render (GskGLRenderJob *job,
GskRenderNode *root);

View File

@@ -152,8 +152,7 @@ gsk_cairo_renderer_render_texture (GskRenderer *renderer,
static void
gsk_cairo_renderer_render (GskRenderer *renderer,
GskRenderNode *root,
const cairo_region_t *region,
GskOffload *offload)
const cairo_region_t *region)
{
GskCairoRenderer *self = GSK_CAIRO_RENDERER (renderer);
cairo_t *cr;

View File

@@ -128,8 +128,7 @@ gsk_renderer_real_render_texture (GskRenderer *self,
static void
gsk_renderer_real_render (GskRenderer *self,
GskRenderNode *root,
const cairo_region_t *region,
GskOffload *offload)
const cairo_region_t *region)
{
GSK_RENDERER_WARN_NOT_IMPLEMENTED_METHOD (self, render);
}
@@ -453,7 +452,7 @@ gsk_renderer_render (GskRenderer *renderer,
gsk_render_node_diff (priv->prev_node, root, clip, offload);
}
renderer_class->render (renderer, root, clip, offload);
renderer_class->render (renderer, root, clip);
if (GSK_RENDERER_DEBUG_CHECK (renderer, RENDERER))
{

View File

@@ -50,8 +50,7 @@ struct _GskRendererClass
const graphene_rect_t *viewport);
void (* render) (GskRenderer *renderer,
GskRenderNode *root,
const cairo_region_t *invalid,
GskOffload *offload);
const cairo_region_t *invalid);
};
GskProfiler * gsk_renderer_get_profiler (GskRenderer *renderer);

View File

@@ -349,8 +349,7 @@ gsk_vulkan_renderer_render_texture (GskRenderer *renderer,
static void
gsk_vulkan_renderer_render (GskRenderer *renderer,
GskRenderNode *root,
const cairo_region_t *region,
GskOffload *offload)
const cairo_region_t *region)
{
GskVulkanRenderer *self = GSK_VULKAN_RENDERER (renderer);
GskVulkanRender *render;