Closed
Bug 732819
Opened 13 years ago
Closed 12 years ago
"ASSERTION: don't think me need this any more" with font-size: 0
Categories
(Core :: SVG, defect)
Tracking
()
RESOLVED
FIXED
mozilla16
People
(Reporter: jruderman, Assigned: jwatt)
References
Details
(Keywords: assertion, testcase)
Attachments
(2 files)
(deleted),
image/svg+xml
|
Details | |
(deleted),
patch
|
roc
:
review+
|
Details | Diff | Splinter Review |
###!!! ASSERTION: don't think me need this any more: 'Not Reached', file layout/svg/base/src/nsSVGOuterSVGFrame.h, line 148
This assertion was added in
https://hg.mozilla.org/mozilla-central/rev/2b4aed5ee94e#l2.17
as a (possibly-unreviewed) addition to the patch in bug 732429.
Fix this and I'll let you know if the fuzzer turns up any other testcases that trigger the assertion ;)
Assignee | ||
Comment 1•13 years ago
|
||
Yeah, that was an assertion I added in just before pushing, since I wondered if we could actually just remove much of the code that was touched in the patch for bug 732429. We tried removing that code in bug 411334, but as noted in bug 411334 comment 3, doing so regressed gearflowers. Removing the code no longer seems to regress gearflowers, hence why I wondered if it could die now.
Thanks for fuzzing and finding this issue. It seems like we shouldn't be trying to paint SVG text that has a font-size of zero. Preventing that should be a simple patch, but I don't have time even for that right now. If nobody else writes the patch in the meantime, I'll get to it later.
Assignee | ||
Comment 2•13 years ago
|
||
Gearflowers is http://kde-look.org/CONTENT/content-files/19524-gearflowers.svg.gz BTW.
Assignee | ||
Comment 3•12 years ago
|
||
Assignee: nobody → jwatt
Attachment #637748 -
Flags: review?(roc)
Comment on attachment 637748 [details] [diff] [review]
patch
Review of attachment 637748 [details] [diff] [review]:
-----------------------------------------------------------------
You had this in the other patch. I don't care which one it lands in.
Attachment #637748 -
Flags: review?(roc) → review+
Assignee | ||
Comment 5•12 years ago
|
||
Target Milestone: --- → mozilla16
Assignee | ||
Updated•12 years ago
|
Flags: in-testsuite+
Assignee | ||
Updated•12 years ago
|
Keywords: regression
Assignee | ||
Comment 6•12 years ago
|
||
I checked in a test for this as well, which required a follow-up to fix it:
https://hg.mozilla.org/integration/mozilla-inbound/rev/ef04a2a3284e
Comment 7•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/fed6b276ad32
https://hg.mozilla.org/mozilla-central/rev/ef04a2a3284e
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•