Bug 1579231. Add some additional debugging output to blob grouping. r=mstange

Also rearranges some corresponding code.

Differential Revision: https://phabricator.services.mozilla.com/D44902

--HG--
extra : moz-landing-system : lando
This commit is contained in:
Jeff Muizelaar 2019-09-05 22:21:23 +00:00
parent d46adaea8a
commit 725bfc2aef

View File

@ -1470,6 +1470,16 @@ void WebRenderCommandBuilder::DoGroupingForDisplayList(
// allocating much larger textures than necessary in webrender.
//
// Dont bother fixing this unless we run into this in the real world, though.
auto layerBounds = LayerIntRect::FromUnknownRect(
ScaleToOutsidePixelsOffset(groupBounds, scale.width, scale.height,
appUnitsPerDevPixel, residualOffset));
GP("scale: %f %f - %d - %f %f\n", scale.width, scale.height,
group.mAppUnitsPerDevPixel, residualOffset.x, residualOffset.y);
GP("LayerBounds: %d %d %d %d\n", layerBounds.x, layerBounds.y,
layerBounds.width, layerBounds.height);
GP("Inherrited scale %f %f\n", scale.width, scale.height);
GP("Bounds: %d %d %d %d vs %d %d %d %d\n", p.x, p.y, p.width, p.height, q.x,
q.y, q.width, q.height);
if (!group.mGroupBounds.IsEqualEdges(groupBounds) ||
@ -1514,10 +1524,8 @@ void WebRenderCommandBuilder::DoGroupingForDisplayList(
group.mResidualOffset = residualOffset;
group.mGroupBounds = groupBounds;
group.mAppUnitsPerDevPixel = appUnitsPerDevPixel;
group.mLayerBounds = LayerIntRect::FromUnknownRect(
ScaleToOutsidePixelsOffset(group.mGroupBounds, scale.width, scale.height,
group.mAppUnitsPerDevPixel, residualOffset));
group.mImageBounds = group.mLayerBounds.ToUnknownRect();
group.mLayerBounds = layerBounds;
group.mImageBounds = layerBounds.ToUnknownRect();
group.mClippedImageBounds = group.mImageBounds;
const nsRect& untransformedPaintRect =