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 346 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.

[sky, 2011/12/08 17:39:05]

I've never seen coordinates abbreviated like that.

[jonathan.backer, 2011/12/08 18:41:44]

Done.

+    window_->layer()->SchedulePaint(gfx::Rect(

[sky, 2011/12/08 17:39:05]

nit: window_->SchedulePaint should work equally well here.

[jonathan.backer, 2011/12/08 18:41:44]

Done.

+       params.x,
+       container->GetSize().height() - params.y - params.height,
+       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 446 matching lines @@
   results->availableRect = results->rect;
   // TODO(derat): Don't hardcode this?
   results->depth = 24;
   results->depthPerComponent = 8;
 }
 
 ui::InputMethod* RenderWidgetHostViewAura::GetInputMethod() {
   aura::Desktop* desktop = aura::Desktop::GetInstance();
   return desktop->GetInputMethod();
 }