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

Unified Diff: cc/test/test_compositor_frame_sink.cc

Issue 2369793002: WIP: Propagate SurfaceID up window tree hierarchy
Patch Set: Fix input events: EventDispatcher ignores container windows Created 4 years, 3 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/surface_sequence_generator.cc ('k') | cc/trees/layer_tree_host.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/test/test_compositor_frame_sink.cc
diff --git a/cc/test/test_compositor_frame_sink.cc b/cc/test/test_compositor_frame_sink.cc
index 35ba8e978350ef9d58009c65e2c87918b519082c..dc5ef6d48906da91633f6c2d6d587466c221d7cd 100644
--- a/cc/test/test_compositor_frame_sink.cc
+++ b/cc/test/test_compositor_frame_sink.cc
@@ -15,6 +15,7 @@
#include "cc/output/texture_mailbox_deleter.h"
static constexpr uint32_t kCompositorClientId = 1;
+static constexpr uint32_t kCompositorSinkId = 1;
namespace cc {
@@ -30,8 +31,9 @@ TestCompositorFrameSink::TestCompositorFrameSink(
bool force_disable_reclaim_resources)
: CompositorFrameSink(std::move(compositor_context_provider),
std::move(worker_context_provider)),
- surface_manager_(new SurfaceManager),
- surface_id_allocator_(new SurfaceIdAllocator(kCompositorClientId)),
+ surface_manager_(new SurfaceManager(nullptr)),
+ surface_id_allocator_(new SurfaceIdAllocator(
+ FrameSinkId(kCompositorClientId, kCompositorSinkId))),
surface_factory_(new SurfaceFactory(surface_manager_.get(), this)),
weak_ptrs_(this) {
std::unique_ptr<SyntheticBeginFrameSource> begin_frame_source;
@@ -88,11 +90,11 @@ bool TestCompositorFrameSink::BindToClient(CompositorFrameSinkClient* client) {
if (!capabilities_.delegated_sync_points_required && context_provider())
context_provider()->SetLostContextCallback(base::Closure());
- surface_manager_->RegisterSurfaceClientId(surface_id_allocator_->client_id());
+ surface_manager_->RegisterFrameSinkId(surface_id_allocator_->frame_sink_id());
surface_manager_->RegisterSurfaceFactoryClient(
- surface_id_allocator_->client_id(), this);
+ surface_id_allocator_->frame_sink_id(), this);
display_->Initialize(this, surface_manager_.get(),
- surface_id_allocator_->client_id());
+ surface_id_allocator_->frame_sink_id());
display_->renderer_for_testing()->SetEnlargePassTextureAmountForTesting(
enlarge_pass_texture_amount_);
display_->SetVisible(true);
@@ -106,9 +108,9 @@ void TestCompositorFrameSink::DetachFromClient() {
if (!delegated_surface_id_.is_null())
surface_factory_->Destroy(delegated_surface_id_);
surface_manager_->UnregisterSurfaceFactoryClient(
- surface_id_allocator_->client_id());
- surface_manager_->InvalidateSurfaceClientId(
- surface_id_allocator_->client_id());
+ surface_id_allocator_->frame_sink_id());
+ surface_manager_->InvalidateFrameSinkId(
+ surface_id_allocator_->frame_sink_id());
bound_ = false;
}
display_ = nullptr;
« no previous file with comments | « cc/surfaces/surface_sequence_generator.cc ('k') | cc/trees/layer_tree_host.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698