Merge pull request #10 from m-kubis/mediaplayer_prepare_failure_handling

Mediaplayer prepare failure handling
mklimek · Jun 7, 2016 · f769958 · f769958
2 parents 316b4e3 + 7429ef6
commit f769958
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 5 deletions.
diff --git a/build.gradle b/build.gradle
@@ -3,7 +3,7 @@ buildscript {
         jcenter()
     }
     dependencies {
-        classpath 'com.android.tools.build:gradle:2.1.0-alpha4'
+        classpath 'com.android.tools.build:gradle:2.1.2'
         classpath 'com.github.dcendents:android-maven-gradle-plugin:1.3'
     }
 }

diff --git a/library/build.gradle b/library/build.gradle
@@ -1,11 +1,11 @@
 apply plugin: 'com.android.library'
-apply plugin: 'com.github.dcendents.android-maven'  
+apply plugin: 'com.github.dcendents.android-maven'
 
 group='com.github.mklimek'
 
 android {
     compileSdkVersion 23
-    buildToolsVersion "23.0.2"
+    buildToolsVersion "23.0.3"
 
     defaultConfig {
         minSdkVersion 10

diff --git a/library/src/main/java/com/mklimek/frameviedoview/FrameVideoViewListener.java b/library/src/main/java/com/mklimek/frameviedoview/FrameVideoViewListener.java
@@ -3,5 +3,7 @@
 import android.media.MediaPlayer;
 
 public interface FrameVideoViewListener {
-    void mediaPlayerPrepared(MediaPlayer mediaPlayer);
+    void mediaPlayerPrepared( MediaPlayer mediaPlayer );
+
+    void mediaPlayerPrepareFailed( MediaPlayer mediaPlayer, String error );
 }
diff --git a/library/src/main/java/com/mklimek/frameviedoview/TextureViewImpl.java b/library/src/main/java/com/mklimek/frameviedoview/TextureViewImpl.java
@@ -85,7 +85,10 @@ private void prepare() {
             mediaPlayer.setOnBufferingUpdateListener(this);
             mediaPlayer.prepare();
         } catch (Exception e) {
-            LOG.error("cannot prepare media player with SurfaceTexture", e);
+            if ( listener != null ) {
+                listener.mediaPlayerPrepareFailed( mediaPlayer, e.toString() );
+            }
+            removeVideo();
         }
     }