mozilla
/
gecko-dev
зеркало из https://github.com/mozilla/gecko-dev.git
1
0
Форкнуть 0
Код Задачи Пакеты Проекты Релизы Вики Активность

Backed out changeset 253e90f0ccf3 (bug 1541021) for mochitest failures on test_bounds.html

This commit is contained in:
Narcis Beleuzu 2019-04-07 19:50:14 +03:00
Родитель ea72deb0be
Коммит 8d78f21970
3 изменённых файлов: 4 добавлений и 42 удалений
Показать статистику Скачать Patch файл Скачать Diff файл Показать все файлы Свернуть все файлы

3
dom/svg/test/bounds-helper.svg
Просмотреть файл

@ -3,7 +3,6 @@
xmlns:xlink="http://www.w3.org/1999/xlink" version="1.1">
<style type="text/css">
text { font: 20px monospace; }
#css-trans-rect-2 { transform: scaleX(2) }
</style>
<g id="g">
@ -72,7 +71,5 @@ text { font: 20px monospace; }
<line id="rotatedLine1" x1="160" y1="150" x2="180" y2="170"
stroke="darkmagenta" stroke-width="10"
transform="rotate(-45 160 150)" />
<rect id="css-trans-rect-2" x="5" y="5" width="6" height="6"></rect>
</g>
</svg>

Режим "рядом" Свайп Оверлей

До

Ширина:  |  Высота:  |  Размер: 3.7 KiB

После

Ширина:  |  Высота:  |  Размер: 3.6 KiB

19
dom/svg/test/test_bounds.html
Просмотреть файл

@ -7,11 +7,6 @@ https://bugzilla.mozilla.org/show_bug.cgi?id=463934
<title>Test for Bug 463934</title>
<script type="application/javascript" src="/tests/SimpleTest/SimpleTest.js"></script>
<link rel="stylesheet" type="text/css" href="/tests/SimpleTest/test.css" />
<style>
#css-trans-rect-1 {
transform: scaleX(2)
}
</style>
</head>
<body>
<a target="_blank" href="https://bugzilla.mozilla.org/show_bug.cgi?id=463934">Mozilla Bug 463934</a>
@ -27,9 +22,6 @@ https://bugzilla.mozilla.org/show_bug.cgi?id=463934
<svg id="outer-4" xmlns="http://www.w3.org/2000/svg" width="30" height="30"
style="border: 10px solid black; padding: 10px">
</svg>
<svg id="outer-5" xmlns="http://www.w3.org/2000/svg" width="30" height="30">
<rect id="css-trans-rect-1" width="6" height="6"></rect>
</svg>
</div>
<iframe id="svg" src="bounds-helper.svg"></iframe>
@ -75,10 +67,6 @@ function runTest() {
is(bounds.width, 70, "outer-svg 4 getBoundingClientRect().width");
is(bounds.height, 70, "outer-svg 4 getBoundingClientRect().height");
bounds = document.getElementById("css-trans-rect-1").getBoundingClientRect();
is(bounds.width, 12, "css-trans-rect getBoundingClientRect().width");
is(bounds.height, 6, "css-trans-rect getBoundingClientRect().height");
var doc = $("svg").contentWindow.document;
var svg1Bounds = doc.getElementById("svg1").getBoundingClientRect();
@ -291,13 +279,6 @@ function runTest() {
isWithAbsTolerance(rotatedLine1Bounds.height, 10, 0.1,
"rotatedLine1Bounds.height");
var cssTransRect2Bounds =
doc.getElementById("css-trans-rect-2").getBoundingClientRect();
is(cssTransRect2Bounds.left, 10, "cssTransRect2Bounds.left");
is(cssTransRect2Bounds.top, 5, "cssTransRect2Bounds.top");
is(cssTransRect2Bounds.width, 12, "cssTransRect2Bounds.width");
is(cssTransRect2Bounds.height, 6, "cssTransRect2Bounds.height");
SimpleTest.finish();
}

24
layout/svg/nsSVGUtils.cpp
Просмотреть файл

@ -321,28 +321,12 @@ nsIFrame* nsSVGUtils::GetOuterSVGFrameAndCoveredRegion(nsIFrame* aFrame,
uint32_t flags =
nsSVGUtils::eForGetClientRects | nsSVGUtils::eBBoxIncludeFill |
nsSVGUtils::eBBoxIncludeStroke | nsSVGUtils::eBBoxIncludeMarkers;
auto ctm =
nsLayoutUtils::GetTransformToAncestor(aFrame, outer, 0, nullptr);
float appUnitsPerDevPixel = aFrame->PresContext()->AppUnitsPerDevPixel();
float initPositionX =
NSAppUnitsToFloatPixels(aFrame->GetPosition().x, appUnitsPerDevPixel),
initPositionY =
NSAppUnitsToFloatPixels(aFrame->GetPosition().y, appUnitsPerDevPixel);
Matrix mm;
ctm.CanDraw2D(&mm);
gfxMatrix m = ThebesMatrix(mm);
// Both nsSVGUtils::GetBBox and nsLayoutUtils::GetTransformToAncestor
// will count this displacement, we should remove it here to avoid
// double-counting.
m = m.PreTranslate(-initPositionX, -initPositionY);
gfxMatrix m = nsSVGUtils::GetUserToCanvasTM(aFrame);
SVGBBox bbox = nsSVGUtils::GetBBox(aFrame, flags, &m);
*aRect = nsLayoutUtils::RoundGfxRectToAppRect(
nsRect bounds = nsLayoutUtils::RoundGfxRectToAppRect(
bbox, aFrame->PresContext()->AppUnitsPerDevPixel());
nsMargin bp = outer->GetUsedBorderAndPadding();
*aRect = bounds + nsPoint(bp.left, bp.top);
}
return outer;