Plumb damage rect to browser compositor.

content/browser/renderer_host/render_widget_host_view_aura.cc

 // Copyright (c) 2011 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/renderer_host/render_widget_host_view_aura.h"
 
 #include "base/logging.h"
 #include "content/browser/renderer_host/backing_store_skia.h"
 #include "content/browser/renderer_host/render_widget_host.h"
 #include "content/browser/renderer_host/web_input_event_aura.h"
@@ skipping 316 matching lines @@
   }
 #else
   NOTREACHED();
 #endif
 }
 
 void RenderWidgetHostViewAura::AcceleratedSurfacePostSubBuffer(
     const GpuHostMsg_AcceleratedSurfacePostSubBuffer_Params& params,
     int gpu_host_id) {
 #if defined(UI_COMPOSITOR_IMAGE_TRANSPORT)
-  window_->layer()->SetExternalTexture(
-      accelerated_surface_containers_[params.surface_id]->GetTexture());
+  scoped_refptr<AcceleratedSurfaceContainerLinux> container =
+      accelerated_surface_containers_[params.surface_id];
+    window_->layer()->SetExternalTexture(container->GetTexture());
   glFlush();
 
   if (!window_->layer()->GetCompositor()) {
     // We have no compositor, so we have no way to display the surface
     // Must still send the ACK
     RenderWidgetHost::AcknowledgePostSubBuffer(params.route_id, gpu_host_id);
   } else {
-    // TODO(backer): Plumb the damage rect to the ui compositor so that we
-    // can do a partial swap to display.
+#if defined(USE_WEBKIT_COMPOSITOR)
+    // Co-ords come in GL co-ord space. Need to convert to layer space.
+    window_->layer()->SchedulePaint(gfx::Rect(
+       params.x,
+       container->GetSize().height() - params.y - params.height,

[apatrick_chromium, 2011/12/06 01:10:30]

It looks like a lot of this function is reaching into window_->layer() and that
it could perhaps move there. I don't really know what window_->layer() is though
:)

[jonathan.backer, 2011/12/06 01:25:12]

I'm asking for the layer for a paint in it's co-ordinate system. It may be
possible to combine the two calls (SetExternalTexture and SchedulePaint), but
they have independent uses.

+       params.width,
+       params.height));
+#else
     window_->layer()->ScheduleDraw();
+#endif
 
     // Add sending an ACK to the list of things to do OnCompositingEnded
     on_compositing_ended_callbacks_.push_back(
         base::Bind(&RenderWidgetHost::AcknowledgePostSubBuffer,
                    params.route_id, gpu_host_id));
     ui::Compositor* compositor = window_->layer()->GetCompositor();
     if (!compositor->HasObserver(this))
       compositor->AddObserver(this);
   }
 #else
@@ skipping 253 matching lines @@
 // static
 void RenderWidgetHostView::GetDefaultScreenInfo(
     WebKit::WebScreenInfo* results) {
   const gfx::Size size = gfx::Screen::GetPrimaryMonitorSize();
   results->rect = WebKit::WebRect(0, 0, size.width(), size.height());
   results->availableRect = results->rect;
   // TODO(derat): Don't hardcode this?
   results->depth = 24;
   results->depthPerComponent = 8;
 }