Bug 16518 - Permanent red background when SVG is used as a background image (DEBUG ONLY)
Summary: Permanent red background when SVG is used as a background image (DEBUG ONLY)
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: SVG (show other bugs)
Version: 528+ (Nightly build)
Hardware: Macintosh OS X 10.4
: P2 Normal
Assignee: Nikolas Zimmermann
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2007-12-19 14:34 PST by Beth Dakin
Modified: 2011-06-01 11:36 PDT (History)
1 user (show)

See Also:

Attachments
HTML of test (229 bytes, text/html)
2007-12-19 14:36 PST, Beth Dakin 		no flags Details
SVG for test (1.57 KB, image/svg+xml)
2007-12-19 14:36 PST, Beth Dakin 		no flags Details
Patch (144.98 KB, patch)
2011-06-01 09:19 PDT, Nikolas Zimmermann 		eric: review+
Details | Formatted Diff | Diff
Show Obsolete (2) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Beth Dakin 2007-12-19 14:34:40 PST 
If an SVG is used as a non-tiling background image and it does not take up the entire space of the element it is the background of, Debug builds always paint a red background behind the image, even when a different color is specified. The red background does not correct itself when you force a repaint. 

I have attached a screenshot of Debug and Release builds, and a test case that shows the problem.

<rdar://problem/5643240>
Comment 1 Beth Dakin 2007-12-19 14:36:11 PST 
Created attachment 17997
Comment 2 Beth Dakin 2007-12-19 14:36:36 PST 
Created attachment 17998 [details]
HTML of test
Comment 3 Beth Dakin 2007-12-19 14:36:58 PST 
Created attachment 17999 [details]
SVG for test
Comment 4 Nikolas Zimmermann 2011-06-01 09:19:44 PDT 
Created attachment 95611 [details]
Patch

Problem is gone, adding Beths testcase to svg/as-image.
Comment 5 Eric Seidel (no email) 2011-06-01 09:24:20 PDT 
Comment on attachment 95611 [details]
Patch

OK.  I suspect we could make the test case simpler.  But this is OK too.
Comment 6 Nikolas Zimmermann 2011-06-01 11:36:29 PDT 
Landed in r87829.