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

Commit fa80c7b

Browse files
author
jonahwilliams
committed
test tweaks
1 parent e15b5f3 commit fa80c7b

File tree

3 files changed

+8
-7
lines changed

3 files changed

+8
-7
lines changed

impeller/aiks/aiks_unittests.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2440,7 +2440,7 @@ TEST_P(AiksTest, OpacityPeepHoleApplicationTest) {
24402440
Entity entity;
24412441
entity.SetContents(SolidColorContents::Make(
24422442
PathBuilder{}.AddRect(rect).TakePath(), Color::Red()));
2443-
entity_pass->AddEntity(entity);
2443+
entity_pass->AddEntity(std::move(entity));
24442444
paint.color = Color::Red().WithAlpha(0.5);
24452445

24462446
delegate = std::make_shared<OpacityPeepholePassDelegate>(paint);

impeller/aiks/canvas.cc

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -71,9 +71,10 @@ static std::shared_ptr<Contents> CreateContentsForGeometryWithFilters(
7171
return contents_copy;
7272
}
7373

74-
static std::shared_ptr<Contents> CreateContentsWithFilters(const Paint& paint,
75-
const Path& path = {},
76-
bool cover = false) {
74+
static std::shared_ptr<Contents> CreateContentsWithFilters(
75+
const Paint& paint,
76+
const Path& path = {},
77+
bool cover = false) {
7778
std::shared_ptr<Geometry> geometry;
7879
if (cover) {
7980
geometry = Geometry::MakeCover();

impeller/entity/entity_unittests.cc

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,7 @@ auto CreatePassWithRectPath(Rect rect,
103103
Entity entity;
104104
entity.SetContents(SolidColorContents::Make(
105105
PathBuilder{}.AddRect(rect).TakePath(), Color::Red()));
106-
subpass->AddEntity(entity);
106+
subpass->AddEntity(std::move(entity));
107107
subpass->SetDelegate(std::make_unique<TestPassDelegate>(collapse));
108108
subpass->SetBoundsLimit(bounds_hint);
109109
return subpass;
@@ -152,7 +152,7 @@ TEST_P(EntityTest, EntityPassCanMergeSubpassIntoParent) {
152152
contents->SetColor(Color::Blue());
153153
entity.SetContents(std::move(contents));
154154

155-
pass.AddEntity(entity);
155+
pass.AddEntity(std::move(entity));
156156

157157
ASSERT_TRUE(OpenPlaygroundHere(pass));
158158
}
@@ -2465,7 +2465,7 @@ TEST_P(EntityTest, AdvancedBlendCoverageHintIsNotResetByEntityPass) {
24652465
RenderTarget::kDefaultColorAttachmentConfig, stencil_config);
24662466
auto content_context = ContentContext(
24672467
GetContext(), TypographerContextSkia::Make(), test_allocator);
2468-
pass->AddEntity(entity);
2468+
pass->AddEntity(std::move(entity));
24692469

24702470
EXPECT_TRUE(pass->Render(content_context, rt));
24712471

0 commit comments

Comments
 (0)