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

Unified Diff: content/browser/renderer_host/delegated_frame_host.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
Index: content/browser/renderer_host/delegated_frame_host.cc
diff --git a/content/browser/renderer_host/delegated_frame_host.cc b/content/browser/renderer_host/delegated_frame_host.cc
index 268d45bd683a19b8e912a22c11363739bb656219..169842d84f817154c2bd9c415ddb9e02468891d2 100644
--- a/content/browser/renderer_host/delegated_frame_host.cc
+++ b/content/browser/renderer_host/delegated_frame_host.cc
@@ -44,7 +44,7 @@ void SatisfyCallback(cc::SurfaceManager* manager,
const cc::SurfaceSequence& sequence) {
std::vector<uint32_t> sequences;
sequences.push_back(sequence.sequence);
- manager->DidSatisfySequences(sequence.client_id, &sequences);
+ manager->DidSatisfySequences(sequence.frame_sink_id, &sequences);
}
void RequireCallback(cc::SurfaceManager* manager,
@@ -77,11 +77,11 @@ DelegatedFrameHost::DelegatedFrameHost(DelegatedFrameHostClient* client)
ImageTransportFactory* factory = ImageTransportFactory::GetInstance();
factory->GetContextFactory()->AddObserver(this);
id_allocator_.reset(new cc::SurfaceIdAllocator(
- factory->GetContextFactory()->AllocateSurfaceClientId()));
- factory->GetSurfaceManager()->RegisterSurfaceClientId(
- id_allocator_->client_id());
+ factory->GetContextFactory()->AllocateFrameSinkId()));
+ factory->GetSurfaceManager()->RegisterFrameSinkId(
+ id_allocator_->frame_sink_id());
factory->GetSurfaceManager()->RegisterSurfaceFactoryClient(
- id_allocator_->client_id(), this);
+ id_allocator_->frame_sink_id(), this);
}
void DelegatedFrameHost::WasShown(const ui::LatencyInfo& latency_info) {
@@ -207,8 +207,8 @@ void DelegatedFrameHost::EndFrameSubscription() {
frame_subscriber_.reset();
}
-uint32_t DelegatedFrameHost::GetSurfaceClientId() {
- return id_allocator_->client_id();
+cc::FrameSinkId DelegatedFrameHost::GetFrameSinkId() {
+ return id_allocator_->frame_sink_id();
}
cc::SurfaceId DelegatedFrameHost::SurfaceIdAtPoint(
@@ -831,9 +831,9 @@ DelegatedFrameHost::~DelegatedFrameHost() {
if (!surface_id_.is_null())
surface_factory_->Destroy(surface_id_);
factory->GetSurfaceManager()->UnregisterSurfaceFactoryClient(
- id_allocator_->client_id());
- factory->GetSurfaceManager()->InvalidateSurfaceClientId(
- id_allocator_->client_id());
+ id_allocator_->frame_sink_id());
+ factory->GetSurfaceManager()->InvalidateFrameSinkId(
+ id_allocator_->frame_sink_id());
DCHECK(!vsync_manager_.get());
}
@@ -848,7 +848,7 @@ void DelegatedFrameHost::SetCompositor(ui::Compositor* compositor) {
vsync_manager_ = compositor_->vsync_manager();
vsync_manager_->AddObserver(this);
- compositor_->AddSurfaceClient(id_allocator_->client_id());
+ compositor_->AddFrameSink(id_allocator_->frame_sink_id());
}
void DelegatedFrameHost::ResetCompositor() {
@@ -865,7 +865,7 @@ void DelegatedFrameHost::ResetCompositor() {
vsync_manager_ = nullptr;
}
- compositor_->RemoveSurfaceClient(id_allocator_->client_id());
+ compositor_->RemoveFrameSink(id_allocator_->frame_sink_id());
compositor_ = nullptr;
}

Powered by Google App Engine
This is Rietveld 408576698