Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Commit 9159986

Browse files
committed
add Skia bug number to workaround comment
1 parent 444f8c1 commit 9159986

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

flow/layers/layer.h

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -190,10 +190,11 @@ class Layer {
190190
// Determines if the Paint() method is necessary based on the properties
191191
// of the indicated PaintContext object.
192192
bool needs_painting(PaintContext& context) const {
193-
// Skia bug - quickReject returns false if the bounds are empty.
194-
// (Waiting for Skia to acknowledge test case and file a bug.)
195-
if (paint_bounds_.isEmpty())
193+
// Workaround for Skia bug (quickReject does not reject empty bounds).
194+
// https://bugs.chromium.org/p/skia/issues/detail?id=10951
195+
if (paint_bounds_.isEmpty()) {
196196
return false;
197+
}
197198
return !context.internal_nodes_canvas->quickReject(paint_bounds_);
198199
}
199200

0 commit comments

Comments
 (0)