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

Unified Diff: cc/surfaces/direct_compositor_frame_sink_unittest.cc

Issue 2383373002: Reduce SurfaceIdAllocator usage and tie SurfaceFactory to a single FrameSinkId (Closed)
Patch Set: Rebased 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
« no previous file with comments | « cc/surfaces/direct_compositor_frame_sink.cc ('k') | cc/surfaces/display_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/surfaces/direct_compositor_frame_sink_unittest.cc
diff --git a/cc/surfaces/direct_compositor_frame_sink_unittest.cc b/cc/surfaces/direct_compositor_frame_sink_unittest.cc
index 48aff5d7ad73434af670edeffde66a1e7197ee2b..4291539c1ecafb91c2dca9b9e986b16372196bbf 100644
--- a/cc/surfaces/direct_compositor_frame_sink_unittest.cc
+++ b/cc/surfaces/direct_compositor_frame_sink_unittest.cc
@@ -33,11 +33,10 @@ class DirectCompositorFrameSinkTest : public testing::Test {
DirectCompositorFrameSinkTest()
: now_src_(new base::SimpleTestTickClock()),
task_runner_(new OrderedSimpleTaskRunner(now_src_.get(), true)),
- allocator_(kArbitraryFrameSinkId),
display_size_(1920, 1080),
display_rect_(display_size_),
context_provider_(TestContextProvider::Create()) {
- surface_manager_.RegisterFrameSinkId(allocator_.frame_sink_id());
+ surface_manager_.RegisterFrameSinkId(kArbitraryFrameSinkId);
std::unique_ptr<FakeOutputSurface> display_output_surface =
FakeOutputSurface::Create3d();
@@ -57,8 +56,8 @@ class DirectCompositorFrameSinkTest : public testing::Test {
std::move(scheduler),
base::MakeUnique<TextureMailboxDeleter>(task_runner_.get())));
compositor_frame_sink_.reset(new DirectCompositorFrameSink(
- &surface_manager_, &allocator_, display_.get(), context_provider_,
- nullptr));
+ kArbitraryFrameSinkId, &surface_manager_, display_.get(),
+ context_provider_, nullptr));
compositor_frame_sink_->BindToClient(&compositor_frame_sink_client_);
display_->Resize(display_size_);
@@ -96,7 +95,6 @@ class DirectCompositorFrameSinkTest : public testing::Test {
protected:
std::unique_ptr<base::SimpleTestTickClock> now_src_;
scoped_refptr<OrderedSimpleTaskRunner> task_runner_;
- SurfaceIdAllocator allocator_;
const gfx::Size display_size_;
const gfx::Rect display_rect_;
« no previous file with comments | « cc/surfaces/direct_compositor_frame_sink.cc ('k') | cc/surfaces/display_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698