Make GPU back-pressure work with remote CALayers

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 25 matching lines @@
 #include "content/public/common/result_codes.h"
 #include "content/public/common/sandboxed_process_launcher_delegate.h"
 #include "gpu/command_buffer/service/gpu_switches.h"
 #include "ipc/ipc_channel_handle.h"
 #include "ipc/ipc_switches.h"
 #include "ipc/message_filter.h"
 #include "media/base/media_switches.h"
 #include "ui/events/latency_info.h"
 #include "ui/gl/gl_switches.h"
 
-#if defined(OS_MACOSX)
-#include <IOSurface/IOSurfaceAPI.h>
-#include "base/mac/scoped_cftyperef.h"
-#include "content/common/gpu/surface_handle_types_mac.h"
-#endif
-
 #if defined(OS_WIN)
 #include "base/win/windows_version.h"
 #include "content/common/sandbox_win.h"
 #include "sandbox/win/src/sandbox_policy.h"
 #include "ui/gfx/switches.h"
 #endif
 
 #if defined(USE_OZONE)
 #include "ui/ozone/public/ozone_switches.h"
 #endif
@@ skipping 689 matching lines @@
 }
 
 void GpuProcessHost::OnGpuMemoryUmaStatsReceived(
     const GPUMemoryUmaStats& stats) {
   TRACE_EVENT0("gpu", "GpuProcessHost::OnGpuMemoryUmaStatsReceived");
   uma_memory_stats_received_ = true;
   uma_memory_stats_ = stats;
 }
 
 #if defined(OS_MACOSX)
-namespace {
-void HoldIOSurfaceReference(base::ScopedCFTypeRef<IOSurfaceRef> io_surface) {}
-}  // namespace
-
 void GpuProcessHost::OnAcceleratedSurfaceBuffersSwapped(
     const IPC::Message& message) {
   RenderWidgetResizeHelper::Get()->PostGpuProcessMsg(host_id_, message);
-
-  if (!IsDelegatedRendererEnabled())
-    return;
-
-  GpuHostMsg_AcceleratedSurfaceBuffersSwapped::Param param;
-  if (!GpuHostMsg_AcceleratedSurfaceBuffersSwapped::Read(&message, &param))
-    return;
-  const GpuHostMsg_AcceleratedSurfaceBuffersSwapped_Params& params = param.a;
-
-  if (GetSurfaceHandleType(params.surface_handle) ==
-          kSurfaceHandleTypeIOSurface) {
-    // As soon as the frame is acked, the IOSurface may be thrown away by the
-    // GPU process. Open the IOSurface and post a task referencing it to the UI
-    // thread. This will keep the IOSurface from being thrown away until the UI
-    // thread can open another reference to it, if needed.
-    base::ScopedCFTypeRef<IOSurfaceRef> io_surface(IOSurfaceLookup(
-        IOSurfaceIDFromSurfaceHandle(params.surface_handle)));
-    BrowserThread::PostTask(BrowserThread::UI,
-                            FROM_HERE,
-                            base::Bind(HoldIOSurfaceReference, io_surface));
-  }
-
-  // If delegated rendering is enabled, then immediately acknowledge this frame
-  // on the IO thread instead of the UI thread. The UI thread will wait on the
-  // GPU process. If the UI thread were to be responsible for acking swaps,
-  // then there would be a cycle and a potential deadlock. Back-pressure from
-  // the GPU is provided through the compositor's output surface.
-  AcceleratedSurfaceMsg_BufferPresented_Params ack_params;
-  ack_params.sync_point = 0;
-  ack_params.renderer_id = 0;
-  Send(new AcceleratedSurfaceMsg_BufferPresented(params.route_id, ack_params));
 }
 #endif
 
 void GpuProcessHost::OnProcessLaunched() {
   UMA_HISTOGRAM_TIMES("GPU.GPUProcessLaunchTime",
                       base::TimeTicks::Now() - init_start_time_);
 }
 
 void GpuProcessHost::OnProcessCrashed(int exit_code) {
   SendOutstandingReplies();
@@ skipping 271 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