compositor: fix plane coords in view_accumulate_damage()
authorPekka Paalanen <pekka.paalanen@collabora.co.uk>
Mon, 23 Feb 2015 12:08:25 +0000 (14:08 +0200)
committerPekka Paalanen <pekka.paalanen@collabora.co.uk>
Thu, 26 Feb 2015 07:07:14 +0000 (09:07 +0200)
commit502f5e06a2011ff07fb972719a668af24b0fc7e8
treeccfd76e5b191961405d4961ee51a78ab4f2920ec
parentc7d7fdfd11db9c4725149c821f4cdc0fe98f8eca
compositor: fix plane coords in view_accumulate_damage()

All things everywhere, except this one case, assume weston_plane::damage
is in global coordinates. Document it.

view_accumulate_damage() is wrong in converting damage to plane
coordinates (similar to global coordinate except translated). Fix this
by removing the unwanted translation, and use only global coordinates.

We have not seen this bug manifest in real life because we get lucky:
the origin of the primary plane is always at 0, 0. We do not use
non-primary planes, except cursor plane on DRM backend where the actual
damage coordinates are ignored.

Signed-off-by: Pekka Paalanen <pekka.paalanen@collabora.co.uk>
Reviewed-by: Bryce Harrington <bryce@osg.samsung.com>
src/compositor.c
src/compositor.h