gsk: Fix mask transform handling
authorMatthias Clasen <mclasen@redhat.com>
Mon, 1 May 2023 00:40:00 +0000 (20:40 -0400)
committerMatthias Clasen <mclasen@redhat.com>
Mon, 1 May 2023 17:45:20 +0000 (13:45 -0400)
We must reset the modelview transform when
offscreen, or else we end up applying
it twice.

gsk/gl/gskglrenderjob.c

index 061265780f9ffb71931ca98504a9a69c14e28740..df19d98209d4f0af6ba3b457c8782b5b2b331741 100644 (file)
@@ -3391,11 +3391,14 @@ gsk_gl_render_job_visit_mask_node (GskGLRenderJob      *job,
   mask_offscreen.reset_clip = TRUE;
   mask_offscreen.do_not_cache = TRUE;
 
+  gsk_gl_render_job_set_modelview (job, NULL);
+
   /* TODO: We create 2 textures here as big as the mask node, but both
    * nodes might be a lot smaller than that.
    */
   if (!gsk_gl_render_job_visit_node_with_offscreen (job, source, &source_offscreen))
     {
+      gsk_gl_render_job_pop_modelview (job);
       gsk_gl_render_job_visit_node (job, source);
       return;
     }
@@ -3404,11 +3407,14 @@ gsk_gl_render_job_visit_mask_node (GskGLRenderJob      *job,
 
   if (!gsk_gl_render_job_visit_node_with_offscreen (job, mask, &mask_offscreen))
     {
+      gsk_gl_render_job_pop_modelview (job);
       return;
     }
 
   g_assert (mask_offscreen.was_offscreen);
 
+  gsk_gl_render_job_pop_modelview (job);
+
   gsk_gl_render_job_begin_draw (job, CHOOSE_PROGRAM (job, mask));
   gsk_gl_program_set_uniform_texture (job->current_program,
                                       UNIFORM_SHARED_SOURCE, 0,