Commit 5e6b9100 authored by Chris Harrelson's avatar Chris Harrelson Committed by Commit Bot

Enable GeometryMapper in one more use case in compositing

This is now possible because PrePaint is before CompositingAssignments.

Change-Id: Id6ba9873834ceb2d96f55f15dc50f4a34b430a7f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2343772Reviewed-by: default avatarXianzhu Wang <wangxianzhu@chromium.org>
Commit-Queue: Chris Harrelson <chrishtr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#796222}
parent 7dc24677
......@@ -1641,15 +1641,13 @@ void CompositedLayerMapping::UpdateLocalClipRectForSquashedLayer(
// disallowed squashing.
DCHECK(ancestor_paint_info);
// FIXME: this is a potential performance issue. We should consider caching
// these clip rects or otherwise optimizing.
ClipRectsContext clip_rects_context(
ancestor_paint_info->paint_layer,
&ancestor_paint_info->paint_layer->GetLayoutObject().FirstFragment(),
kUncachedClipRects);
ClipRect parent_clip_rect;
paint_info.paint_layer
->Clipper(PaintLayer::GeometryMapperOption::kDoNotUseGeometryMapper)
->Clipper(PaintLayer::GeometryMapperOption::kUseGeometryMapper)
.CalculateBackgroundClipRect(clip_rects_context, parent_clip_rect);
// Convert from ancestor to local coordinates.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment