mirror of
https://github.com/mozilla/gecko-dev.git
synced 2024-11-26 06:11:37 +00:00
Bug 1075399, part 4 - Don't make an exception for arcs when simulating zero length SVG subpaths. r=longsonr
This commit is contained in:
Jonathan Watt
parent
be41e2dbec
commit
16d9ba973d
@ -262,7 +262,6 @@ ApproximateZeroLengthSubpathSquareCaps(PathBuilder* aPB,
|
||||
|
||||
Float tinyLength = aStrokeWidth / 512;
|
||||
|
||||
aPB->MoveTo(aPoint);
|
||||
aPB->LineTo(aPoint + Point(tinyLength, 0));
|
||||
aPB->MoveTo(aPoint);
|
||||
}
|
||||
@ -285,7 +284,8 @@ ApproximateZeroLengthSubpathSquareCaps(const gfxPoint &aPoint, gfxContext *aCtx)
|
||||
#define MAYBE_APPROXIMATE_ZERO_LENGTH_SUBPATH_SQUARE_CAPS_TO_DT \
|
||||
do { \
|
||||
if (!subpathHasLength && hasLineCaps && aStrokeWidth > 0 && \
|
||||
subpathContainsNonArc && SVGPathSegUtils::IsValidType(prevSegType) && \
|
||||
subpathContainsNonMoveTo && \
|
||||
SVGPathSegUtils::IsValidType(prevSegType) && \
|
||||
(!IsMoveto(prevSegType) || segType == PATHSEG_CLOSEPATH)) { \
|
||||
ApproximateZeroLengthSubpathSquareCaps(builder, segStart, aStrokeWidth);\
|
||||
} \
|
||||
@ -312,7 +312,7 @@ SVGPathData::BuildPath(PathBuilder* builder,
|
||||
|
||||
bool hasLineCaps = aStrokeLineCap != NS_STYLE_STROKE_LINECAP_BUTT;
|
||||
bool subpathHasLength = false; // visual length
|
||||
bool subpathContainsNonArc = false;
|
||||
bool subpathContainsNonMoveTo = false;
|
||||
|
||||
uint32_t segType = PATHSEG_UNKNOWN;
|
||||
uint32_t prevSegType = PATHSEG_UNKNOWN;
|
||||
@ -335,7 +335,7 @@ SVGPathData::BuildPath(PathBuilder* builder,
|
||||
{
|
||||
case PATHSEG_CLOSEPATH:
|
||||
// set this early to allow drawing of square caps for "M{x},{y} Z":
|
||||
subpathContainsNonArc = true;
|
||||
subpathContainsNonMoveTo = true;
|
||||
MAYBE_APPROXIMATE_ZERO_LENGTH_SUBPATH_SQUARE_CAPS_TO_DT;
|
||||
segEnd = pathStart;
|
||||
builder->Close();
|
||||
@ -346,7 +346,6 @@ SVGPathData::BuildPath(PathBuilder* builder,
|
||||
pathStart = segEnd = Point(mData[i], mData[i+1]);
|
||||
builder->MoveTo(segEnd);
|
||||
subpathHasLength = false;
|
||||
subpathContainsNonArc = false;
|
||||
break;
|
||||
|
||||
case PATHSEG_MOVETO_REL:
|
||||
@ -354,7 +353,6 @@ SVGPathData::BuildPath(PathBuilder* builder,
|
||||
pathStart = segEnd = segStart + Point(mData[i], mData[i+1]);
|
||||
builder->MoveTo(segEnd);
|
||||
subpathHasLength = false;
|
||||
subpathContainsNonArc = false;
|
||||
break;
|
||||
|
||||
case PATHSEG_LINETO_ABS:
|
||||
@ -363,7 +361,6 @@ SVGPathData::BuildPath(PathBuilder* builder,
|
||||
subpathHasLength = true;
|
||||
builder->LineTo(segEnd);
|
||||
}
|
||||
subpathContainsNonArc = true;
|
||||
break;
|
||||
|
||||
case PATHSEG_LINETO_REL:
|
||||
@ -372,7 +369,6 @@ SVGPathData::BuildPath(PathBuilder* builder,
|
||||
subpathHasLength = true;
|
||||
builder->LineTo(segEnd);
|
||||
}
|
||||
subpathContainsNonArc = true;
|
||||
break;
|
||||
|
||||
case PATHSEG_CURVETO_CUBIC_ABS:
|
||||
@ -383,7 +379,6 @@ SVGPathData::BuildPath(PathBuilder* builder,
|
||||
subpathHasLength = true;
|
||||
builder->BezierTo(cp1, cp2, segEnd);
|
||||
}
|
||||
subpathContainsNonArc = true;
|
||||
break;
|
||||
|
||||
case PATHSEG_CURVETO_CUBIC_REL:
|
||||
@ -394,7 +389,6 @@ SVGPathData::BuildPath(PathBuilder* builder,
|
||||
subpathHasLength = true;
|
||||
builder->BezierTo(cp1, cp2, segEnd);
|
||||
}
|
||||
subpathContainsNonArc = true;
|
||||
break;
|
||||
|
||||
case PATHSEG_CURVETO_QUADRATIC_ABS:
|
||||
@ -407,7 +401,6 @@ SVGPathData::BuildPath(PathBuilder* builder,
|
||||
subpathHasLength = true;
|
||||
builder->BezierTo(tcp1, tcp2, segEnd);
|
||||
}
|
||||
subpathContainsNonArc = true;
|
||||
break;
|
||||
|
||||
case PATHSEG_CURVETO_QUADRATIC_REL:
|
||||
@ -420,7 +413,6 @@ SVGPathData::BuildPath(PathBuilder* builder,
|
||||
subpathHasLength = true;
|
||||
builder->BezierTo(tcp1, tcp2, segEnd);
|
||||
}
|
||||
subpathContainsNonArc = true;
|
||||
break;
|
||||
|
||||
case PATHSEG_ARC_ABS:
|
||||
@ -452,7 +444,6 @@ SVGPathData::BuildPath(PathBuilder* builder,
|
||||
subpathHasLength = true;
|
||||
builder->LineTo(segEnd);
|
||||
}
|
||||
subpathContainsNonArc = true;
|
||||
break;
|
||||
|
||||
case PATHSEG_LINETO_HORIZONTAL_REL:
|
||||
@ -461,7 +452,6 @@ SVGPathData::BuildPath(PathBuilder* builder,
|
||||
subpathHasLength = true;
|
||||
builder->LineTo(segEnd);
|
||||
}
|
||||
subpathContainsNonArc = true;
|
||||
break;
|
||||
|
||||
case PATHSEG_LINETO_VERTICAL_ABS:
|
||||
@ -470,7 +460,6 @@ SVGPathData::BuildPath(PathBuilder* builder,
|
||||
subpathHasLength = true;
|
||||
builder->LineTo(segEnd);
|
||||
}
|
||||
subpathContainsNonArc = true;
|
||||
break;
|
||||
|
||||
case PATHSEG_LINETO_VERTICAL_REL:
|
||||
@ -479,7 +468,6 @@ SVGPathData::BuildPath(PathBuilder* builder,
|
||||
subpathHasLength = true;
|
||||
builder->LineTo(segEnd);
|
||||
}
|
||||
subpathContainsNonArc = true;
|
||||
break;
|
||||
|
||||
case PATHSEG_CURVETO_CUBIC_SMOOTH_ABS:
|
||||
@ -490,7 +478,6 @@ SVGPathData::BuildPath(PathBuilder* builder,
|
||||
subpathHasLength = true;
|
||||
builder->BezierTo(cp1, cp2, segEnd);
|
||||
}
|
||||
subpathContainsNonArc = true;
|
||||
break;
|
||||
|
||||
case PATHSEG_CURVETO_CUBIC_SMOOTH_REL:
|
||||
@ -501,7 +488,6 @@ SVGPathData::BuildPath(PathBuilder* builder,
|
||||
subpathHasLength = true;
|
||||
builder->BezierTo(cp1, cp2, segEnd);
|
||||
}
|
||||
subpathContainsNonArc = true;
|
||||
break;
|
||||
|
||||
case PATHSEG_CURVETO_QUADRATIC_SMOOTH_ABS:
|
||||
@ -514,7 +500,6 @@ SVGPathData::BuildPath(PathBuilder* builder,
|
||||
subpathHasLength = true;
|
||||
builder->BezierTo(tcp1, tcp2, segEnd);
|
||||
}
|
||||
subpathContainsNonArc = true;
|
||||
break;
|
||||
|
||||
case PATHSEG_CURVETO_QUADRATIC_SMOOTH_REL:
|
||||
@ -527,13 +512,15 @@ SVGPathData::BuildPath(PathBuilder* builder,
|
||||
subpathHasLength = true;
|
||||
builder->BezierTo(tcp1, tcp2, segEnd);
|
||||
}
|
||||
subpathContainsNonArc = true;
|
||||
break;
|
||||
|
||||
default:
|
||||
NS_NOTREACHED("Bad path segment type");
|
||||
return nullptr; // according to spec we'd use everything up to the bad seg anyway
|
||||
}
|
||||
|
||||
subpathContainsNonMoveTo = segType != PATHSEG_MOVETO_ABS &&
|
||||
segType != PATHSEG_MOVETO_REL;
|
||||
i += argCount;
|
||||
prevSegType = segType;
|
||||
segStart = segEnd;
|
||||
|
||||
@ -66,8 +66,10 @@ path.coverer {
|
||||
</g>
|
||||
|
||||
<g transform="translate(25,125)">
|
||||
<path class="circles-not-expected" d="M0,0 A0,10 0 0 0 0,0 M20,20 L30,30 M50,50 A0,10 0 0 0 50,50 M70,70 L80,80 M100,100 A0,10 0 0 0 100,100"/>
|
||||
<path class="coverer" d="M20,20 L30,30 M70,70 L80,80"/>
|
||||
<circle cx="0" cy="0" r="8"/>
|
||||
<circle cx="50" cy="50" r="8"/>
|
||||
<circle cx="100" cy="100" r="8"/>
|
||||
<path class="circles-expected" d="M0,0 A0,10 0 0 0 0,0 M20,20 L30,30 M50,50 A0,10 0 0 0 50,50 M70,70 L80,80 M100,100 A0,10 0 0 0 100,100"/>
|
||||
</g>
|
||||
|
||||
<g transform="translate(25,175)">
|
||||
@ -95,8 +97,10 @@ path.coverer {
|
||||
</g>
|
||||
|
||||
<g transform="translate(175,125)">
|
||||
<path class="circles-not-expected" d="M0,0 A0,10 0 0 0 0,0 A0,10 0 0 0 0,0 M20,20 L30,30 M50,50 A0,10 0 0 0 50,50 A0,10 0 0 0 50,50 M70,70 L80,80 M100,100 A0,10 0 0 0 100,100 A0,10 0 0 0 100,100"/>
|
||||
<path class="coverer" d="M20,20 L30,30 M70,70 L80,80"/>
|
||||
<circle cx="0" cy="0" r="8"/>
|
||||
<circle cx="50" cy="50" r="8"/>
|
||||
<circle cx="100" cy="100" r="8"/>
|
||||
<path class="circles-expected" d="M0,0 A0,10 0 0 0 0,0 A0,10 0 0 0 0,0 M20,20 L30,30 M50,50 A0,10 0 0 0 50,50 A0,10 0 0 0 50,50 M70,70 L80,80 M100,100 A0,10 0 0 0 100,100 A0,10 0 0 0 100,100"/>
|
||||
</g>
|
||||
|
||||
<g transform="translate(175,175)">
|
||||
|
||||
|
Before Width: | Height: | Size: 4.5 KiB After Width: | Height: | Size: 4.6 KiB |
@ -66,8 +66,10 @@ path.coverer {
|
||||
</g>
|
||||
|
||||
<g transform="translate(25,125)">
|
||||
<path class="squares-not-expected" d="M0,0 A0,10 0 0 0 0,0 M20,20 L30,30 M50,50 A0,10 0 0 0 50,50 M70,70 L80,80 M100,100 A0,10 0 0 0 100,100"/>
|
||||
<path class="coverer" d="M20,20 L30,30 M70,70 L80,80"/>
|
||||
<rect x="-9" y="-9" width="18" height="18"/>
|
||||
<rect x="41" y="41" width="18" height="18"/>
|
||||
<rect x="91" y="91" width="18" height="18"/>
|
||||
<path class="squares-expected" d="M0,0 A0,10 0 0 0 0,0 M20,20 L30,30 M50,50 A0,10 0 0 0 50,50 M70,70 L80,80 M100,100 A0,10 0 0 0 100,100"/>
|
||||
</g>
|
||||
|
||||
<g transform="translate(25,175)">
|
||||
@ -95,8 +97,10 @@ path.coverer {
|
||||
</g>
|
||||
|
||||
<g transform="translate(175,125)">
|
||||
<path class="squares-not-expected" d="M0,0 A0,10 0 0 0 0,0 A0,10 0 0 0 0,0 M20,20 L30,30 M50,50 A0,10 0 0 0 50,50 A0,10 0 0 0 50,50 M70,70 L80,80 M100,100 A0,10 0 0 0 100,100 A0,10 0 0 0 100,100"/>
|
||||
<path class="coverer" d="M20,20 L30,30 M70,70 L80,80"/>
|
||||
<rect x="-9" y="-9" width="18" height="18"/>
|
||||
<rect x="41" y="41" width="18" height="18"/>
|
||||
<rect x="91" y="91" width="18" height="18"/>
|
||||
<path class="squares-expected" d="M0,0 A0,10 0 0 0 0,0 A0,10 0 0 0 0,0 M20,20 L30,30 M50,50 A0,10 0 0 0 50,50 A0,10 0 0 0 50,50 M70,70 L80,80 M100,100 A0,10 0 0 0 100,100 A0,10 0 0 0 100,100"/>
|
||||
</g>
|
||||
|
||||
<g transform="translate(175,175)">
|
||||
|
||||
|
Before Width: | Height: | Size: 4.7 KiB After Width: | Height: | Size: 4.9 KiB |
Loading…
Reference in New Issue
Block a user