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

Unified Diff: cc/surfaces/compositor_frame_sink_support.cc

Issue 2493223002: Change exo::SurfaceFactoryOwner to exo::ExoCompositorFrameSink (Closed)
Patch Set: exo::Surface uses CompositorFrameSink accessor from CompositorFrameSinkHolder Created 4 years 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/compositor_frame_sink_support.cc
diff --git a/cc/surfaces/compositor_frame_sink_support.cc b/cc/surfaces/compositor_frame_sink_support.cc
index 011d9a091583d27ac07bb3598771baae74a3fc0a..1bcc64d7c9b73ba36296965c726ba22c99bef916 100644
--- a/cc/surfaces/compositor_frame_sink_support.cc
+++ b/cc/surfaces/compositor_frame_sink_support.cc
@@ -8,6 +8,7 @@
#include "cc/scheduler/begin_frame_source.h"
#include "cc/surfaces/compositor_frame_sink_support_client.h"
#include "cc/surfaces/display.h"
+#include "cc/surfaces/surface.h"
#include "cc/surfaces/surface_manager.h"
namespace cc {
@@ -78,6 +79,23 @@ void CompositorFrameSinkSupport::SubmitCompositorFrame(
}
}
+void CompositorFrameSinkSupport::Require(const LocalFrameId& local_frame_id,
+ const SurfaceSequence& sequence) {
+ Surface* surface = surface_manager_->GetSurfaceForId(
+ SurfaceId(frame_sink_id_, local_frame_id_));
+
+ // If this fails then that implies that a client is trying to reference a
+ // surface that no longer exists. This could result in flicker or missing
+ // components of UI.
+ DCHECK(surface);
+ surface->AddDestructionDependency(sequence);
+}
+
+void CompositorFrameSinkSupport::Satisfy(const SurfaceSequence& sequence) {
+ std::vector<uint32_t> sequences = {sequence.sequence};
+ surface_manager_->DidSatisfySequences(frame_sink_id_, &sequences);
+}
+
void CompositorFrameSinkSupport::DidReceiveCompositorFrameAck() {
DCHECK_GT(ack_pending_count_, 0);
ack_pending_count_--;

Powered by Google App Engine
This is Rietveld 408576698