Skip to content

Commit

Permalink
Revert "Remove empty closed paths"
Browse files Browse the repository at this point in the history
This reverts commit 572074cb06425797e7e110511db405134cf67e2f.

Reason for revert:
caused regression bug: crbug.com/1521614

Original change's description:
> Remove empty closed paths
>
> As discussed in the github issue,
> whatwg/html#1079, it's generally agreed to
> skip drawing line caps for closed paths. This is a follow up cl of:
> https://chromium-review.googlesource.com/c/chromium/src/+/4864061.
>
> In this cl, it verifies if each closed path is empty, then it removes
> the empty closed subpaths from path. As the subpaths are removed, the
> corresponding line caps are skipped.
>
> Bug: 644067
>
> Change-Id: I14c5c4819a31eb9249d01574ff1b5c60826c3b9e
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4916516
> Commit-Queue: Yi Xu <[email protected]>
> Reviewed-by: Jean-Philippe Gravel <[email protected]>
> Reviewed-by: Justin Novosad <[email protected]>
> Cr-Commit-Position: refs/heads/main@{#1247735}

Bug: 644067
Change-Id: I33ecfd76574d6f60e6f8c471a38183f87757e3ef
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5258624
Reviewed-by: Aaron Krajeski <[email protected]>
Reviewed-by: Jean-Philippe Gravel <[email protected]>
Commit-Queue: Yi Xu <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1255711}
  • Loading branch information
Yi Xu authored and chromium-wpt-export-bot committed Feb 2, 2024
1 parent f1d9aa9 commit 64b75a9
Show file tree
Hide file tree
Showing 21 changed files with 0 additions and 852 deletions.

This file was deleted.

This file was deleted.

This file was deleted.

This file was deleted.

This file was deleted.

This file was deleted.

This file was deleted.

This file was deleted.

This file was deleted.

This file was deleted.

This file was deleted.

This file was deleted.

Loading

0 comments on commit 64b75a9

Please sign in to comment.