Commit 65a674b7 authored by Xianzhu Wang's avatar Xianzhu Wang Committed by Commit Bot

Update RasterInvalidatorTest.ImplicitFullLayerInvalidation

The line "ASSERT_EQ(1u, invalidations.size());" was mistakenly removed
in https://chromium-review.googlesource.com/c/chromium/src/+/1955484/3/third_party/blink/renderer/platform/graphics/paint/raster_invalidator_test.cc#b98.

Change the test to use EXPECT_THAT(..., ElementsAre()) for better
readability of code and failure message.

TBR=pdr@chromium.org

Change-Id: If818878796390e86d5b30761be27b985c2593516
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1955486Reviewed-by: default avatarXianzhu Wang <wangxianzhu@chromium.org>
Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#722717}
parent 2ca1525c
...@@ -98,10 +98,12 @@ TEST_P(RasterInvalidatorTest, ImplicitFullLayerInvalidation) { ...@@ -98,10 +98,12 @@ TEST_P(RasterInvalidatorTest, ImplicitFullLayerInvalidation) {
invalidator_.SetTracksRasterInvalidations(true); invalidator_.SetTracksRasterInvalidations(true);
invalidator_.Generate(artifact, kDefaultLayerBounds, invalidator_.Generate(artifact, kDefaultLayerBounds,
DefaultPropertyTreeState()); DefaultPropertyTreeState());
const auto& invalidations = TrackedRasterInvalidations(); const auto& client = artifact->PaintChunks()[0].id.client;
EXPECT_EQ(IntRect(IntPoint(), kDefaultLayerBounds.Size()), EXPECT_THAT(TrackedRasterInvalidations(),
invalidations[0].rect); ElementsAre(RasterInvalidationInfo{
EXPECT_EQ(PaintInvalidationReason::kFullLayer, invalidations[0].reason); &client, client.DebugName(),
IntRect(IntPoint(), kDefaultLayerBounds.Size()),
PaintInvalidationReason::kFullLayer}));
FinishCycle(*artifact); FinishCycle(*artifact);
invalidator_.SetTracksRasterInvalidations(false); invalidator_.SetTracksRasterInvalidations(false);
} }
......
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