Commit 130d595e authored by rwlbuis@webkit.org's avatar rwlbuis@webkit.org
Browse files

ASSERT (and crash) with dynamically moved <font-face>

https://bugs.webkit.org/show_bug.cgi?id=64839

Reviewed by Antti Koivisto.

Source/WebCore:

Reset the style declaration when rmeoving the font-face element from the document.

Test: svg/custom/font-face-move.svg

* svg/SVGFontFaceElement.cpp:
(WebCore::SVGFontFaceElement::removedFromDocument):

LayoutTests:

* svg/custom/font-face-move-expected.txt: Added.
* svg/custom/font-face-move.svg: Added.

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@108097 268f45cc-cd09-0410-ab3c-d52691b4dbfc
parent c47a416b
2012-02-17 Rob Buis <rbuis@rim.com>
ASSERT (and crash) with dynamically moved <font-face>
https://bugs.webkit.org/show_bug.cgi?id=64839
Reviewed by Antti Koivisto.
* svg/custom/font-face-move-expected.txt: Added.
* svg/custom/font-face-move.svg: Added.
2012-02-17 Julien Chaffraix <jchaffraix@webkit.org>
 
Unreviewed Chromium rebaseline after r107971.
<svg xmlns="http://www.w3.org/2000/svg">
<defs id="defs">
<font id="a">
<font-face id="bfont" font-family="b" />
</font>
</defs>
<text font-family="b">PASS</text>
<script>
if (window.layoutTestController)
layoutTestController.dumpAsText();
var defs = document.getElementById("defs");
var bfont = document.getElementById("bfont");
defs.appendChild(bfont);
</script>
</svg>
2012-02-17 Rob Buis <rbuis@rim.com>
ASSERT (and crash) with dynamically moved <font-face>
https://bugs.webkit.org/show_bug.cgi?id=64839
Reviewed by Antti Koivisto.
Reset the style declaration when rmeoving the font-face element from the document.
Test: svg/custom/font-face-move.svg
* svg/SVGFontFaceElement.cpp:
(WebCore::SVGFontFaceElement::removedFromDocument):
2012-02-17 Martin Robinson <mrobinson@igalia.com>
 
Fix some warnings encountered during the GTK+ build
......@@ -329,6 +329,7 @@ void SVGFontFaceElement::removedFromDocument()
{
removeFromMappedElementSheet();
SVGElement::removedFromDocument();
m_fontFaceRule->declaration()->parseDeclaration(emptyString());
}
void SVGFontFaceElement::childrenChanged(bool changedByParser, Node* beforeChange, Node* afterChange, int childCountDelta)
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment