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

Unified Diff: content/renderer/android/synchronous_compositor_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 side-by-side diff with in-line comments
Download patch
Index: content/renderer/android/synchronous_compositor_output_surface.cc
diff --git a/content/renderer/android/synchronous_compositor_output_surface.cc b/content/renderer/android/synchronous_compositor_output_surface.cc
index 2cbcf4dc70fa57683c82be939eccf0c795c5ba71..63bbbd92de605375c0ab1086176e107741e364cb 100644
--- a/content/renderer/android/synchronous_compositor_output_surface.cc
+++ b/content/renderer/android/synchronous_compositor_output_surface.cc
@@ -338,12 +338,12 @@ void SynchronousCompositorOutputSurface::InvokeComposite(
void SynchronousCompositorOutputSurface::OnReclaimResources(
uint32_t output_surface_id,
- const cc::CompositorFrameAck& ack) {
+ const cc::ReturnedResourceArray& resources) {
// Ignore message if it's a stale one coming from a different output surface
// (e.g. after a lost context).
if (output_surface_id != output_surface_id_)
return;
- ReclaimResources(&ack);
+ ReclaimResources(resources);
}
void SynchronousCompositorOutputSurface::SetMemoryPolicy(size_t bytes_limit) {
@@ -397,8 +397,7 @@ bool SynchronousCompositorOutputSurface::CalledOnValidThread() const {
void SynchronousCompositorOutputSurface::ReturnResources(
const cc::ReturnedResourceArray& resources) {
DCHECK(resources.empty());
- cc::CompositorFrameAck ack;
- client_->ReclaimResources(&ack);
+ client_->ReclaimResources(resources);
}
void SynchronousCompositorOutputSurface::SetBeginFrameSource(

Powered by Google App Engine
This is Rietveld 408576698