Mac ÜC: Make resize smooth

content/browser/gpu/gpu_process_host.cc

 // Copyright (c) 2012 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/gpu/gpu_process_host.h"
 
 #include "base/base64.h"
 #include "base/base_switches.h"
 #include "base/basictypes.h"
 #include "base/bind.h"
@@ skipping 834 matching lines @@
 void GpuProcessHost::OnAcceleratedSurfaceBuffersSwapped(
     const GpuHostMsg_AcceleratedSurfaceBuffersSwapped_Params& params) {
   TRACE_EVENT0("gpu", "GpuProcessHost::OnAcceleratedSurfaceBuffersSwapped");
 
   if (!ui::LatencyInfo::Verify(params.latency_info,
                                "GpuHostMsg_AcceleratedSurfaceBuffersSwapped"))
     return;
 
   gfx::AcceleratedWidget native_widget =
       GpuSurfaceTracker::Get()->AcquireNativeWidget(params.surface_id);
+
+  // Retrieve the render widget helper to forward this message to on the UI
+  // thread.
+  RenderWidgetHelper* helper = NULL;
+  int render_widget_id = 0;
   if (native_widget) {
-    RenderWidgetHelper::OnNativeSurfaceBuffersSwappedOnIOThread(this, params);
-    return;
+    int render_process_id = 0;
+    // TODO(ccameron): The render process and widget IDs have a race between
+    // the UI and the IO threads, and may potentially be wrong. Move the
+    // functionality in RenderProcessHost::WaitForBackingStoreMsg to a place
+    // where it can accept IPCs from all renderers (rather than this hack where
+    // we fake the render process and widget ID of a renderer for the browser).
+    // http://crbug.com/392031
+    if (RenderWidgetHelper::GetRenderWidgetIDForWidget(
+            native_widget, &render_process_id, &render_widget_id)) {
+      helper = RenderWidgetHelper::FromProcessHostID(render_process_id);
+    }
+  } else {
+    int render_process_id = 0;
+    if (GpuSurfaceTracker::Get()->GetRenderWidgetIDForSurface(
+        params.surface_id, &render_process_id, &render_widget_id)) {
+      helper = RenderWidgetHelper::FromProcessHostID(render_process_id);
+    }
   }
 
-  gfx::GLSurfaceHandle surface_handle =
-      GpuSurfaceTracker::Get()->GetSurfaceHandle(params.surface_id);
-  // Compositor window is always gfx::kNullPluginWindow.
-  // TODO(jbates) http://crbug.com/105344 This will be removed when there are no
-  // plugin windows.
-  if (surface_handle.handle != gfx::kNullPluginWindow ||
-      surface_handle.transport_type == gfx::TEXTURE_TRANSPORT) {
-    RouteOnUIThread(GpuHostMsg_AcceleratedSurfaceBuffersSwapped(params));
-    return;
-  }
-
-  AcceleratedSurfaceMsg_BufferPresented_Params ack_params;
-  ack_params.sync_point = 0;
-
-  int render_process_id = 0;
-  int render_widget_id = 0;
-  if (!GpuSurfaceTracker::Get()->GetRenderWidgetIDForSurface(
-      params.surface_id, &render_process_id, &render_widget_id)) {
-    Send(new AcceleratedSurfaceMsg_BufferPresented(params.route_id,
-                                                   ack_params));
-    return;
-  }
-  RenderWidgetHelper* helper =
-      RenderWidgetHelper::FromProcessHostID(render_process_id);
-  if (!helper) {
+  // If this is not targetting a native widget and we don't find a helper, then
+  // ack immediately and ignore the message.
+  if (!helper && !native_widget) {
+    AcceleratedSurfaceMsg_BufferPresented_Params ack_params;
+    ack_params.sync_point = 0;
     Send(new AcceleratedSurfaceMsg_BufferPresented(params.route_id,
                                                    ack_params));
     return;
   }
 
+  // If this is targetting a native widget, then ack the swap immediately to
+  // avoid introducing a deadlock between the browser and GPU processes (rate
+  // limiting is done through the compositor).
+  if (native_widget) {
+    AcceleratedSurfaceMsg_BufferPresented_Params ack_params;
+    ack_params.sync_point = 0;
+    Send(new AcceleratedSurfaceMsg_BufferPresented(params.route_id,
+                                                   ack_params));
+  }
+
   // Pass the SwapBuffers on to the RenderWidgetHelper to wake up the UI thread
   // if the browser is waiting for a new frame. Otherwise the RenderWidgetHelper
   // will forward to the RenderWidgetHostView via RenderProcessHostImpl and
   // RenderWidgetHostImpl.
   ViewHostMsg_CompositorSurfaceBuffersSwapped_Params view_params;
+  view_params.use_native_widget = native_widget;
   view_params.surface_id = params.surface_id;
   view_params.surface_handle = params.surface_handle;
   view_params.route_id = params.route_id;
   view_params.size = params.size;
   view_params.scale_factor = params.scale_factor;
   view_params.gpu_process_host_id = host_id_;
   view_params.latency_info = params.latency_info;
-  helper->DidReceiveBackingStoreMsg(ViewHostMsg_CompositorSurfaceBuffersSwapped(
-      render_widget_id,
-      view_params));
+  if (helper) {
+    helper->DidReceiveBackingStoreMsg(
+        ViewHostMsg_CompositorSurfaceBuffersSwapped(
+            render_widget_id,
+            view_params));
+  } else if (native_widget) {
+    BrowserThread::PostTask(
+        BrowserThread::UI,
+        FROM_HERE,
+        base::Bind(&RenderWidgetHelper::OnNativeSurfaceBuffersSwappedOnUIThread,
+                       view_params));
+  }
 }
 #endif  // OS_MACOSX
 
 void GpuProcessHost::OnProcessLaunched() {
   UMA_HISTOGRAM_TIMES("GPU.GPUProcessLaunchTime",
                       base::TimeTicks::Now() - init_start_time_);
 }
 
 void GpuProcessHost::OnProcessCrashed(int exit_code) {
   SendOutstandingReplies();
@@ skipping 204 matching lines @@
   TRACE_EVENT0("gpu", "GpuProcessHost::OnCacheShader");
   ClientIdToShaderCacheMap::iterator iter =
       client_id_to_shader_cache_.find(client_id);
   // If the cache doesn't exist then this is an off the record profile.
   if (iter == client_id_to_shader_cache_.end())
     return;
   iter->second->Cache(GetShaderPrefixKey() + ":" + key, shader);
 }
 
 }  // namespace content