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

Unified Diff: cc/surfaces/direct_surface_reference_factory.cc

Issue 2614423003: Adding SatisfySequence and RequireSequence to SurfaceManager (Closed)
Patch Set: up Created 3 years, 11 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/direct_surface_reference_factory.cc
diff --git a/cc/surfaces/direct_surface_reference_factory.cc b/cc/surfaces/direct_surface_reference_factory.cc
index 6165c973a4a93fa8bec81d75683b7d22b3cceeba..47d0d4b22b0fd60e53d66011eaca1053c90ea46e 100644
--- a/cc/surfaces/direct_surface_reference_factory.cc
+++ b/cc/surfaces/direct_surface_reference_factory.cc
@@ -19,20 +19,13 @@ void DirectSurfaceReferenceFactory::SatisfySequence(
const SurfaceSequence& sequence) const {
if (!manager_)
return;
- std::vector<uint32_t> sequences;
- sequences.push_back(sequence.sequence);
- manager_->DidSatisfySequences(sequence.frame_sink_id, &sequences);
+ manager_->SatisfySequence(sequence);
}
void DirectSurfaceReferenceFactory::RequireSequence(
const SurfaceId& surface_id,
const SurfaceSequence& sequence) const {
- auto* surface = manager_->GetSurfaceForId(surface_id);
- if (!surface) {
- LOG(ERROR) << "Attempting to require callback on nonexistent surface";
- return;
- }
- surface->AddDestructionDependency(sequence);
+ manager_->RequireSequence(surface_id, sequence);
}
} // namespace cc

Powered by Google App Engine
This is Rietveld 408576698