Aura: Resize locks with --ui-enable-threaded-compositing

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/base_switches.h"
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/command_line.h"
@@ skipping 104 matching lines @@
         base::Bind(&AcceleratedSurfaceBuffersSwappedCompletedForGPU,
                    host_id,
                    route_id,
                    alive));
     return;
   }
 
   GpuProcessHost* host = GpuProcessHost::FromID(host_id);
   if (host) {
     if (alive)
-      host->Send(new AcceleratedSurfaceMsg_BufferPresented(route_id, 0));
+      host->Send(new AcceleratedSurfaceMsg_BufferPresented(route_id, false, 0));

[piman, 2012/10/25 00:14:08]

Should it really be false here? I think this path is taken for both success and
failure, though admittedly probably only on non-aura builds (I haven't checked
all the call chains), where it doesn't matter.
Maybe we need to pipe it through?

[jonathan.backer, 2012/10/25 15:19:04]

Piping it through. I'll send presented = false on error and presented = true on
success.

     else
       host->ForceShutdown();
   }
 }
 
 #if defined(OS_WIN)
 // This sends a ViewMsg_SwapBuffers_ACK directly to the renderer process
 // (RenderWidget). This path is currently not used with the threaded compositor.
 void AcceleratedSurfaceBuffersSwappedCompletedForRenderer(
     int surface_id,
@@ skipping 818 matching lines @@
 
 void GpuProcessHost::CreateCommandBufferError(
     const CreateCommandBufferCallback& callback, int32 route_id) {
   callback.Run(route_id);
 }
 
 void GpuProcessHost::CreateImageError(
     const CreateImageCallback& callback, const gfx::Size size) {
   callback.Run(size);
 }