After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 663477 - Support OpacityMask for Path elements
Support OpacityMask for Path elements
Status: RESOLVED FIXED
Product: libgxps
Classification: Platform
Component: general
unspecified
Other Linux
: Normal enhancement
: ---
Assigned To: libgxps maintainers
libgxps maintainers
Depends on:
Blocks:
 
 
Reported: 2011-11-05 21:15 UTC by Jason Crain
Modified: 2012-01-07 14:47 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
adds OpacityMask support for Path elements (3.35 KB, patch)
2011-11-05 21:15 UTC, Jason Crain
none Details | Review
example XPS document showing Path with OpacityMask (2.50 KB, application/octet-stream)
2011-11-05 21:17 UTC, Jason Crain
  Details
adds OpacityMask support for Path elements (6.07 KB, patch)
2011-11-27 02:34 UTC, Jason Crain
committed Details | Review

Description Jason Crain 2011-11-05 21:15:54 UTC
Created attachment 200789 [details] [review]
adds OpacityMask support for Path elements

Included patch adds OpacityMask support for Path elements.
Comment 1 Jason Crain 2011-11-05 21:17:17 UTC
Created attachment 200790 [details]
example XPS document showing Path with OpacityMask
Comment 2 Carlos Garcia Campos 2011-11-06 15:15:51 UTC
This patch seems to make some documents slower to render, and it causes an infinite loop in one of the XPS documents I have for testing (sent to you in private email).
Comment 3 Jason Crain 2011-11-27 02:34:04 UTC
Created attachment 202217 [details] [review]
adds OpacityMask support for Path elements

Please try out this patch.  It calls cairo_clip before cairo_push_group and cairo_mask and is much faster.  Rendering is slightly rearranged to make it easier to calculate extents.
Comment 4 Carlos Garcia Campos 2012-01-07 14:47:23 UTC
Review of attachment 202217 [details] [review]:

Sorry for the delay. I've just applied it to git master. Thanks!