Commit 238442fb authored by John Budorick's avatar John Budorick Committed by Commit Bot

Add two missing fields to RenderPassSize.

crrev.com/c/2047453 appears to have changed the layout of RenderPassSize
in some cases, causing it to not match the size of RenderPass due to two
missing boolean fields, e.g.

https://ci.chromium.org/p/chromium/builders/ci/android-pie-x86-fyi-rel/365

This CL adds the two missing fields.

Bug: 1022533
Change-Id: Icd7a275adc671baa5f989cfd7421799e4f2f289d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2062608Reviewed-by: default avatarccameron <ccameron@chromium.org>
Reviewed-by: default avatarKhushal <khushalsagar@chromium.org>
Commit-Queue: John Budorick <jbudorick@chromium.org>
Cr-Commit-Position: refs/heads/master@{#742206}
parent 0cce5e77
......@@ -32,6 +32,8 @@ struct RenderPassSize {
base::Optional<gfx::RRectF> backdrop_filter_bounds;
gfx::ContentColorUsage content_color_usage;
bool has_transparent_background;
bool cache_render_pass;
bool has_damage_from_contributing_component;
bool generate_mipmap;
std::vector<std::unique_ptr<CopyOutputRequest>> copy_callbacks;
QuadList quad_list;
......@@ -120,6 +122,9 @@ TEST(RenderPassTest, CopyShouldBeIdenticalExceptIdAndQuads) {
copy->backdrop_filter_bounds.value(), 0.001));
EXPECT_EQ(pass->content_color_usage, copy->content_color_usage);
EXPECT_EQ(pass->has_transparent_background, copy->has_transparent_background);
EXPECT_EQ(pass->cache_render_pass, copy->cache_render_pass);
EXPECT_EQ(pass->has_damage_from_contributing_content,
copy->has_damage_from_contributing_content);
EXPECT_EQ(pass->generate_mipmap, copy->generate_mipmap);
EXPECT_EQ(0u, copy->quad_list.size());
......
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