Partial swaps on OSX

content/browser/renderer_host/render_widget_host_view_mac.mm

 // 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_mac.h"
 
 #include <QuartzCore/QuartzCore.h>
 
 #include "base/bind.h"
 #include "base/debug/trace_event.h"
@@ skipping 875 matching lines @@
   }
 
   if (params.route_id != 0) {
     RenderWidgetHost::AcknowledgeSwapBuffers(params.route_id, gpu_host_id);
   }
 }
 
 void RenderWidgetHostViewMac::AcceleratedSurfacePostSubBuffer(
     const GpuHostMsg_AcceleratedSurfacePostSubBuffer_Params& params,
     int gpu_host_id) {
-  NOTIMPLEMENTED();
+  TRACE_EVENT0("browser",
+      "RenderWidgetHostViewMac::AcceleratedSurfacePostSubBuffer");
+  CHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
+  AcceleratedPluginView* view = ViewForPluginWindowHandle(params.window);
+j  DCHECK(view);

[apatrick_chromium, 2011/12/05 22:42:45]

j at beginning of line

[jonathan.backer, 2011/12/06 18:29:31]

<whine> I want focus follows mouse on OSX </whine>

Done.

+  if (view) {
+    last_frame_was_accelerated_ = (params.window ==
+        plugin_container_manager_.root_container_handle());
+    plugin_container_manager_.SetSurfaceWasPaintedTo(params.window,
+                                                     params.surface_id,
+                                                     params.x,
+                                                     params.y,
+                                                     params.width,
+                                                     params.height);
+
+    // The surface is hidden until its first paint, to not show gargabe.

[apatrick_chromium, 2011/12/05 22:42:45]

gargabe -> garbage

[jonathan.backer, 2011/12/06 18:29:31]

Done.

+    if (plugin_container_manager_.SurfaceShouldBeVisible(params.window))
+      [view setHidden:NO];
+    [view drawView];
+  }
+
+  if (params.route_id != 0) {
+    RenderWidgetHost::AcknowledgePostSubBuffer(params.route_id, gpu_host_id);
+  }
 }
 
 void RenderWidgetHostViewMac::UpdateRootGpuViewVisibility(
     bool show_gpu_widget) {
   // Plugins are destroyed on page navigate. The compositor layer on the other
   // hand is created on demand and then stays alive until its renderer process
   // dies (usually on cross-domain navigation). Instead, only a flag
   // |is_accelerated_compositing_active()| is flipped when the compositor output
   // should be shown/hidden.
   // Show/hide the view belonging to the compositor here.
@@ skipping 1801 matching lines @@
   if (!string) return NO;
 
   // If the user is currently using an IME, confirm the IME input,
   // and then insert the text from the service, the same as TextEdit and Safari.
   [self confirmComposition];
   [self insertText:string];
   return YES;
 }
 
 @end