Bug 11979 – SVG text does not respect font-size scaling in bbox calculation

• Component: SVG
• Status: RESOLVED DUPLICATE

[Eric Seidel (no email)]

SVG text does not respect font-size scaling in bbox calculation

See attached testcase.  After fixing bug 11973 our behavior is slightly better than before, but GREEN is still not entirely green.

[Eric Seidel (no email)]

Created attachment 12028 [details]
test demonstrating the incorrect behavior

[Eric Seidel (no email)]

Created attachment 12032 [details]
Another test showing how bad our current behavior is

[Eric Seidel (no email)]

Ok, so this is going to be harder to fix than I originally thought.

This should be split into two bugs.  One to cover the text gradient painting issue, and another to cover text having broken bounding box code.

The problem with gradients is not related to how text calculates bounding boxes, but rather due to the SVG text painting system.  SVG Text makes manual calls into SVGPaintServer::setup() and SVGPaintServer::teardown() instead of calling paint().  SVGPaintServerGradientCG isn't exactly designed for this code path.  The whole SVG text/paintserver interaction looks to be kinda a hack (certainly I am not w/o blame here :), and likely needs to be redesigned.  One example would be passing some sort of "painting text" bool to setup/teardown instead of setting a bool on the paintserver.  Another would be fixing the whole fill/stroke at once problem which we've talked about for so long.

[Eric Seidel (no email)]

Comment on attachment 12032 [details]
Another test showing how bad our current behavior is

obsoleting this test as it does not apply to this bug.  I've filed http://bugs.webkit.org/show_bug.cgi?id=11984 to track fixing this test case.

[Eric Seidel (no email)]

Wildfox has done some investigation here.

[Eric Seidel (no email)]

Fixed by bug 11984.

*** This bug has been marked as a duplicate of 11984 ***