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

Side by Side Diff: cc/surfaces/surface_display_output_surface.cc

Issue 2147873003: cc: Dedup IPCs to return resources to client (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add bool is_swap_ack + remove CompositorFrameAck Created 4 years, 5 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "cc/surfaces/surface_display_output_surface.h" 5 #include "cc/surfaces/surface_display_output_surface.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "cc/output/compositor_frame.h" 8 #include "cc/output/compositor_frame.h"
9 #include "cc/output/compositor_frame_ack.h"
10 #include "cc/surfaces/display.h" 9 #include "cc/surfaces/display.h"
11 #include "cc/surfaces/surface.h" 10 #include "cc/surfaces/surface.h"
12 #include "cc/surfaces/surface_id_allocator.h" 11 #include "cc/surfaces/surface_id_allocator.h"
13 #include "cc/surfaces/surface_manager.h" 12 #include "cc/surfaces/surface_manager.h"
14 13
15 namespace cc { 14 namespace cc {
16 15
17 SurfaceDisplayOutputSurface::SurfaceDisplayOutputSurface( 16 SurfaceDisplayOutputSurface::SurfaceDisplayOutputSurface(
18 SurfaceManager* surface_manager, 17 SurfaceManager* surface_manager,
19 SurfaceIdAllocator* surface_id_allocator, 18 SurfaceIdAllocator* surface_id_allocator,
(...skipping 108 matching lines...) Expand 10 before | Expand all | Expand 10 after
128 } 127 }
129 128
130 uint32_t SurfaceDisplayOutputSurface::GetFramebufferCopyTextureFormat() { 129 uint32_t SurfaceDisplayOutputSurface::GetFramebufferCopyTextureFormat() {
131 // This is a delegating output surface, no framebuffer/direct drawing support. 130 // This is a delegating output surface, no framebuffer/direct drawing support.
132 NOTREACHED(); 131 NOTREACHED();
133 return 0; 132 return 0;
134 } 133 }
135 134
136 void SurfaceDisplayOutputSurface::ReturnResources( 135 void SurfaceDisplayOutputSurface::ReturnResources(
137 const ReturnedResourceArray& resources) { 136 const ReturnedResourceArray& resources) {
138 CompositorFrameAck ack;
139 ack.resources = resources;
140 if (client_) 137 if (client_)
141 client_->ReclaimResources(&ack); 138 client_->ReclaimResources(resources);
142 } 139 }
143 140
144 void SurfaceDisplayOutputSurface::SetBeginFrameSource( 141 void SurfaceDisplayOutputSurface::SetBeginFrameSource(
145 BeginFrameSource* begin_frame_source) { 142 BeginFrameSource* begin_frame_source) {
146 DCHECK(client_); 143 DCHECK(client_);
147 client_->SetBeginFrameSource(begin_frame_source); 144 client_->SetBeginFrameSource(begin_frame_source);
148 } 145 }
149 146
150 void SurfaceDisplayOutputSurface::DisplayOutputSurfaceLost() { 147 void SurfaceDisplayOutputSurface::DisplayOutputSurfaceLost() {
151 output_surface_lost_ = true; 148 output_surface_lost_ = true;
152 DidLoseOutputSurface(); 149 DidLoseOutputSurface();
153 } 150 }
154 151
155 void SurfaceDisplayOutputSurface::DisplaySetMemoryPolicy( 152 void SurfaceDisplayOutputSurface::DisplaySetMemoryPolicy(
156 const ManagedMemoryPolicy& policy) { 153 const ManagedMemoryPolicy& policy) {
157 SetMemoryPolicy(policy); 154 SetMemoryPolicy(policy);
158 } 155 }
159 156
160 void SurfaceDisplayOutputSurface::SwapBuffersComplete(SurfaceDrawStatus drawn) { 157 void SurfaceDisplayOutputSurface::SwapBuffersComplete(SurfaceDrawStatus drawn) {
161 // TODO(danakj): Why the lost check? 158 // TODO(danakj): Why the lost check?
162 if (!output_surface_lost_) 159 if (!output_surface_lost_)
163 client_->DidSwapBuffersComplete(); 160 client_->DidSwapBuffersComplete();
164 } 161 }
165 162
166 } // namespace cc 163 } // namespace cc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698