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

Unified Diff: cc/surfaces/display.h

Issue 2854163003: [cc] Plumb BeginFrameAcks through SurfaceManager to DisplayScheduler. (Closed)
Patch Set: track state in DisplayScheduler rather than Surface Created 3 years, 7 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/display.h
diff --git a/cc/surfaces/display.h b/cc/surfaces/display.h
index 314cb405ecc09be589ede5976d850fe8e4013f71..a8742587277a23f6c941af626f7ba2d915c0527e 100644
--- a/cc/surfaces/display.h
+++ b/cc/surfaces/display.h
@@ -40,7 +40,6 @@ class RendererSettings;
class ResourceProvider;
class SharedBitmapManager;
class SoftwareRenderer;
-class SurfaceAggregator;
class TextureMailboxDeleter;
// A Display produces a surface that can be used to draw to a physical display
@@ -78,6 +77,8 @@ class CC_SURFACES_EXPORT Display : public DisplaySchedulerClient,
// DisplaySchedulerClient implementation.
bool DrawAndSwap() override;
+ const SurfaceAggregator::SurfaceIndexMap& GetPreviousContainedSurfaces()
+ override;
// OutputSurfaceClient implementation.
void SetNeedsRedrawRect(const gfx::Rect& damage_rect) override;
@@ -88,6 +89,11 @@ class CC_SURFACES_EXPORT Display : public DisplaySchedulerClient,
// SurfaceObserver implementation.
void OnSurfaceDamaged(const SurfaceId& surface, bool* changed) override;
void OnSurfaceCreated(const SurfaceInfo& surface_info) override;
+ void OnSurfaceDestroyed(const SurfaceId& surface_id) override;
+ void OnSurfaceReceivedBeginFrame(const SurfaceId& surface_id,
+ const BeginFrameArgs& args) override;
+ void OnSurfaceFinishedBeginFrame(const SurfaceId& surface_id,
+ const BeginFrameAck& ack) override;
bool has_scheduler() const { return !!scheduler_; }
DirectRenderer* renderer_for_testing() const { return renderer_.get(); }

Powered by Google App Engine
This is Rietveld 408576698