Backed out changeset 6f3c000986ca (bug 1019326) for causing mochitest failures @ dom/svg/test/test_getBBox-options.html CLOSED TREE

This commit is contained in:
Sandor Molnar 2024-06-07 11:06:26 +03:00
parent d3824dae47
commit 7c91c0996b
9 changed files with 288 additions and 282 deletions

View file

@ -102,7 +102,7 @@ already_AddRefed<SVGRect> SVGGraphicsElement::GetBBox(
}
uint32_t flags = 0;
if (aOptions.mFill) {
flags |= SVGUtils::eBBoxIncludeFillGeometry;
flags |= SVGUtils::eBBoxIncludeFill;
}
if (aOptions.mStroke) {
flags |= SVGUtils::eBBoxIncludeStroke;
@ -114,11 +114,11 @@ already_AddRefed<SVGRect> SVGGraphicsElement::GetBBox(
flags |= SVGUtils::eBBoxIncludeClipped;
}
if (flags == 0) {
return do_AddRef(new SVGRect(this, {}));
return do_AddRef(new SVGRect(this, gfx::Rect()));
}
if (flags == SVGUtils::eBBoxIncludeMarkers ||
flags == SVGUtils::eBBoxIncludeClipped) {
flags |= SVGUtils::eBBoxIncludeFillGeometry;
flags |= SVGUtils::eBBoxIncludeFill;
}
flags |= SVGUtils::eUseUserSpaceOfUseElement;
return do_AddRef(new SVGRect(this, ToRect(SVGUtils::GetBBox(frame, flags))));

View file

@ -36,8 +36,6 @@
<circle cx="100" cy="100" r="5"/>
<g/>
</g>
<!-- fill="none" should have a non-zero bounding box-->
<rect id="none" width="10" height="10" fill="none"/>
<use x="100" y="100" id="use_v" href="#v"/>
<use x="100" y="100" id="use_h" href="#h"/>
<use x="100" y="100" id="use_e" href="#e"/>

Before

Width:  |  Height:  |  Size: 1.4 KiB

After

Width:  |  Height:  |  Size: 1.3 KiB

View file

@ -147,28 +147,28 @@
<text id="text6" font-size="20px" font-familiy="monospace" fill="blue" x="0" y="80" clip-path="url('#rect06')">99</text>
<!-- image -->
<image id="image1" x="150" y="150" width="200" height="200" preserveApectRatio="none" clip="rect(10px,90px,90px,10px)"
<image id="image1" x="150" y="150" width="200" height="200" preserveApectRatio="none" clip="rect(200,300,300,200)"
xlink:href="data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAABAAAAAQCAIAAACQkWg2AAAAKXRFWHRDcmVhdGlvbiBUaW1lAJCFIDMwIDQgMjAxNCAwOToyODo0MyArMDkwMH73aTcAAAAHdElNRQfeBB4AHhEfV06+AAAACXBIWXMAAB7BAAAewQHDaVRTAAAABGdBTUEAALGPC/xhBQAAABlJREFUeNpjZPj/n4EUwESS6lENoxqGlAYASU8CHux7qQ4AAAAASUVORK5CYII="/>
<image id="image2" x="2" y="2" width="200" height="200" clip-path="url('#circle1')" preserveApectRatio="none"
<image id="image2" x="2" y="2" width="200" height="200" clip-path="url('#circle1')" preserveApectRatio="none"
xlink:href="data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAABAAAAAQCAIAAACQkWg2AAAAKXRFWHRDcmVhdGlvbiBUaW1lAJCFIDMwIDQgMjAxNCAwOToyODo0MyArMDkwMH73aTcAAAAHdElNRQfeBB4AHhEfV06+AAAACXBIWXMAAB7BAAAewQHDaVRTAAAABGdBTUEAALGPC/xhBQAAABlJREFUeNpjZPj/n4EUwESS6lENoxqGlAYASU8CHux7qQ4AAAAASUVORK5CYII="/>
<image id="image3" x="205" y="2" width="200" height="200" clip-path="url('#circle1')" preserveApectRatio="none"
<image id="image3" x="205" y="2" width="200" height="200" clip-path="url('#circle1')" preserveApectRatio="none"
xlink:href="data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAABAAAAAQCAIAAACQkWg2AAAAKXRFWHRDcmVhdGlvbiBUaW1lAJCFIDMwIDQgMjAxNCAwOToyODo0MyArMDkwMH73aTcAAAAHdElNRQfeBB4AHhEfV06+AAAACXBIWXMAAB7BAAAewQHDaVRTAAAABGdBTUEAALGPC/xhBQAAABlJREFUeNpjZPj/n4EUwESS6lENoxqGlAYASU8CHux7qQ4AAAAASUVORK5CYII="/>
<image id="image4" x="2" y="205" width="200" height="200" clip-path="url('#circle1')" preserveApectRatio="none"
<image id="image4" x="2" y="205" width="200" height="200" clip-path="url('#circle1')" preserveApectRatio="none"
xlink:href="data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAABAAAAAQCAIAAACQkWg2AAAAKXRFWHRDcmVhdGlvbiBUaW1lAJCFIDMwIDQgMjAxNCAwOToyODo0MyArMDkwMH73aTcAAAAHdElNRQfeBB4AHhEfV06+AAAACXBIWXMAAB7BAAAewQHDaVRTAAAABGdBTUEAALGPC/xhBQAAABlJREFUeNpjZPj/n4EUwESS6lENoxqGlAYASU8CHux7qQ4AAAAASUVORK5CYII="/>
<image id="image5" x="205" y="205" width="200" height="200" clip-path="url('#circle1')" preserveApectRatio="none"
xlink:href="data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAABAAAAAQCAIAAACQkWg2AAAAKXRFWHRDcmVhdGlvbiBUaW1lAJCFIDMwIDQgMjAxNCAwOToyODo0MyArMDkwMH73aTcAAAAHdElNRQfeBB4AHhEfV06+AAAACXBIWXMAAB7BAAAewQHDaVRTAAAABGdBTUEAALGPC/xhBQAAABlJREFUeNpjZPj/n4EUwESS6lENoxqGlAYASU8CHux7qQ4AAAAASUVORK5CYII="/>
<image id="image6" x="2" y="2" width="200" height="200" clip-path="url('#circle2')" preserveApectRatio="none"
<image id="image6" x="2" y="2" width="200" height="200" clip-path="url('#circle2')" preserveApectRatio="none"
xlink:href="data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAABAAAAAQCAIAAACQkWg2AAAAKXRFWHRDcmVhdGlvbiBUaW1lAJCFIDMwIDQgMjAxNCAwOToyODo0MyArMDkwMH73aTcAAAAHdElNRQfeBB4AHhEfV06+AAAACXBIWXMAAB7BAAAewQHDaVRTAAAABGdBTUEAALGPC/xhBQAAABlJREFUeNpjZPj/n4EUwESS6lENoxqGlAYASU8CHux7qQ4AAAAASUVORK5CYII="/>
<image id="image7" x="205" y="2" width="200" height="200" clip-path="url('#circle2')" preserveApectRatio="none"
<image id="image7" x="205" y="2" width="200" height="200" clip-path="url('#circle2')" preserveApectRatio="none"
xlink:href="data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAABAAAAAQCAIAAACQkWg2AAAAKXRFWHRDcmVhdGlvbiBUaW1lAJCFIDMwIDQgMjAxNCAwOToyODo0MyArMDkwMH73aTcAAAAHdElNRQfeBB4AHhEfV06+AAAACXBIWXMAAB7BAAAewQHDaVRTAAAABGdBTUEAALGPC/xhBQAAABlJREFUeNpjZPj/n4EUwESS6lENoxqGlAYASU8CHux7qQ4AAAAASUVORK5CYII="/>
<image id="image8" x="2" y="205" width="200" height="200" clip-path="url('#circle2')" preserveApectRatio="none"
<image id="image8" x="2" y="205" width="200" height="200" clip-path="url('#circle2')" preserveApectRatio="none"
xlink:href="data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAABAAAAAQCAIAAACQkWg2AAAAKXRFWHRDcmVhdGlvbiBUaW1lAJCFIDMwIDQgMjAxNCAwOToyODo0MyArMDkwMH73aTcAAAAHdElNRQfeBB4AHhEfV06+AAAACXBIWXMAAB7BAAAewQHDaVRTAAAABGdBTUEAALGPC/xhBQAAABlJREFUeNpjZPj/n4EUwESS6lENoxqGlAYASU8CHux7qQ4AAAAASUVORK5CYII="/>
<image id="image9" x="205" y="205" width="200" height="200" clip-path="url('#circle2')" preserveApectRatio="none"
@ -246,48 +246,48 @@
<use id="use8" xlink:href="#rect-10" x="50" y="50" clip-path="url('#m_atr')"/>
<!-- foreignObject -->
<foreignObject id="fo1" x="2" y="2" width="200" height="200" clip-path="url('#circle1')" clip="rect(2px,102px,102px,2px)">
<foreignObject id="fo1" x="2" y="2" width="200" height="200" clip-path="url('#circle1')" clip="rect(2,102,102,2)">
<div xmlns="http://www.w3.org/1999/xhtml" style="background-color:pink;width:100%;height:100%;">
There are issues of dynamic loading required for tiling. According to 'postpone' attribute of Resource Priorities, the dynamic loading is controlled by positional relation with bounding box of embedded contents and container's viewport. However, each bounding boxes of embedded contents should be whole earth basically when this method is used. (green part on Example) Tiling is impossible unless this situation is changed.
There are issues of dynamic loading required for tiling. Accroding to 'postpone' attribute of Resource Priorities, the dynamic loading is controlled by positional relation with bounding box of embedded contents and container's viewport. However, each bounding boxes of embedded contents should be whole earth basically when this method is used. (green part on Example) Tiling is impossible unless this situation is changed.
</div>
</foreignObject>
<foreignObject id="fo2" x="205" y="2" width="200" height="200" clip-path="url('#circle1')" >
<div xmlns="http://www.w3.org/1999/xhtml" style="background-color:pink;width:100%;height:100%;">
There are issues of dynamic loading required for tiling. According to 'postpone' attribute of Resource Priorities, the dynamic loading is controlled by positional relation with bounding box of embedded contents and container's viewport. However, each bounding boxes of embedded contents should be whole earth basically when this method is used. (green part on Example) Tiling is impossible unless this situation is changed.
There are issues of dynamic loading required for tiling. Accroding to 'postpone' attribute of Resource Priorities, the dynamic loading is controlled by positional relation with bounding box of embedded contents and container's viewport. However, each bounding boxes of embedded contents should be whole earth basically when this method is used. (green part on Example) Tiling is impossible unless this situation is changed.
</div>
</foreignObject>
<foreignObject id="fo3" x="2" y="205" width="200" height="200" clip-path="url('#circle1')" >
<div xmlns="http://www.w3.org/1999/xhtml" style="background-color:pink;width:100%;height:100%;">
There are issues of dynamic loading required for tiling. According to 'postpone' attribute of Resource Priorities, the dynamic loading is controlled by positional relation with bounding box of embedded contents and container's viewport. However, each bounding boxes of embedded contents should be whole earth basically when this method is used. (green part on Example) Tiling is impossible unless this situation is changed.
There are issues of dynamic loading required for tiling. Accroding to 'postpone' attribute of Resource Priorities, the dynamic loading is controlled by positional relation with bounding box of embedded contents and container's viewport. However, each bounding boxes of embedded contents should be whole earth basically when this method is used. (green part on Example) Tiling is impossible unless this situation is changed.
</div>
</foreignObject>
<foreignObject id="fo4" x="205" y="205" width="200" height="200" clip-path="url('#circle1')" clip="rect(2px,102px,102px,2px)">
<foreignObject id="fo4" x="205" y="205" width="200" height="200" clip-path="url('#circle1')" clip="rect(2,102,102,2)">
<div xmlns="http://www.w3.org/1999/xhtml" style="background-color:pink;width:100%;height:100%;">
There are issues of dynamic loading required for tiling. According to 'postpone' attribute of Resource Priorities, the dynamic loading is controlled by positional relation with bounding box of embedded contents and container's viewport. However, each bounding boxes of embedded contents should be whole earth basically when this method is used. (green part on Example) Tiling is impossible unless this situation is changed.
There are issues of dynamic loading required for tiling. Accroding to 'postpone' attribute of Resource Priorities, the dynamic loading is controlled by positional relation with bounding box of embedded contents and container's viewport. However, each bounding boxes of embedded contents should be whole earth basically when this method is used. (green part on Example) Tiling is impossible unless this situation is changed.
</div>
</foreignObject>
<foreignObject id="fo5" x="250" y="250" width="200" height="200" clip-path="url('#rect8')">
<div xmlns="http://www.w3.org/1999/xhtml" style="background-color:pink;width:100%;height:100%;">
There are issues of dynamic loading required for tiling. According to 'postpone' attribute of Resource Priorities, the dynamic loading is controlled by positional relation with bounding box of embedded contents and container's viewport. However, each bounding boxes of embedded contents should be whole earth basically when this method is used. (green part on Example) Tiling is impossible unless this situation is changed.
There are issues of dynamic loading required for tiling. Accroding to 'postpone' attribute of Resource Priorities, the dynamic loading is controlled by positional relation with bounding box of embedded contents and container's viewport. However, each bounding boxes of embedded contents should be whole earth basically when this method is used. (green part on Example) Tiling is impossible unless this situation is changed.
</div>
</foreignObject>
<foreignObject id="fo6" x="0" y="0" width="200" height="200" clip-path="url('#rect9')">
<div xmlns="http://www.w3.org/1999/xhtml" style="background-color:pink;width:100%;height:100%;">
There are issues of dynamic loading required for tiling. According to 'postpone' attribute of Resource Priorities, the dynamic loading is controlled by positional relation with bounding box of embedded contents and container's viewport. However, each bounding boxes of embedded contents should be whole earth basically when this method is used. (green part on Example) Tiling is impossible unless this situation is changed.
There are issues of dynamic loading required for tiling. Accroding to 'postpone' attribute of Resource Priorities, the dynamic loading is controlled by positional relation with bounding box of embedded contents and container's viewport. However, each bounding boxes of embedded contents should be whole earth basically when this method is used. (green part on Example) Tiling is impossible unless this situation is changed.
</div>
</foreignObject>
<foreignObject id="fo7" x="0" y="0" width="200" height="200" clip-path="url('#rect8')">
<div xmlns="http://www.w3.org/1999/xhtml" style="background-color:pink;width:100%;height:100%;">
There are issues of dynamic loading required for tiling. According to 'postpone' attribute of Resource Priorities, the dynamic loading is controlled by positional relation with bounding box of embedded contents and container's viewport. However, each bounding boxes of embedded contents should be whole earth basically when this method is used. (green part on Example) Tiling is impossible unless this situation is changed.
There are issues of dynamic loading required for tiling. Accroding to 'postpone' attribute of Resource Priorities, the dynamic loading is controlled by positional relation with bounding box of embedded contents and container's viewport. However, each bounding boxes of embedded contents should be whole earth basically when this method is used. (green part on Example) Tiling is impossible unless this situation is changed.
</div>
</foreignObject>
<foreignObject id="fo8" x="0" y="0" width="200" height="200" clip-path="url('#m_atr')">
<div xmlns="http://www.w3.org/1999/xhtml" style="background-color:pink;width:100%;height:100%;">
There are issues of dynamic loading required for tiling. According to 'postpone' attribute of Resource Priorities, the dynamic loading is controlled by positional relation with bounding box of embedded contents and container's viewport. However, each bounding boxes of embedded contents should be whole earth basically when this method is used. (green part on Example) Tiling is impossible unless this situation is changed.
There are issues of dynamic loading required for tiling. Accroding to 'postpone' attribute of Resource Priorities, the dynamic loading is controlled by positional relation with bounding box of embedded contents and container's viewport. However, each bounding boxes of embedded contents should be whole earth basically when this method is used. (green part on Example) Tiling is impossible unless this situation is changed.
</div>
</foreignObject>

Before

Width:  |  Height:  |  Size: 20 KiB

After

Width:  |  Height:  |  Size: 20 KiB

View file

@ -15,7 +15,7 @@ support-files = [
"bounds-helper.svg",
"dataTypes-helper.svg",
"fragments-helper.svg",
"getBBox-options-helper.svg",
"getBBox-method-helper.svg",
"getCTM-helper.svg",
"getSubStringLength-helper.svg",
"matrixUtils.js",
@ -82,7 +82,7 @@ support-files = [
["test_fragments.html"]
["test_getBBox-options.html"]
["test_getBBox-method.html"]
["test_getCTM.html"]

View file

@ -77,7 +77,6 @@ function run() {
checkBBox("v", 95, 45, 10, 155, 0.001);
checkBBox("h", 195, 45, 105, 55, 0.001);
checkBBox("e", 95, 95, 10, 10, 0.001);
checkBBox("none", 0, 0, 10, 10, 0.001);
checkBBox("use_v", 195, 145, 10, 155, 0.001);
checkBBox("use_h", 295, 145, 105, 55, 0.001);
checkBBox("use_e", 195, 195, 10, 10, 0.001);

View file

@ -0,0 +1,248 @@
<!DOCTYPE HTML>
<!--
https://bugzilla.mozilla.org/show_bug.cgi?id=999964
-->
<head>
<meta charset="utf-8"/>
<title>Test case for Bug 999964</title>
<script src="/tests/SimpleTest/SimpleTest.js"></script>
<link rel="stylesheet" type="text/css" href="/tests/SimpleTest/test.css"/>
</head>
<body>
<a target="_blank" href="https://bugzilla.mozilla.org/show_bug.cgi?id=999964">Mozilla Bug 999964</a>
<p id="display"></p>
<div id="content" style="display: none"></div>
<iframe id="svg" src="getBBox-method-helper.svg"></iframe>
<pre id="test">
<script class="testbody" type="application/javascript">
/** Test case for Bug 999964 **/
SimpleTest.waitForExplicitFinish();
function run() {
var flag = SpecialPowers.getBoolPref("svg.new-getBBox.enabled");
if (!flag) {
ok(!flag, "skip test for bug999964.");
SimpleTest.finish();
return;
}
var doc = $("svg").contentDocument;
function isFuzzy(a, b, error, name) {
ok(!(Math.abs(a - b) > error), name, "got " + a + ", expected " + b + " (within " + error + ")");
}
function getBBox(id, opt) {
return doc.getElementById(id).getBBox(opt);
}
function checkBBox(id, opt, x, y, width, height, error) {
var bbox = getBBox(id, opt);
isFuzzy(bbox.x, x, error, id + ".getBBox().x");
isFuzzy(bbox.y, y, error, id + ".getBBox().y");
isFuzzy(bbox.width, width, error, id + ".getBBox().width");
isFuzzy(bbox.height, height, error, id + ".getBBox().height");
}
function compareBBox1(id1, id2) {
var bbox1 = getBBox(id1);
var bbox2 = getBBox(id2);
is(bbox1.x, bbox2.x, id1 + ".getBBox().x");
is(bbox1.y, bbox2.y, id1 + ".getBBox().y");
isFuzzy(bbox1.width, bbox2.width, 0.0002, id1 + ".getBBox().width");
isFuzzy(bbox1.height, bbox2.height, 0.0001, id1 + ".getBBox().height");
}
function compareBBox2(id1, id2) {
// without 'x'
var bbox1 = getBBox(id1);
var bbox2 = getBBox(id2);
is(bbox1.y, bbox2.y, id1 + ".getBBox().y");
isFuzzy(bbox1.width, bbox2.width, 0.0002, id1 + ".getBBox().width");
isFuzzy(bbox1.height, bbox2.height, 0.0001, id1 + ".getBBox().height");
}
var opt = { fill: true, stroke: true, markers: true, clipped: true };
// <text>
// fill
opt = { fill: true, stroke: false, markers: false, clipped: false };
compareBBox1("text1", "text3");
compareBBox1("text2", "text4");
compareBBox1("text5", "text6");
// all
opt = { fill: true, stroke: true, markers: true, clipped: true };
compareBBox2("text1", "text3");
compareBBox2("text2", "text4");
compareBBox2("text5", "text6");
// clipped
opt = { fill: false, stroke: false, markers: false, clipped: true };
compareBBox2("text1", "text3");
compareBBox2("text2", "text4");
compareBBox2("text5", "text6");
// <image>
opt = { fill: true, stroke: true, markers: true, clipped: true };
checkBBox("image1", opt, 250, 250, 100, 100);
checkBBox("image2", opt, 53, 53, 149, 149);
checkBBox("image3", opt, 205, 53, 148, 149);
checkBBox("image4", opt, 53, 205, 149, 148);
checkBBox("image5", opt, 205, 205, 148, 148);
checkBBox("image6", opt, 52, 52, 100, 100);
checkBBox("image7", opt, 255, 52, 100, 100);
checkBBox("image8", opt, 52, 255, 100, 100);
checkBBox("image9", opt, 255, 255, 100, 100);
checkBBox("image10", opt, 200, 200, 200, 200);
checkBBox("image11", opt, 0, 0, 0, 0);
checkBBox("image12", opt, 43, 43, 714, 660);
checkBBox("image13", opt, 50, 50, 300, 300);
checkBBox("image14", opt, 0, 0, 0, 0);
opt = { fill: true, stroke: false, markers: false, clipped: false };
checkBBox("image1", opt, 150, 150, 200, 200, 0);
checkBBox("image2", opt, 2, 2, 200, 200, 0);
checkBBox("image3", opt, 205, 2, 200, 200, 0);
checkBBox("image4", opt, 2, 205, 200, 200, 0);
checkBBox("image5", opt, 205, 205, 200, 200, 0);
checkBBox("image6", opt, 2, 2, 200, 200, 0);
checkBBox("image7", opt, 205, 2, 200, 200, 0);
checkBBox("image8", opt, 2, 205, 200, 200, 0);
checkBBox("image9", opt, 205, 205, 200, 200, 0);
checkBBox("image10", opt, 0, 0, 400, 400, 0);
checkBBox("image11", opt, 0, 0, 400, 400, 0);
checkBBox("image12", opt, 25, 43, 768, 768, 0);
checkBBox("image13", opt, 0, 0, 400, 400, 0);
// <path>
opt = { fill: true, stroke: true, markers: true, clipped: true };
checkBBox("path1", opt, 2, 17, 120, 95, 0);
checkBBox("path2", opt, 156, 21, 116, 91, 0);
checkBBox("path3", opt, 6, 121, 116, 91, 0);
checkBBox("path4", opt, 2, 17, 98, 83, 0);
checkBBox("path5", opt, 156, 21, 44, 79, 0);
checkBBox("path6", opt, 6, 150, 94, 62, 0);
checkBBox("path7", opt, 2, 17, 98, 83, 0);
checkBBox("path8", opt, 156, 21, 94, 79, 0);
checkBBox("path9", opt, 6, 121, 94, 79, 0);
checkBBox("path10", opt, 10, 25, 100, 75, 0);
checkBBox("path11", opt, 160, 25, 100, 75, 0);
checkBBox("path12", opt, 10, 125, 100, 75, 0);
opt = { fill: true, stroke: false, markers: false, clipped: true };
checkBBox("path1", opt, 10, 25, 100, 75, 0);
checkBBox("path2", opt, 160, 25, 100, 75, 0);
checkBBox("path3", opt, 10, 125, 100, 75, 0);
checkBBox("path4", opt, 10, 25, 90, 75, 0);
checkBBox("path5", opt, 160, 25, 40, 75, 0);
checkBBox("path6", opt, 10, 150, 90, 50, 0);
checkBBox("path7", opt, 10, 25, 90, 75, 0);
checkBBox("path8", opt, 160, 25, 90, 75, 0);
checkBBox("path9", opt, 10, 125, 90, 75, 0);
checkBBox("path10", opt, 10, 25, 100, 75, 0);
checkBBox("path11", opt, 160, 25, 100, 75, 0);
checkBBox("path12", opt, 10, 125, 100, 75, 0);
opt = { fill: true, stroke: false, markers: false, clipped: false };
checkBBox("path1", opt, 10, 25, 100, 75, 0);
checkBBox("path2", opt, 160, 25, 100, 75, 0);
checkBBox("path3", opt, 10, 125, 100, 75, 0);
checkBBox("path4", opt, 10, 25, 100, 75, 0);
checkBBox("path5", opt, 160, 25, 100, 75, 0);
checkBBox("path6", opt, 10, 125, 100, 75, 0);
checkBBox("path7", opt, 10, 25, 100, 75, 0);
checkBBox("path8", opt, 160, 25, 100, 75, 0);
checkBBox("path9", opt, 10, 125, 100, 75, 0);
checkBBox("path10", opt, 10, 25, 100, 75, 0);
checkBBox("path11", opt, 160, 25, 100, 75, 0);
checkBBox("path12", opt, 10, 125, 100, 75, 0);
checkBBox("path13", opt, 0, 0, 100, 100, 0);
opt = { fill: false, stroke: true, markers: false, clipped: false };
checkBBox("path1", opt, 2, 17, 116, 91, 0);
checkBBox("path2", opt, 156, 21, 108, 83, 0);
checkBBox("path3", opt, 6, 121, 108, 83, 0);
checkBBox("path4", opt, 2, 17, 116, 91, 0);
checkBBox("path5", opt, 156, 21, 108, 83, 0);
checkBBox("path6", opt, 6, 121, 108, 83, 0);
checkBBox("path7", opt, 2, 17, 116, 91, 0);
checkBBox("path8", opt, 156, 21, 108, 83, 0);
checkBBox("path9", opt, 6, 121, 108, 83, 0);
checkBBox("path10", opt, 2, 17, 116, 91, 0);
checkBBox("path11", opt, 156, 21, 108, 83, 0);
checkBBox("path12", opt, 6, 121, 108, 83, 0);
opt = { fill: false, stroke: false, markers: true, clipped: false };
checkBBox("path1", opt, 10, 25, 112, 87, 0);
checkBBox("path2", opt, 160, 25, 112, 87, 0);
checkBBox("path3", opt, 10, 125, 112, 87, 0);
checkBBox("path4", opt, 10, 25, 112, 87, 0);
checkBBox("path5", opt, 160, 25, 112, 87, 0);
checkBBox("path6", opt, 10, 125, 112, 87, 0);
checkBBox("path7", opt, 10, 25, 112, 87, 0);
checkBBox("path8", opt, 160, 25, 112, 87, 0);
checkBBox("path9", opt, 10, 125, 112, 87, 0);
checkBBox("path10", opt, 10, 25, 112, 87, 0);
checkBBox("path11", opt, 160, 25, 112, 87, 0);
checkBBox("path12", opt, 10, 125, 112, 87, 0);
// <use>
opt = { fill: true, stroke: false, markers: false, clipped: false };
checkBBox("use1", opt, 70, 70, 180, 180, 0);
checkBBox("use2", opt, 250, 70, 180, 180, 0);
checkBBox("use3", opt, 70, 250, 180, 180, 0);
checkBBox("use4", opt, 22, 22, 180, 180, 0);
checkBBox("use5", opt, 225, 22, 180, 180, 0);
checkBBox("use6", opt, 22, 225, 180, 180, 0);
checkBBox("use7", opt, 225, 225, 180, 180, 0);
opt = { fill: true, stroke: true, markers: true, clipped: true };
checkBBox("use1", opt, 70, 66, 180, 94, 0);
checkBBox("use2", opt, 250, 70, 180, 90, 0);
checkBBox("use3", opt, 70, 250, 180, 90, 0);
checkBBox("use4", opt, 18, 18, 134, 134, 0);
checkBBox("use5", opt, 221, 18, 134, 134, 0);
checkBBox("use6", opt, 18, 221, 134, 134, 0);
checkBBox("use7", opt, 221, 221, 134, 134, 0);
checkBBox("use8", opt, 0, 0, 0, 0, 0);
// <foreignObject>
opt = { fill: true, stroke: false, markers: false, clipped: false };
checkBBox("fo1", opt, 2, 2, 200, 200, 0);
checkBBox("fo2", opt, 205, 2, 200, 200, 0);
checkBBox("fo3", opt, 2, 205, 200, 200, 0);
checkBBox("fo4", opt, 205, 205, 200, 200, 0);
checkBBox("fo5", opt, 250, 250, 200, 200, 0);
checkBBox("fo6", opt, 0, 0, 200, 200, 0);
checkBBox("fo7", opt, 0, 0, 200, 200, 0);
opt = { fill: true, stroke: true, markers: true, clipped: true };
checkBBox("fo1", opt, 53, 53, 51, 51, 0);
checkBBox("fo2", opt, 205, 53, 148, 149, 0);
checkBBox("fo3", opt, 53, 205, 149, 148, 0);
checkBBox("fo4", opt, 207, 207, 100, 100, 0);
checkBBox("fo5", opt, 0, 0, 0, 0, 0);
checkBBox("fo6", opt, 100, 100, 100, 100, 0);
checkBBox("fo7", opt, 10, 10, 180, 180, 0);
checkBBox("fo8", opt, 0, 0, 0, 0, 0);
// from http://www.w3.org/Graphics/SVG/Test/20110816/harness/htmlObjectApproved/masking-path-07-b.html
opt = { fill: true, stroke: true, markers: true, clipped: true };
checkBBox("rect-1", opt, 10, 10, 140, 140, 0);
checkBBox("rect-2", opt, 50, 30, 25, 100, 0);
checkBBox("rect-3", opt, 50, 50, 100, 100, 0);
checkBBox("g1", opt, 50, 50, 100, 100, 0);
SimpleTest.finish();
}
window.addEventListener("load", run);
</script>
</pre>
</body>
</html>

View file

@ -1,244 +0,0 @@
<!DOCTYPE HTML>
<!--
https://bugzilla.mozilla.org/show_bug.cgi?id=999964
-->
<head>
<meta charset="utf-8"/>
<title>Test case for Bug 999964</title>
<script src="/tests/SimpleTest/SimpleTest.js"></script>
<link rel="stylesheet" type="text/css" href="/tests/SimpleTest/test.css"/>
</head>
<body>
<a target="_blank" href="https://bugzilla.mozilla.org/show_bug.cgi?id=999964">Mozilla Bug 999964</a>
<p id="display"></p>
<div id="content" style="display: none"></div>
<iframe id="svg" src="getBBox-method-helper.svg"></iframe>
<pre id="test">
<script class="testbody" type="application/javascript">
/** Test case for Bug 999964 **/
SimpleTest.waitForExplicitFinish();
function run() {
SpecialPowers.pushPrefEnv({'set': [['svg.new-getBBox.enabled', true]]}, runTests);
}
function runTests() {
var doc = $("svg").contentDocument;
function getBBox(id, opt) {
return doc.getElementById(id).getBBox(opt);
}
function checkBBox(id, opt, x, y, width, height, epsilon) {
if (epsilon == undefined) {
epsilon = 0.1;
}
var bbox = getBBox(id, opt);
isfuzzy(bbox.x, x, epsilon, id + ".getBBox().x " + JSON.stringify(opt));
isfuzzy(bbox.y, y, epsilon, id + ".getBBox().y " + JSON.stringify(opt));
isfuzzy(bbox.width, width, epsilon, id + ".getBBox().width " + JSON.stringify(opt));
isfuzzy(bbox.height, height, epsilon, id + ".getBBox().height " + JSON.stringify(opt));
}
function compareBBox1(id1, id2) {
var bbox1 = getBBox(id1);
var bbox2 = getBBox(id2);
is(bbox1.x, bbox2.x, id1 + ".getBBox().x");
is(bbox1.y, bbox2.y, id1 + ".getBBox().y");
isfuzzy(bbox1.width, bbox2.width, 0.0002, id1 + ".getBBox().width");
isfuzzy(bbox1.height, bbox2.height, 0.0001, id1 + ".getBBox().height");
}
function compareBBox2(id1, id2) {
// without 'x'
var bbox1 = getBBox(id1);
var bbox2 = getBBox(id2);
is(bbox1.y, bbox2.y, id1 + ".getBBox().y");
isfuzzy(bbox1.width, bbox2.width, 0.0002, id1 + ".getBBox().width");
isfuzzy(bbox1.height, bbox2.height, 0.0001, id1 + ".getBBox().height");
}
var opt = { fill: true, stroke: true, markers: true, clipped: true };
// <text>
// fill
opt = { fill: true, stroke: false, markers: false, clipped: false };
compareBBox1("text1", "text3");
compareBBox1("text2", "text4");
compareBBox1("text5", "text6");
// all
opt = { fill: true, stroke: true, markers: true, clipped: true };
compareBBox2("text1", "text3");
compareBBox2("text2", "text4");
compareBBox2("text5", "text6");
// clipped
opt = { fill: false, stroke: false, markers: false, clipped: true };
compareBBox2("text1", "text3");
compareBBox2("text2", "text4");
compareBBox2("text5", "text6");
// <image>
opt = { fill: true, stroke: true, markers: true, clipped: true };
checkBBox("image1", opt, 160, 160, 80, 80);
checkBBox("image2", opt, 53, 53, 149, 149);
checkBBox("image3", opt, 205, 53, 148, 149);
checkBBox("image4", opt, 53, 205, 149, 148);
checkBBox("image5", opt, 205, 205, 148, 148);
checkBBox("image6", opt, 52, 52, 100, 100);
checkBBox("image7", opt, 255, 52, 100, 100);
checkBBox("image8", opt, 52, 255, 100, 100);
checkBBox("image9", opt, 255, 255, 100, 100);
checkBBox("image10", opt, 200, 200, 200, 200);
checkBBox("image11", opt, 0, 0, 0, 0);
checkBBox("image12", opt, 43, 43, 714, 660);
checkBBox("image13", opt, 50, 50, 300, 300);
checkBBox("image14", opt, 0, 0, 0, 0);
opt = { fill: true, stroke: false, markers: false, clipped: false };
checkBBox("image1", opt, 150, 150, 200, 200);
checkBBox("image2", opt, 2, 2, 200, 200);
checkBBox("image3", opt, 205, 2, 200, 200);
checkBBox("image4", opt, 2, 205, 200, 200);
checkBBox("image5", opt, 205, 205, 200, 200);
checkBBox("image6", opt, 2, 2, 200, 200);
checkBBox("image7", opt, 205, 2, 200, 200);
checkBBox("image8", opt, 2, 205, 200, 200);
checkBBox("image9", opt, 205, 205, 200, 200);
checkBBox("image10", opt, 0, 0, 400, 400);
checkBBox("image11", opt, 0, 0, 400, 400);
checkBBox("image12", opt, 25, 43, 768, 768);
checkBBox("image13", opt, 0, 0, 400, 400);
// <path>
opt = { fill: true, stroke: true, markers: true, clipped: true };
checkBBox("path1", opt, 6.2, 20.8, 115.8, 91.2, 2);
checkBBox("path2", opt, 156.2, 20.8, 115.8, 91.2, 2);
checkBBox("path3", opt, 6.2, 120.7, 115.8, 91.2, 2);
checkBBox("path4", opt, 6.2, 20.8, 93.8, 79.2, 2);
checkBBox("path5", opt, 156.2, 20.8, 43.8, 79.2, 2);
checkBBox("path6", opt, 6.2, 150, 93.8, 62, 2);
checkBBox("path7", opt, 6.2, 20.8, 93.8, 79.2, 2);
checkBBox("path8", opt, 156.2, 20.8, 93.8, 79.2, 2);
checkBBox("path9", opt, 6.2, 120.8, 93.8, 79.2, 2);
checkBBox("path10", opt, 10, 25, 100, 75);
checkBBox("path11", opt, 160, 25, 100, 75);
checkBBox("path12", opt, 10, 125, 100, 75);
opt = { fill: true, stroke: false, markers: false, clipped: true };
checkBBox("path1", opt, 10, 25, 100, 75);
checkBBox("path2", opt, 160, 25, 100, 75);
checkBBox("path3", opt, 10, 125, 100, 75);
checkBBox("path4", opt, 10, 25, 90, 75);
checkBBox("path5", opt, 160, 25, 40, 75);
checkBBox("path6", opt, 10, 150, 90, 50);
checkBBox("path7", opt, 10, 25, 90, 75);
checkBBox("path8", opt, 160, 25, 90, 75);
checkBBox("path9", opt, 10, 125, 90, 75);
checkBBox("path10", opt, 10, 25, 100, 75);
checkBBox("path11", opt, 160, 25, 100, 75);
checkBBox("path12", opt, 10, 125, 100, 75);
opt = { fill: true, stroke: false, markers: false, clipped: false };
checkBBox("path1", opt, 10, 25, 100, 75);
checkBBox("path2", opt, 160, 25, 100, 75);
checkBBox("path3", opt, 10, 125, 100, 75);
checkBBox("path4", opt, 10, 25, 100, 75);
checkBBox("path5", opt, 160, 25, 100, 75);
checkBBox("path6", opt, 10, 125, 100, 75);
checkBBox("path7", opt, 10, 25, 100, 75);
checkBBox("path8", opt, 160, 25, 100, 75);
checkBBox("path9", opt, 10, 125, 100, 75);
checkBBox("path10", opt, 10, 25, 100, 75);
checkBBox("path11", opt, 160, 25, 100, 75);
checkBBox("path12", opt, 10, 125, 100, 75);
checkBBox("path13", opt, 0, 0, 100, 100);
opt = { fill: false, stroke: true, markers: false, clipped: false };
checkBBox("path1", opt, 6.2, 20.8, 107.8, 83.2, 4);
checkBBox("path2", opt, 156.2, 20.8, 107.8, 83.2, 4);
checkBBox("path3", opt, 6.2, 120.8, 107.8, 83.2, 4);
checkBBox("path4", opt, 6.2, 20.8, 107.8, 83.2, 4);
checkBBox("path5", opt, 156.2, 20.8, 107.8, 83.2, 4);
checkBBox("path6", opt, 6.2, 120.8, 107.8, 83.2, 4);
checkBBox("path7", opt, 6.2, 20.8, 107.8, 83.2, 4);
checkBBox("path8", opt, 156.2, 20.8, 107.8, 83.2, 4);
checkBBox("path9", opt, 6.2, 120.8, 107.8, 83.2, 4);
checkBBox("path10", opt, 6.2, 20.8, 107.8, 83.2, 4);
checkBBox("path11", opt, 156.2, 20.8, 107.8, 83.2, 4);
checkBBox("path12", opt, 6.2, 120.8, 107.8, 83.2, 4);
opt = { fill: false, stroke: false, markers: true, clipped: false };
checkBBox("path1", opt, 10, 25, 112, 87);
checkBBox("path2", opt, 160, 25, 112, 87);
checkBBox("path3", opt, 10, 125, 112, 87);
checkBBox("path4", opt, 10, 25, 112, 87);
checkBBox("path5", opt, 160, 25, 112, 87);
checkBBox("path6", opt, 10, 125, 112, 87);
checkBBox("path7", opt, 10, 25, 112, 87);
checkBBox("path8", opt, 160, 25, 112, 87);
checkBBox("path9", opt, 10, 125, 112, 87);
checkBBox("path10", opt, 10, 25, 112, 87);
checkBBox("path11", opt, 160, 25, 112, 87);
checkBBox("path12", opt, 10, 125, 112, 87);
// <use>
opt = { fill: true, stroke: false, markers: false, clipped: false };
checkBBox("use1", opt, 70, 70, 180, 180);
checkBBox("use2", opt, 250, 70, 180, 180);
checkBBox("use3", opt, 70, 250, 180, 180);
checkBBox("use4", opt, 22, 22, 180, 180);
checkBBox("use5", opt, 225, 22, 180, 180);
checkBBox("use6", opt, 22, 225, 180, 180);
checkBBox("use7", opt, 225, 225, 180, 180);
opt = { fill: true, stroke: true, markers: true, clipped: true };
checkBBox("use1", opt, 70, 66, 180, 94);
checkBBox("use2", opt, 250, 70, 180, 90);
checkBBox("use3", opt, 70, 250, 180, 90);
checkBBox("use4", opt, 18, 18, 134, 134);
checkBBox("use5", opt, 221, 18, 134, 134);
checkBBox("use6", opt, 18, 221, 134, 134);
checkBBox("use7", opt, 221, 221, 134, 134);
checkBBox("use8", opt, 0, 0, 0, 0);
// <foreignObject>
opt = { fill: true, stroke: false, markers: false, clipped: false };
checkBBox("fo1", opt, 2, 2, 200, 200);
checkBBox("fo2", opt, 205, 2, 200, 200);
checkBBox("fo3", opt, 2, 205, 200, 200);
checkBBox("fo4", opt, 205, 205, 200, 200);
checkBBox("fo5", opt, 250, 250, 200, 200);
checkBBox("fo6", opt, 0, 0, 200, 200);
checkBBox("fo7", opt, 0, 0, 200, 200);
opt = { fill: true, stroke: true, markers: true, clipped: true };
checkBBox("fo1", opt, 53, 53, 51, 51);
checkBBox("fo2", opt, 205, 53, 148, 149);
checkBBox("fo3", opt, 53, 205, 149, 148);
checkBBox("fo4", opt, 207, 207, 100, 100);
checkBBox("fo5", opt, 0, 0, 0, 0);
checkBBox("fo6", opt, 100, 100, 100, 100);
checkBBox("fo7", opt, 10, 10, 180, 180);
checkBBox("fo8", opt, 0, 0, 0, 0);
// from http://www.w3.org/Graphics/SVG/Test/20110816/harness/htmlObjectApproved/masking-path-07-b.html
opt = { fill: true, stroke: true, markers: true, clipped: true };
checkBBox("rect-1", opt, 10, 10, 140, 140);
checkBBox("rect-2", opt, 50, 30, 25, 100);
checkBBox("rect-3", opt, 50, 50, 100, 100);
checkBBox("g1", opt, 50, 50, 100, 100);
SimpleTest.finish();
}
window.addEventListener("load", run);
</script>
</pre>
</body>
</html>

View file

@ -424,15 +424,15 @@ SVGBBox SVGClipPathFrame::GetBBoxForClipPathFrame(const SVGBBox& aBBox,
}
nsIContent* node = GetContent()->GetFirstChild();
SVGBBox unionBBox;
SVGBBox unionBBox, tmpBBox;
for (; node; node = node->GetNextSibling()) {
if (nsIFrame* frame = node->GetPrimaryFrame()) {
ISVGDisplayableFrame* svg = do_QueryFrame(frame);
if (svg) {
gfxMatrix matrix =
SVGUtils::GetTransformMatrixInUserSpace(frame) * aMatrix;
SVGBBox tmpBBox = svg->GetBBoxContribution(
gfx::ToMatrix(matrix), SVGUtils::eBBoxIncludeFillGeometry);
tmpBBox = svg->GetBBoxContribution(gfx::ToMatrix(matrix),
SVGUtils::eBBoxIncludeFill);
SVGClipPathFrame* clipPathFrame;
if (SVGObserverUtils::GetAndObserveClipPath(frame, &clipPathFrame) !=
SVGObserverUtils::eHasRefsSomeInvalid &&
@ -449,8 +449,9 @@ SVGBBox SVGClipPathFrame::GetBBoxForClipPathFrame(const SVGBBox& aBBox,
}
if (clipPathThatClipsClipPath) {
unionBBox.Intersect(clipPathThatClipsClipPath->GetBBoxForClipPathFrame(
aBBox, aMatrix, aFlags));
tmpBBox = clipPathThatClipsClipPath->GetBBoxForClipPathFrame(aBBox, aMatrix,
aFlags);
unionBBox.Intersect(tmpBBox);
}
return unionBBox;
}

View file

@ -809,7 +809,8 @@ gfxRect SVGUtils::GetClipRectForFrame(const nsIFrame* aFrame, float aX,
const auto& rect = effects->mClip.AsRect();
nsRect coordClipRect = rect.ToLayoutRect();
nsIntRect clipPxRect = coordClipRect.ToOutsidePixels(AppUnitsPerCSSPixel());
nsIntRect clipPxRect = coordClipRect.ToOutsidePixels(
aFrame->PresContext()->AppUnitsPerDevPixel());
gfxRect clipRect =
gfxRect(clipPxRect.x, clipPxRect.y, clipPxRect.width, clipPxRect.height);
if (rect.right.IsAuto()) {
@ -908,14 +909,17 @@ gfxRect SVGUtils::GetBBox(nsIFrame* aFrame, uint32_t aFlags,
// Account for 'clipped'.
if (aFlags & SVGUtils::eBBoxIncludeClipped) {
gfxRect clipRect;
float x, y, width, height;
gfxRect fillBBox =
svg->GetBBoxContribution({}, SVGUtils::eBBoxIncludeFill).ToThebesRect();
x = fillBBox.x;
y = fillBBox.y;
width = fillBBox.width;
height = fillBBox.height;
// XXX Should probably check for overflow: clip too.
bool hasClip = aFrame->StyleDisplay()->IsScrollableOverflow();
if (hasClip) {
clipRect = SVGUtils::GetClipRectForFrame(aFrame, 0.0f, 0.0f,
fillBBox.width, fillBBox.height);
clipRect.MoveBy(fillBBox.TopLeft());
clipRect = SVGUtils::GetClipRectForFrame(aFrame, x, y, width, height);
if (aFrame->IsSVGForeignObjectFrame() || aFrame->IsSVGUseFrame()) {
clipRect = matrix.TransformBounds(clipRect);
}
@ -923,14 +927,14 @@ gfxRect SVGUtils::GetBBox(nsIFrame* aFrame, uint32_t aFlags,
SVGClipPathFrame* clipPathFrame;
if (SVGObserverUtils::GetAndObserveClipPath(aFrame, &clipPathFrame) ==
SVGObserverUtils::eHasRefsSomeInvalid) {
bbox = gfxRect();
bbox = gfxRect(0, 0, 0, 0);
} else {
if (clipPathFrame) {
SVGClipPathElement* clipContent =
static_cast<SVGClipPathElement*>(clipPathFrame->GetContent());
if (clipContent->IsUnitsObjectBoundingBox()) {
matrix.PreTranslate(fillBBox.TopLeft());
matrix.PreScale(fillBBox.width, fillBBox.height);
matrix.PreTranslate(gfxPoint(x, y));
matrix.PreScale(width, height);
} else if (aFrame->IsSVGForeignObjectFrame()) {
matrix = gfxMatrix();
}
@ -945,7 +949,7 @@ gfxRect SVGUtils::GetBBox(nsIFrame* aFrame, uint32_t aFlags,
}
if (bbox.IsEmpty()) {
bbox = gfxRect();
bbox = gfxRect(0, 0, 0, 0);
}
}
}