Skip to content

GitLab

T
tangled
Commit e137f062 authored by eae@chromium.org's avatar eae@chromium.org
Browse files
Options

Revert "[CSS Shapes] clamp RasterShape shapeMargin to reference box size" 

This reverts r258671 (commit 04d85dfa329176dc489bdcf9d2654f38e72a484e) as it broke android.

BUG=355167
TBR=pdr@chromium.org

Review URL: https://codereview.chromium.org/209303002

git-svn-id: svn://svn.chromium.org/blink/trunk@169771 bbb929c8-8fbe-4397-9dbb-9b2b20218538
parent bc8826ef
Hide whitespace changes
Inline Side-by-side
Showing with 9 additions and 65 deletions
third_party/WebKit/LayoutTests/fast/shapes/shape-outside-floats/shape-outside-one-pixel-expected.html deleted 100644 → 0
View file @ bc8826ef
<!DOCTYPE html>
<html>
<head>
<style>
#content {
font: 50px/1 Ahem, sans-serif;
-webkit-font-smoothing: antialiased;
color: green;
}
#sandbag {
float: left;
width: 150px;
height: 100px;
}
</style>
</head>
<body>
<p>The shape-outside in this test is a 2x2 image with only the upper left pixel set. The shape
element's margin-box is and the the shape-margin is big enough (is greater than 150 * sqrt(2))
so that the shape-margin boundary extends beyond the margin-box's bounds. The shape-outside boundary
is clipped to the margin-box, so it's as if the shape-outside was equal to the margin-box.</p>
<div id="content">
<div id="sandbag"></div>X<br>X
</div>
</body>
</html>
third_party/WebKit/LayoutTests/fast/shapes/shape-outside-floats/shape-outside-one-pixel.html deleted 100644 → 0
View file @ bc8826ef
<!DOCTYPE html>
<html>
<head>
<style>
#content {
font: 50px/1 Ahem, sans-serif;
-webkit-font-smoothing: antialiased;
color: green;
}
#image-shape {
float: left;
shape-outside: url("data:image/svg+xml;utf8,<svg xmlns='http://www.w3.org/2000/svg' width='2px' height='2px'><rect width='1' height='1' fill='blue'/></svg>");
shape-margin: 500px;
margin-right: 50px;
margin-bottom: 50px;
width: 100px;
height: 50px;
}
</style>
</head>
<body>
<p>The shape-outside in this test is a 2x2 image with only the upper left pixel set. The shape
element's margin-box is and the the shape-margin is big enough (is greater than 150 * sqrt(2))
so that the shape-margin boundary extends beyond the margin-box's bounds. The shape-outside boundary
is clipped to the margin-box, so it's as if the shape-outside was equal to the margin-box.</p>
<div id="content">
<div id="image-shape"></div>
X<br>X
</div>
</body>
</html>
third_party/WebKit/Source/core/rendering/shapes/RasterShape.cpp
View file @ e137f062
......@@ -255,10 +255,11 @@ const RasterShapeIntervals& RasterShape::marginIntervals() const
if (!shapeMargin())
return *m_intervals;
int shapeMarginInt = clampToPositiveInteger(ceil(shapeMargin()));
int maxShapeMarginInt = std::max(m_marginRectSize.width(), m_marginRectSize.height()) * sqrt(2);
if (!m_marginIntervals)
m_marginIntervals = m_intervals->computeShapeMarginIntervals(std::min(shapeMarginInt, maxShapeMarginInt));
int marginBoundaryRadius = std::min(clampToInteger(ceil(shapeMargin())), std::max(m_imageSize.width(), m_imageSize.height()));
if (!m_marginIntervals) {
ASSERT(marginBoundaryRadius >= 0);
m_marginIntervals = m_intervals->computeShapeMarginIntervals(marginBoundaryRadius);
}
return *m_marginIntervals;
}
......
third_party/WebKit/Source/core/rendering/shapes/RasterShape.h
View file @ e137f062
......@@ -92,9 +92,9 @@ private:
class RasterShape FINAL : public Shape {
WTF_MAKE_NONCOPYABLE(RasterShape);
public:
RasterShape(PassOwnPtr<RasterShapeIntervals> intervals, const IntSize& marginRectSize)
RasterShape(PassOwnPtr<RasterShapeIntervals> intervals, const IntSize& imageSize)
: m_intervals(intervals)
, m_marginRectSize(marginRectSize)
, m_imageSize(imageSize)
{
}
......@@ -111,7 +111,7 @@ private:
OwnPtr<RasterShapeIntervals> m_intervals;
mutable OwnPtr<RasterShapeIntervals> m_marginIntervals;
IntSize m_marginRectSize;
IntSize m_imageSize;
};
} // namespace WebCore
......
third_party/WebKit/Source/core/rendering/shapes/Shape.h
View file @ e137f062
......@@ -62,7 +62,7 @@ typedef Vector<LineSegment> SegmentList;
class Shape {
public:
static PassOwnPtr<Shape> createShape(const BasicShape*, const LayoutSize& logicalBoxSize, WritingMode, Length margin, Length padding);
static PassOwnPtr<Shape> createRasterShape(Image*, float threshold, const LayoutRect& imageRect, const LayoutRect& marginRect, WritingMode, Length margin, Length padding);
static PassOwnPtr<Shape> createRasterShape(Image*, float threshold, const LayoutRect& drawRect, const LayoutRect& clipRect, WritingMode, Length margin, Length padding);
static PassOwnPtr<Shape> createLayoutBoxShape(const RoundedRect&, WritingMode, const Length& margin, const Length& padding);
virtual ~Shape() { }
......
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