Drop idle MediaCodec instances after a few seconds within AVDA.

media/gpu/android_video_decode_accelerator.h

 // Copyright (c) 2013 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 MEDIA_GPU_ANDROID_VIDEO_DECODE_ACCELERATOR_H_
 #define MEDIA_GPU_ANDROID_VIDEO_DECODE_ACCELERATOR_H_
 
 #include <stdint.h>
 
 #include <list>
@@ skipping 353 matching lines @@
 
   // Returns true if frame's COPY_REQUIRED flag needs to be set when using
   // deferred strategy.
   static bool UseTextureCopyForDeferredStrategy(
       const gpu::GpuPreferences& gpu_preferences);
 
   // Indicates if MediaCodec should not be used for software decoding since we
   // have safer versions elsewhere.
   bool IsMediaCodecSoftwareDecodingForbidden() const;
 
+  // Closes idle MediaCodec instances if no Decode() call is received within a
+  // few seconds.
+  void CloseIdleCodec();
+
   // Used to DCHECK that we are called on the correct thread.
   base::ThreadChecker thread_checker_;
 
   // To expose client callbacks from VideoDecodeAccelerator.
   Client* client_;
 
   // Callback to set the correct gl context.
   MakeGLContextCurrentCallback make_context_current_cb_;
 
   // Callback to get the GLES2Decoder instance.
@@ skipping 93 matching lines @@
   // Indicates if ResetCodecState() should be called upon the next call to
   // Decode(). Allows us to avoid trashing the last few frames of a playback
   // when the EOS buffer is received.
   bool codec_needs_reset_;
 
   // Copy of the VDA::Config we were given.
   Config config_;
 
   OnDestroyingSurfaceCallback on_destroying_surface_cb_;
 
+  // Timer used to destroy the MediaCodec if no Decode() call is received after
+  // a seconds of idle activity. This prevents older devices from falling over

[liberato (no reviews please), 2016/07/13 05:25:55]

"a seconds"

+  // when too many MediaCodec instances are outstanding.
+  base::OneShotTimer no_decode_timeout_;
+
   // WeakPtrFactory for posting tasks back to |this|.
   base::WeakPtrFactory<AndroidVideoDecodeAccelerator> weak_this_factory_;
 
   friend class AndroidVideoDecodeAcceleratorTest;
 };
 
 }  // namespace media
 
 #endif  // MEDIA_GPU_ANDROID_VIDEO_DECODE_ACCELERATOR_H_