Files
serenity/Tests/LibGfx/TestPath.cpp
Nico Weber 66378471a0 LibGfx: When stroking zero-lengths paths, draw round and square end caps
PDF 1.7 spec, p231, 4.4.2 Path-Painting Operators, Stroking:

"If a subpath is degenerate (consists of a single-point closed path or
of two or more points at the same coordinates), the S operator paints it
only if round line caps have been specified, producing a filled circle
centered at the single point. If butt or projecting square line caps
have been specified, S produces no output, because the orientation of
the caps would be indeterminate. (This rule applies only to zero-length
subpaths of the path being stroked, and not to zero-length dashes in a
dash pattern. In the latter case, the line caps are always painted,
since their orientation is determined by the direction of the underlying
path.) A single-point open subpath (specified by a trailing m operator)
produces no output."

In practice, Chrome, Firefox, and Preview all also draw a square
for square line endings (Preview a square rotated 45 degrees). Chrome
even draws something weird-looking for butt caps. (Acrobat only draws
the round cap, per spec.)

https://html.spec.whatwg.org/multipage/canvas.html#trace-a-path sounds
like zero-lengths paths should be ignored for canvas, but in practice
Chrome and Firefox do draw them. (Safari doesn't.)

We don't do linecaps in SVGs yet, but
https://www.w3.org/TR/SVG/paths.html#ZeroLengthSegments says:

"As mentioned in Stroke Properties, linecaps must be painted for
zero-length subpaths when stroke-linecap has a value of round or
square."

With this commit, we now draw round and square linecaps for
zero-lengths paths, which is what's apparently desired most of the
time. Maybe we can add a setting to pick different behavior for
PDF (only draw round caps), canvas (don't draw caps on zero-length
paths), and SVG (draw round and square caps) in the future.
2024-10-16 18:18:43 -04:00

113 lines
3.1 KiB
C++

/*
* Copyright (c) 2024, Nico Weber <thakis@chromium.org>
*
* SPDX-License-Identifier: BSD-2-Clause
*/
#include <LibTest/TestCase.h>
#include <LibGfx/Path.h>
TEST_CASE(path_to_fill_short_wide_line_with_butt_linecap)
{
// Test drawing a horizontal rect by stroking a vertical short wide line.
{
int width = 100;
int height = 1;
Gfx::Path path;
path.move_to({ width / 2, 0 });
path.line_to({ width / 2, height });
auto fill = path.stroke_to_fill(width, Gfx::Path::CapStyle::Butt);
EXPECT_EQ(fill.bounding_box(), Gfx::FloatRect({ 0, 0 }, { width, height }));
}
// Test drawing a vertical rect by stroking a horizontal short wide line.
{
int width = 1;
int height = 100;
Gfx::Path path;
path.move_to({ 0, height / 2 });
path.line_to({ width, height / 2 });
auto fill = path.stroke_to_fill(height, Gfx::Path::CapStyle::Butt);
EXPECT_EQ(fill.bounding_box(), Gfx::FloatRect({ 0, 0 }, { width, height }));
}
}
TEST_CASE(path_to_fill_square_linecap)
{
int line_width = 10;
int width = 100;
Gfx::Path path;
path.move_to({ line_width / 2, line_width / 2 });
path.line_to({ width - line_width / 2, line_width / 2 });
auto fill = path.stroke_to_fill(line_width, Gfx::Path::CapStyle::Square);
EXPECT_EQ(fill.bounding_box(), Gfx::FloatRect({ 0, 0 }, { width, line_width }));
}
TEST_CASE(path_to_fill_single_point)
{
Gfx::Path path;
path.move_to({ 10, 10 });
path.line_to({ 10, 10 });
{
auto fill = path.stroke_to_fill(8, Gfx::Path::CapStyle::Butt);
EXPECT(fill.is_empty());
}
{
auto fill = path.stroke_to_fill(8, Gfx::Path::CapStyle::Round);
EXPECT(!fill.is_empty());
}
{
auto fill = path.stroke_to_fill(8, Gfx::Path::CapStyle::Square);
EXPECT_EQ(fill.bounding_box(), Gfx::FloatRect({ 6, 6 }, { 8, 8 }));
}
}
TEST_CASE(path_to_fill_two_single_points)
{
Gfx::Path path;
path.move_to({ 10, 10 });
path.line_to({ 10, 10 });
path.move_to({ 20, 20 });
path.line_to({ 20, 20 });
{
auto fill = path.stroke_to_fill(8, Gfx::Path::CapStyle::Butt);
EXPECT(fill.is_empty());
}
{
auto fill = path.stroke_to_fill(8, Gfx::Path::CapStyle::Round);
EXPECT(!fill.is_empty());
}
{
auto fill = path.stroke_to_fill(8, Gfx::Path::CapStyle::Square);
EXPECT(!fill.is_empty());
}
}
TEST_CASE(path_to_string)
{
{
Gfx::Path path;
path.move_to({ 10, 10 });
path.line_to({ 20, 20 });
path.quadratic_bezier_curve_to({ 30, 30 }, { 40, 40 });
path.cubic_bezier_curve_to({ 50, 50 }, { 60, 60 }, { 10, 10 });
path.close();
EXPECT_EQ(path.to_byte_string(), "M 10,10 L 20,20 Q 30,30 40,40 C 50,50 60,60 10,10 Z");
}
{
Gfx::Path path;
path.move_to({ 10, 10 });
path.line_to({ 20, 20 });
path.quadratic_bezier_curve_to({ 30, 30 }, { 40, 40 });
path.cubic_bezier_curve_to({ 50, 50 }, { 60, 60 }, { 10, 10 });
EXPECT_EQ(path.to_byte_string(), "M 10,10 L 20,20 Q 30,30 40,40 C 50,50 60,60 10,10");
}
}