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

Unified Diff: content/browser/renderer_host/offscreen_canvas_surface_impl.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: content/browser/renderer_host/offscreen_canvas_surface_impl.cc
diff --git a/content/browser/renderer_host/offscreen_canvas_surface_impl.cc b/content/browser/renderer_host/offscreen_canvas_surface_impl.cc
index 045fa0be3e259f534873e317b83ef3d1adad9767..4cae9fbc34516d0c72c86a51a6ac0d9443249b3b 100644
--- a/content/browser/renderer_host/offscreen_canvas_surface_impl.cc
+++ b/content/browser/renderer_host/offscreen_canvas_surface_impl.cc
@@ -57,20 +57,11 @@ void OffscreenCanvasSurfaceImpl::OnSurfaceCreated(
void OffscreenCanvasSurfaceImpl::Require(const cc::SurfaceId& surface_id,
const cc::SurfaceSequence& sequence) {
- cc::SurfaceManager* manager = GetSurfaceManager();
- cc::Surface* surface = manager->GetSurfaceForId(surface_id);
- if (!surface) {
- DLOG(ERROR) << "Attempting to require callback on nonexistent surface";
- return;
- }
- surface->AddDestructionDependency(sequence);
+ GetSurfaceManager()->RequireSequence(surface_id, sequence);
}
void OffscreenCanvasSurfaceImpl::Satisfy(const cc::SurfaceSequence& sequence) {
- std::vector<uint32_t> sequences;
- sequences.push_back(sequence.sequence);
- cc::SurfaceManager* manager = GetSurfaceManager();
- manager->DidSatisfySequences(sequence.frame_sink_id, &sequences);
+ GetSurfaceManager()->SatisfySequence(sequence);
}
} // namespace content
« cc/surfaces/surface_manager.cc ('K') | « content/browser/frame_host/cross_process_frame_connector.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698