Commit 4af9a3ba authored by krit@webkit.org's avatar krit@webkit.org

-webkit-clip-path does not apply origin for polygon()

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

Reviewed by Tim Horton.

Source/WebCore:

The polygon() shape function did not apply origin of bouding box on created path. The shape was
not moved to the correct position.

Tests: svg/clip-path/clip-path-shape-polygon-relative-expected.svg
       svg/clip-path/clip-path-shape-polygon-relative.svg

* rendering/style/BasicShapes.cpp:
(WebCore::BasicShapePolygon::path): Apply origin of bounding box.

LayoutTests:

Check that the origin of the bounding box gets applied to the clip path.

* svg/clip-path/clip-path-shape-polygon-relative-expected.svg: Added.
* svg/clip-path/clip-path-shape-polygon-relative.svg: Added.


git-svn-id: http://svn.webkit.org/repository/webkit/trunk@127548 268f45cc-cd09-0410-ab3c-d52691b4dbfc
parent 455b78f8
2012-09-04 Dirk Schulze <krit@webkit.org>
-webkit-clip-path does not apply origin for polygon()
https://bugs.webkit.org/show_bug.cgi?id=95656
Reviewed by Tim Horton.
Check that the origin of the bounding box gets applied to the clip path.
* svg/clip-path/clip-path-shape-polygon-relative-expected.svg: Added.
* svg/clip-path/clip-path-shape-polygon-relative.svg: Added.
2012-09-04 Kenichi Ishibashi <bashi@chromium.org>
[Chromium] Unreviewed test expectations update after r127534
<svg xmlns="http://www.w3.org/2000/svg">
<rect x="50" y="50" width="200" height="200" fill="green"/>
</svg>
\ No newline at end of file
<svg xmlns="http://www.w3.org/2000/svg">
<rect x="50" y="50" width="200" height="200" fill="green" style="-webkit-clip-path: polygon(0% 0%, 100% 0%, 100% 100%, 0% 100%)"/>
</svg>
\ No newline at end of file
2012-09-04 Dirk Schulze <krit@webkit.org>
-webkit-clip-path does not apply origin for polygon()
https://bugs.webkit.org/show_bug.cgi?id=95656
Reviewed by Tim Horton.
The polygon() shape function did not apply origin of bouding box on created path. The shape was
not moved to the correct position.
Tests: svg/clip-path/clip-path-shape-polygon-relative-expected.svg
svg/clip-path/clip-path-shape-polygon-relative.svg
* rendering/style/BasicShapes.cpp:
(WebCore::BasicShapePolygon::path): Apply origin of bounding box.
2012-09-04 Adam Barth <abarth@chromium.org>
Make chromium-linux build without WTF_DEPRECATED_STRING_OPERATORS
......@@ -82,11 +82,11 @@ void BasicShapePolygon::path(Path& path, const FloatRect& boundingBox)
if (!length)
return;
path.moveTo(FloatPoint(floatValueForLength(m_values.at(0), boundingBox.width()),
floatValueForLength(m_values.at(1), boundingBox.width())));
path.moveTo(FloatPoint(floatValueForLength(m_values.at(0), boundingBox.width()) + boundingBox.x(),
floatValueForLength(m_values.at(1), boundingBox.width()) + boundingBox.y()));
for (size_t i = 2; i < length; i = i + 2) {
path.addLineTo(FloatPoint(floatValueForLength(m_values.at(i), boundingBox.width()),
floatValueForLength(m_values.at(i + 1), boundingBox.width())));
path.addLineTo(FloatPoint(floatValueForLength(m_values.at(i), boundingBox.width()) + boundingBox.x(),
floatValueForLength(m_values.at(i + 1), boundingBox.width()) + boundingBox.y()));
}
path.closeSubpath();
}
......
Markdown is supported
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