summaryrefslogtreecommitdiff
path: root/boilerplate
diff options
context:
space:
mode:
authorCarl Worth <cworth@cworth.org>2008-02-20 03:15:17 -0800
committerCarl Worth <cworth@cworth.org>2008-02-20 04:04:04 -0800
commit43302f0408d92138ac12afcd855d44c430074859 (patch)
tree7259cf3a3052de57f905d917b7c468d345e03fd4 /boilerplate
parent635fed77676f2446cbedfcf52a85f96e32c3a241 (diff)
Re-enable pdf tests with gradients and add poppler-based reference images
This freezes the current (buggy) poppler result in the test suite so that we will notice any future regressions.
Diffstat (limited to 'boilerplate')
-rw-r--r--boilerplate/cairo-boilerplate-pdf.c42
1 files changed, 0 insertions, 42 deletions
diff --git a/boilerplate/cairo-boilerplate-pdf.c b/boilerplate/cairo-boilerplate-pdf.c
index c89f29839..cc26d91bc 100644
--- a/boilerplate/cairo-boilerplate-pdf.c
+++ b/boilerplate/cairo-boilerplate-pdf.c
@@ -44,41 +44,6 @@ typedef struct _pdf_target_closure
#define ARRAY_LENGTH(__array) ((int) (sizeof (__array) / sizeof (__array[0])))
-/* We currently ignore tests that generate valid PDF output, but which
- * poppler currently misrenders. This lets us avoid false negatives in
- * the test suite, (at the significant cost that we won't notice any
- * regressions in the PDF output of these tests).
- *
- * So obviously, we have an interest in ensuring that the poppler bugs
- * get fixed sooner rather than later, so that we can re-enable these
- * tests. As such, we're trying to be good citizens by reporting all
- * such poppler bugs that we identify to the poppler bugzilla.
- *
- * Here's a tracking bug explaining the situation:
- *
- * Poppler does not yet handle everything in the cairo test suite
- * https://bugs.freedesktop.org/show_bug.cgi?id=12143
- *
- * Here's the rule: To add any test to this list (based on a poppler
- * bug), you must first create a new bug report against poppler,
- * marked as blocking 12143, and list that bug here in a comment for
- * the ignored test(s).
- *
- * And when this list shrinks to nothing, we can close bug 12143.
- */
-static const char *pdf_ignored_tests[] = {
- /* These first four failures are due to:
- *
- * Poppler doesn't correctly handle gradients with transparency
- * https://bugs.freedesktop.org/show_bug.cgi?id=12144
- */
- "gradient-alpha",
- "linear-gradient",
- "radial-gradient",
- "text-pattern",
- "trap-clip",
-};
-
cairo_surface_t *
_cairo_boilerplate_pdf_create_surface (const char *name,
cairo_content_t content,
@@ -90,13 +55,6 @@ _cairo_boilerplate_pdf_create_surface (const char *name,
pdf_target_closure_t *ptc;
cairo_surface_t *surface;
- if (mode == CAIRO_BOILERPLATE_MODE_TEST) {
- int i;
- for (i = 0; i < ARRAY_LENGTH (pdf_ignored_tests); i++)
- if (strcmp (name, pdf_ignored_tests[i]) == 0)
- return NULL;
- }
-
/* Sanitize back to a real cairo_content_t value. */
if (content == CAIRO_TEST_CONTENT_COLOR_ALPHA_FLATTENED)
content = CAIRO_CONTENT_COLOR_ALPHA;