Question

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

surfaceCreated：

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

surfaceDestroy：

@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来解决它，然后在我使用它时在我的类中保持对它的显式引用。也许这有帮助。

“表面已被释放”内部“surfaceCreated”

2 个答案: