“surfaceCreated”里面的“surface has been released”

Question

我知道这是一个常见问题，但是此堆栈跟踪显示其他问题。您可以看到，即使在 surfaceCreated 内部调用了 setDisplay(holder)，它仍然会抛出 IllegalArgumentException。这也不是罕见的例外，昨天在约 3,000,000 次剪辑视图中发生了约 125,000 次。我可以向您保证，mCurrentPlayer 也已正确初始化。

表面创建：

@Override
public void surfaceCreated(SurfaceHolder holder) {
    mIsSurfaceCreated = true;
    mCurrentPlayer.setDisplay(holder);
}

表面销毁：

@Override
public void surfaceDestroyed(SurfaceHolder holder) {
    mIsSurfaceCreated = false;

    // Could be called after player was released in onDestroy.
    if (mCurrentPlayer != null) {
        mCurrentPlayer.setDisplay(null);
    }
}

堆栈跟踪：

java.lang.IllegalArgumentException: The surface has been released
    at android.media.MediaPlayer._setVideoSurface(Native Method)
    at android.media.MediaPlayer.setDisplay(MediaPlayer.java:660)
    at com.xxx.xxx.view.VideoPlayerView.surfaceCreated(VideoPlayerView.java:464)
    at android.view.SurfaceView.updateWindow(SurfaceView.java:543)
    at android.view.SurfaceView.access$000(SurfaceView.java:81)
    at android.view.SurfaceView$3.onPreDraw(SurfaceView.java:169)
    at android.view.ViewTreeObserver.dispatchOnPreDraw(ViewTreeObserver.java:590)
    at android.view.ViewRootImpl.performTraversals(ViewRootImpl.java:1644)
    at android.view.ViewRootImpl.handleMessage(ViewRootImpl.java:2505)
    at android.os.Handler.dispatchMessage(Handler.java:99)
    at android.os.Looper.loop(Looper.java:154)
    at android.app.ActivityThread.main(ActivityThread.java:4945)
    at java.lang.reflect.Method.invokeNative(Native Method)
    at java.lang.reflect.Method.invoke(Method.java:511)
    at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:784)
    at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:551)
    at dalvik.system.NativeStart.main(Native Method)

关于其他可能出现问题的任何想法？ SurfaceHolder 是否可能会破坏后台线程上的表面，然后等待主线程（当前由 surfaceCreated 占用）完成它的块，然后才能在主线程上调用 surfaceDestroyed（我什至不认为锁可以修）？还有什么？

更新 -- 在深入挖掘之后，我发现了导致“表面已被释放”为 thrown 的原因：

哪些引用android_view_Surface_getSurface可以找到here：

这是我缺乏 C++ 知识的痛处，它看起来像是试图锁定表面，如果不能，返回的表面将是 null。一旦返回为null，IllegalArgumentException 将被抛出。

Answer 1

我刚刚解决了一个类似的问题。

我的调查显示，SurfaceView 中存在一个错误，导致将无效的表面传递给 surfaceCreated 回调方法。

这是修复它的 android repo 中的提交：link。

似乎在 4.2 版本中引入了 android 源中的修复。而且，从应用程序的崩溃中，我看到无效表面导致的崩溃仅发生在 4.0 和 4.1 上。

所以，我可以假设在 4.0 之前，将无效表面传递给 MediaPlayer 是有效的。并且在 4.0 中 SurfaceView/MediaPlayer 的逻辑发生了变化，导致它不再有效。但是 SurfaceView 中的代码在 4.2 之前没有更新（其中修复了 SurfaceView 中的这个问题）。

我已经检查了 android 的 git repo，实际上，标记为 android-4.0.1_r1 的版本不包含修复，而标记为 android-4.2.1_r1 的版本包含它。

因此，要为不包含修复的平台修复它，仅在平台 4.0 及更高版本需要将其设置为 MediaPlayer 之前手动检查表面是否有效：

@Override public void surfaceCreated(final SurfaceHolder holder) {
    final Surface surface = holder.getSurface();

    if ( surface == null ) return;

    // on pre Ice Scream Sandwich (4.0) versions invalid surfaces seems to be accepted (or at least do not cause crash)
    final boolean invalidSurfaceAccepted = Build.VERSION.SDK_INT < Build.ICE_CREAM_SANDWICH;
    final boolean invalidSurface = ! surface.isValid();

    if ( invalidSurface && ( ! invalidSurfaceAccepted ) ) return;

    _mediaPlayer.setDisplay(holder);
}

这样，在旧平台上，无效表面将成功设置为媒体播放器并播放视频，在 4.0-4.1 平台上它将丢弃无效表面（我认为将使用有效表面再次调用 surfaceCreated）并且在 4.2 和更高版本上，surfaceCreated 将不会被无效的表面调用。

Answer 2

过去我在使用 Android VideoViews/MediaPlayers 时遇到过类似的问题。事实证明，底层的 SurfaceView 正在收集垃圾。我通过向 MediaPlayer 添加一个 onPreparedLister 来解决它，然后在我使用它时在我的类中持有对它的显式引用。也许这会有所帮助。