From e272282175ff952685c32820cc8e018907e543b8 Mon Sep 17 00:00:00 2001 From: "qinmin@chromium.org" Date: Wed, 26 Mar 2014 08:00:46 +0000 Subject: Try catch a top crasher for MSE/EME ReleaseOutputBuffer() often throws exceptions. However, most of the times those exceptions doesn't affect playback on my nexus 4. Just catch the exception for now. Review URL: https://codereview.chromium.org/212013002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@259529 0039d316-1c4b-4281-b951-d872f2087c98 --- .../base/android/java/src/org/chromium/media/MediaCodecBridge.java | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) (limited to 'media/base') diff --git a/media/base/android/java/src/org/chromium/media/MediaCodecBridge.java b/media/base/android/java/src/org/chromium/media/MediaCodecBridge.java index 29181ff..3f15061 100644 --- a/media/base/android/java/src/org/chromium/media/MediaCodecBridge.java +++ b/media/base/android/java/src/org/chromium/media/MediaCodecBridge.java @@ -398,7 +398,12 @@ class MediaCodecBridge { @CalledByNative private void releaseOutputBuffer(int index, boolean render) { - mMediaCodec.releaseOutputBuffer(index, render); + try { + mMediaCodec.releaseOutputBuffer(index, render); + } catch(IllegalStateException e) { + // TODO(qinmin): May need to report the error to the caller. crbug.com/356498. + Log.e(TAG, "Failed to release output buffer", e); + } } @CalledByNative -- cgit v1.1