Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(448)

Unified Diff: cc/surfaces/surface_aggregator_perftest.cc

Issue 2449853004: Getting rid of DelegatedFrameData (Closed)
Patch Set: IsEmpty + rebase Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: cc/surfaces/surface_aggregator_perftest.cc
diff --git a/cc/surfaces/surface_aggregator_perftest.cc b/cc/surfaces/surface_aggregator_perftest.cc
index 828518d732562355153294e514872dd7fa5fdbf4..73f617442caf8cfa32a0b1f483a961ca5487880a 100644
--- a/cc/surfaces/surface_aggregator_perftest.cc
+++ b/cc/surfaces/surface_aggregator_perftest.cc
@@ -5,7 +5,6 @@
#include "base/memory/ptr_util.h"
#include "cc/debug/lap_timer.h"
#include "cc/output/compositor_frame.h"
-#include "cc/output/delegated_frame_data.h"
#include "cc/quads/surface_draw_quad.h"
#include "cc/quads/texture_draw_quad.h"
#include "cc/surfaces/surface_aggregator.h"
@@ -54,14 +53,14 @@ class SurfaceAggregatorPerfTest : public testing::Test {
LocalFrameId local_frame_id(i, 0);
factory_.Create(local_frame_id);
std::unique_ptr<RenderPass> pass(RenderPass::Create());
- std::unique_ptr<DelegatedFrameData> frame_data(new DelegatedFrameData);
+ CompositorFrame frame;
SharedQuadState* sqs = pass->CreateAndAppendSharedQuadState();
for (int j = 0; j < num_textures; j++) {
TransferableResource resource;
resource.id = j;
resource.is_software = true;
- frame_data->resource_list.push_back(resource);
+ frame.resource_list.push_back(resource);
TextureDrawQuad* quad =
pass->CreateAndAppendDrawQuad<TextureDrawQuad>();
@@ -93,9 +92,7 @@ class SurfaceAggregatorPerfTest : public testing::Test {
SurfaceId(kArbitraryFrameSinkId, LocalFrameId(i - 1, 0)));
}
- frame_data->render_pass_list.push_back(std::move(pass));
- CompositorFrame frame;
- frame.delegated_frame_data = std::move(frame_data);
+ frame.render_pass_list.push_back(std::move(pass));
factory_.SubmitCompositorFrame(local_frame_id, std::move(frame),
SurfaceFactory::DrawCallback());
}
@@ -104,7 +101,7 @@ class SurfaceAggregatorPerfTest : public testing::Test {
timer_.Reset();
do {
std::unique_ptr<RenderPass> pass(RenderPass::Create());
- std::unique_ptr<DelegatedFrameData> frame_data(new DelegatedFrameData);
+ CompositorFrame frame;
SharedQuadState* sqs = pass->CreateAndAppendSharedQuadState();
SurfaceDrawQuad* surface_quad =
@@ -118,9 +115,7 @@ class SurfaceAggregatorPerfTest : public testing::Test {
else
pass->damage_rect = gfx::Rect(0, 0, 1, 1);
- frame_data->render_pass_list.push_back(std::move(pass));
- CompositorFrame frame;
- frame.delegated_frame_data = std::move(frame_data);
+ frame.render_pass_list.push_back(std::move(pass));
factory_.SubmitCompositorFrame(LocalFrameId(num_surfaces + 1, 0),
std::move(frame),
SurfaceFactory::DrawCallback());

Powered by Google App Engine
This is Rietveld 408576698