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

Unified Diff: cc/surfaces/display.cc

Issue 2403373002: cc: Clean up OutputSurfaceClient. (Closed)
Patch Set: osclient: rebase Created 4 years, 2 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
« no previous file with comments | « cc/surfaces/display.h ('k') | cc/surfaces/display_scheduler.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/surfaces/display.cc
diff --git a/cc/surfaces/display.cc b/cc/surfaces/display.cc
index 1b2b36ac0ae763aa1ba9b0446f518e2389d08aa9..3550a39efda0071a17fe5a195c511c171bf4916a 100644
--- a/cc/surfaces/display.cc
+++ b/cc/surfaces/display.cc
@@ -330,7 +330,7 @@ bool Display::DrawAndSwap() {
frame.metadata.latency_info.end());
if (scheduler_) {
scheduler_->DidSwapBuffers();
- scheduler_->DidSwapBuffersComplete();
+ scheduler_->DidReceiveSwapBuffersAck();
}
}
@@ -338,9 +338,9 @@ bool Display::DrawAndSwap() {
return true;
}
-void Display::DidSwapBuffersComplete() {
+void Display::DidReceiveSwapBuffersAck() {
if (scheduler_)
- scheduler_->DidSwapBuffersComplete();
+ scheduler_->DidReceiveSwapBuffersAck();
if (renderer_)
renderer_->SwapBuffersComplete();
}
@@ -351,43 +351,12 @@ void Display::DidReceiveTextureInUseResponses(
renderer_->DidReceiveTextureInUseResponses(responses);
}
-void Display::SetBeginFrameSource(BeginFrameSource* source) {
- // The BeginFrameSource is set from the constructor, it doesn't come
- // from the OutputSurface for the Display.
- NOTREACHED();
-}
-
-void Display::SetMemoryPolicy(const ManagedMemoryPolicy& policy) {
- // This is only for LayerTreeHostImpl.
- NOTREACHED();
-}
-
-void Display::OnDraw(const gfx::Transform& transform,
- const gfx::Rect& viewport,
- bool resourceless_software_draw) {
- NOTREACHED();
-}
-
void Display::SetNeedsRedrawRect(const gfx::Rect& damage_rect) {
aggregator_->SetFullDamageForSurface(current_surface_id_);
if (scheduler_)
scheduler_->SurfaceDamaged(current_surface_id_);
}
-void Display::ReclaimResources(const ReturnedResourceArray& resources) {
- NOTREACHED();
-}
-
-void Display::SetExternalTilePriorityConstraints(
- const gfx::Rect& viewport_rect,
- const gfx::Transform& transform) {
- NOTREACHED();
-}
-
-void Display::SetTreeActivationCallback(const base::Closure& callback) {
- NOTREACHED();
-}
-
void Display::OnSurfaceDamaged(const SurfaceId& surface_id, bool* changed) {
if (aggregator_ &&
aggregator_->previous_contained_surfaces().count(surface_id)) {
« no previous file with comments | « cc/surfaces/display.h ('k') | cc/surfaces/display_scheduler.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698