On 2013/09/18 22:47:35, pdr wrote:
> These test failures look real (or need more investigation)
Not sure about this one - it looks bad, but it's actually related/close to what
the test is doing:
<script>
if (window.testRunner)
testRunner.dumpAsText();
onload = function() {
document.body.offsetTop;
document.body.style.zoom=0.9;
document.designMode='on';
window.getSelection().setBaseAndExtent(filterInFirstRoot, 0, 5);
document.execCommand('ForwardDelete');
}
</script>
This test passes if it does not crash.<br/>
<svg id="111">
<text direction="rtl" id="filterInFirstRoot">
<tspan id="tSpanInFirstRoot">abc<U+E47E></tspan>
</text>
<path filter="url(#filterInSecondRoot)"></path>
</svg>
I don't fully understand what the above is trying to do or what the expected
result is (CF generated?). But it seems plausible that
document.execCommand('ForwardDelete') is zapping the 'a' and the rest is left
alone (and should be visible in the text dump).
For unicode-in-tspan-multi-svg-crash.html OTOH, I think the new result is
correct: we were failing to correctly lay out cross-svgroot before, but now that
it's working the unicode chars are showing up. FF is also displaying them.
Still looking at the other test diffs.
Issue 23785014: [SVG] Resources should be laid out in dependecy order.
(Closed)
Created 7 years, 3 months ago by f(malita)
Modified 7 years, 2 months ago
Reviewers: krit, eseidel, ojan, pdr., Stephen Chennney
Base URL: svn://svn.chromium.org/blink/trunk
Comments: 12