Plumb a Browser->GPU surface destruction message for Android VDAs

content/common/gpu_host_messages.h

 // Copyright 2016 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.
 
 // Multiply-included message file, hence no include guard here, but see below
 // for a much smaller-than-usual include guard section.
 
 #include "build/build_config.h"
 #include "content/common/content_export.h"
 #include "content/common/establish_channel_params.h"
@@ skipping 177 matching lines @@
 
 #if defined(OS_MACOSX)
 // Tells the GPU process that the browser process has handled the swap
 // buffers or post sub-buffer request.
 IPC_MESSAGE_CONTROL1(AcceleratedSurfaceMsg_BufferPresented,
                      content::BufferPresentedParams)
 #endif
 
 #if defined(OS_ANDROID)
 // Tells the GPU process to wake up the GPU because we're about to draw.
 IPC_MESSAGE_CONTROL0(GpuMsg_WakeUpGpu)

[no sievers, 2016/04/19 23:30:17]

nit: newline

[watk, 2016/04/22 02:48:55]

Done.

+// Tells the GPU process to release the surface because it's being destroyed.
+IPC_MESSAGE_CONTROL1(GpuMsg_DestroyingVideoSurface, int /* surface_id */)
 #endif
 
 // Tells the GPU process to remove all contexts.
 IPC_MESSAGE_CONTROL0(GpuMsg_Clean)
 
 // Tells the GPU process to crash.
 IPC_MESSAGE_CONTROL0(GpuMsg_Crash)
 
 // Tells the GPU process to hang.
 IPC_MESSAGE_CONTROL0(GpuMsg_Hang)
@@ skipping 52 matching lines @@
 IPC_MESSAGE_CONTROL1(GpuHostMsg_AcceleratedSurfaceBuffersSwapped,
                      content::AcceleratedSurfaceBuffersSwappedParams)
 #endif
 
 #if defined(OS_WIN)
 IPC_MESSAGE_CONTROL2(GpuHostMsg_AcceleratedSurfaceCreatedChildWindow,
                      gpu::SurfaceHandle /* parent_window */,
                      gpu::SurfaceHandle /* child_window */)
 #endif
 
+#if defined(OS_ANDROID)
+// Reponse to a GpuMsg_DestroyingVideoSurface message.
+IPC_MESSAGE_CONTROL1(GpuHostMsg_DestroyingVideoSurfaceHandled,

[no sievers, 2016/04/19 23:30:17]

nit: how about DestroyingVideoSurfaceAck? We use that naming in other places for
simple replies.

[watk, 2016/04/22 02:48:55]

Done.

+                     int /* surface_id */)
+#endif
+
+
 IPC_MESSAGE_CONTROL1(GpuHostMsg_DidCreateOffscreenContext, GURL /* url */)
 
 IPC_MESSAGE_CONTROL3(GpuHostMsg_DidLoseContext,
                      bool /* offscreen */,
                      gpu::error::ContextLostReason /* reason */,
                      GURL /* url */)
 
 IPC_MESSAGE_CONTROL1(GpuHostMsg_DidDestroyOffscreenContext, GURL /* url */)
 
 // Tells the browser about GPU memory usage statistics for UMA logging.
@@ skipping 10 matching lines @@
 // so the browser should stop sending the corresponding information
 IPC_MESSAGE_CONTROL2(GpuHostMsg_RemoveSubscription,
                      int32_t /* client_id */,
                      unsigned int /* target */)
 
 // Message from GPU to add a GPU log message to the about:gpu page.
 IPC_MESSAGE_CONTROL3(GpuHostMsg_OnLogMessage,
                      int /*severity*/,
                      std::string /* header */,
                      std::string /* message */)