Partial swap implementation for surfaceless

content/browser/compositor/gpu_surfaceless_browser_compositor_output_surface.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/compositor/gpu_surfaceless_browser_compositor_output_surface.h"
 
 #include "cc/output/compositor_frame.h"
 #include "content/browser/compositor/buffer_queue.h"
 #include "content/browser/compositor/reflector_impl.h"
 #include "content/browser/gpu/gpu_surface_tracker.h"
@@ skipping 20 matching lines @@
 }
 
 GpuSurfacelessBrowserCompositorOutputSurface::
     ~GpuSurfacelessBrowserCompositorOutputSurface() {
 }
 
 void GpuSurfacelessBrowserCompositorOutputSurface::SwapBuffers(
     cc::CompositorFrame* frame) {
   DCHECK(output_surface_);
 
+  output_surface_->SwapBuffers(frame->gl_frame_data->sub_buffer_rect);
+  // Disable the sub-buffer processing path on the parent classes, because we
+  // have handled it here.
+  frame->gl_frame_data->sub_buffer_rect = gfx::Rect(frame->gl_frame_data->size);

[alexst (slow to review), 2014/09/18 15:39:15]

We have a reflector, which is used for multimon mirroring that copies these as
well. It needs the damage rect too.

I think we can remove this here and implement virtual PostSubBuffer(..) in the
GPU process surface to ignore the rect.

[achaulk, 2014/09/18 16:45:02]

Done.

+
   const gfx::Size& size = frame->gl_frame_data->size;
   context_provider_->ContextGL()->ScheduleOverlayPlaneCHROMIUM(
       0,
       GL_OVERLAY_TRANSFORM_NONE_CHROMIUM,
       output_surface_->current_texture_id(),
       0,
       0,
       size.width(),
       size.height(),
       0,
       0,
       1.0f,
       1.0f);
-  output_surface_->SwapBuffers();
   GpuBrowserCompositorOutputSurface::SwapBuffers(frame);
 }
 
 void GpuSurfacelessBrowserCompositorOutputSurface::OnSwapBuffersComplete() {
   DCHECK(output_surface_);
   output_surface_->PageFlipComplete();
   GpuBrowserCompositorOutputSurface::OnSwapBuffersComplete();
 }
 
 void GpuSurfacelessBrowserCompositorOutputSurface::BindFramebuffer() {
@@ skipping 11 matching lines @@
 
 bool GpuSurfacelessBrowserCompositorOutputSurface::BindToClient(
     cc::OutputSurfaceClient* client) {
   if (!GpuBrowserCompositorOutputSurface::BindToClient(client))
     return false;
   output_surface_.reset(new BufferQueue(context_provider_, internalformat_));
   return output_surface_->Initialize();
 }
 
 }  // namespace content