clutter: Fix offscreen-effect painting of clones
authorDaniel van Vugt <daniel.van.vugt@canonical.com>
Thu, 24 May 2018 09:51:22 +0000 (17:51 +0800)
committerPeter Michael Green <plugwash@raspbian.org>
Thu, 14 Mar 2019 18:39:31 +0000 (18:39 +0000)
commitff5eb68a9c86aab3013a27e53761a49ba7c5a48e
tree7e3235b9d0919ba55a3f1736d8ca6037b84383f3
parentd5c6e2e51024ee78f126df909d98f5826984e174
clutter: Fix offscreen-effect painting of clones

`ClutterOffscreenEffect` had been getting the wrong bounding box in the
case of clones and descendents of clones, causing visibly incorrect
clipping. This was due to `clutter_actor_get_paint_box` only ever being
given the source actor during a paint (which is correct) and not the clone.
Even if we weren't painting a clone but an offscreened descendent of a
clone (like in gnome-shell's desktop zoom), we would get the wrong result.

Fortunately we don't need to know the actual clone/actor being painted so
don't need to call the problematic `clutter_actor_get_paint_box` at all.
The solution is to only keep untransformed rendering in the FBO and leave
the correct transformation for later. The correct clone/actor's
transformation is already set for us as the current cogl modelview matrix
by `clutter_actor_paint`.

Bonus optimization: This all means we don't need to keep `last_matrix_drawn`
or force a full repaint every time some part of the transformation changes.
Because the FBO contents are no longer affected by transformations. As it
should be. In other words, offscreen-effected actors can now move around
on screen without themselves being repainted.

Special thanks to Mai Lavelle for identifying the cause of the problem.

Fixes:
https://bugzilla.gnome.org/show_bug.cgi?id=789050,
https://bugzilla.gnome.org/show_bug.cgi?id=659523#c9,
https://gitlab.gnome.org/GNOME/mutter/issues/196,
https://gitlab.gnome.org/GNOME/mutter/issues/282,
https://gitlab.gnome.org/GNOME/gnome-shell/issues/387,
https://launchpad.net/bugs/1767648,
https://launchpad.net/bugs/1779615

Origin: https://gitlab.gnome.org/vanvugt/mutter/commit/9a466f289318050bde065bc8878947cb2691bc98
Bug-Ubuntu: https://launchpad.net/bugs/1767648, https://launchpad.net/bugs/1779615
Applied-Upstream: 3.31.90

Gbp-Pq: Name clutter-Fix-offscreen-effect-painting-of-clones.patch
clutter/clutter/clutter-actor-box-private.h [new file with mode: 0644]
clutter/clutter/clutter-actor-box.c
clutter/clutter/clutter-offscreen-effect.c
clutter/clutter/clutter-paint-volume.c
clutter/tests/conform/Makefile.am
clutter/tests/conform/actor-offscreen-limit-max-size.c [deleted file]
clutter/tests/conform/actor-offscreen-redirect.c