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

content/browser/media/android/browser_surface_view_manager.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.
 
 #ifndef CONTENT_BROWSER_MEDIA_ANDROID_BROWSER_SURFACE_VIEW_MANAGER_H_
 #define CONTENT_BROWSER_MEDIA_ANDROID_BROWSER_SURFACE_VIEW_MANAGER_H_
 
 #include <stdint.h>
 
 #include <memory>
@@ skipping 20 matching lines @@
   void SetVideoSurface(gfx::ScopedJavaSurface surface) override;
   void DidExitFullscreen(bool release_media_player) override;
 
   void OnCreateFullscreenSurface(const gfx::Size& video_natural_size);
   void OnNaturalSizeChanged(const gfx::Size& size);
 
  private:
   // Send a message to return the surface id to the caller.
   bool SendSurfaceID(int surface_id);
 
+  // Synchronously notify the decoder that the surface is being destroyed so it
+  // can stop rendering to it. This sends a message to the GPU process. Without
+  // this, the MediaCodec decoder will start throwing IllegalStateException, and
+  // crash on some devices (http://crbug.com/598408). There is no way for us to
+  // make sure the surface outlives the decoder because Android initiates the
+  // destruction in some cases.
+  void SendDestroyingVideoSurfaceIfRequired(int surface_id);
+
   RenderFrameHost* const render_frame_host_;
 
   // The surface id of the ContentVideoView surface.
   int surface_id_;
 
   // The fullscreen view that contains a SurfaceView.
   std::unique_ptr<ContentVideoView> content_video_view_;
 
   DISALLOW_COPY_AND_ASSIGN(BrowserSurfaceViewManager);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_MEDIA_ANDROID_BROWSER_SURFACE_VIEW_MANAGER_H_