Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(86)

Issue 2560843002: Don't fail clip-paths with empty bounds (Closed)

Created:
4 years ago by fs
Modified:
4 years ago
Reviewers:
CC:
chromium-reviews
Target Ref:
refs/pending/branch-heads/2924
Project:
chromium
Visibility:
Public.

Description

Don't fail clip-paths with empty bounds An empty nested clip-path should result in an empty clip-path (clipping away everything.) BUG=671543 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Review-Url: https://codereview.chromium.org/2555483003 Cr-Commit-Position: refs/heads/master@{#436605} (cherry picked from commit 4bfec8883cac1e879727b3f95ae6de1b19146473) Committed: https://chromium.googlesource.com/chromium/src/+/26e44a294ddacf179061579505c4b5f6c6d4aa1d

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+15 lines, -1 line) Patch
M third_party/WebKit/LayoutTests/FlagExpectations/enable-slimming-paint-v2 View 1 chunk +1 line, -0 lines 0 comments Download
A third_party/WebKit/LayoutTests/svg/clip-path/nested-empty-clip.html View 1 chunk +11 lines, -0 lines 0 comments Download
A third_party/WebKit/LayoutTests/svg/clip-path/nested-empty-clip-expected.html View 1 chunk +2 lines, -0 lines 0 comments Download
M third_party/WebKit/Source/core/paint/SVGClipPainter.cpp View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 2 (1 generated)
fs
4 years ago (2016-12-07 16:01:40 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
26e44a294ddacf179061579505c4b5f6c6d4aa1d.

Powered by Google App Engine
This is Rietveld 408576698