2 files changed, 13 insertions, 29 deletions
diff --git a/app/src/main/java/de/danoeh/antennapod/activity/AboutActivity.java b/app/src/main/java/de/danoeh/antennapod/activity/AboutActivity.java
index 5dd4a357b..1bcdada44 100644
--- a/app/src/main/java/de/danoeh/antennapod/activity/AboutActivity.java
+++ b/app/src/main/java/de/danoeh/antennapod/activity/AboutActivity.java
@@ -1,7 +1,9 @@
 package de.danoeh.antennapod.activity;
 
+import android.content.Intent;
 import android.content.res.TypedArray;
 import android.graphics.Color;
+import android.net.Uri;
 import android.os.Build;
 import android.os.Bundle;
 import android.support.v7.app.AppCompatActivity;
@@ -54,12 +56,15 @@ public class AboutActivity extends AppCompatActivity {
 
             @Override
             public boolean shouldOverrideUrlLoading(WebView view, String url) {
-                if (!url.startsWith("http")) {
+                if (url.startsWith("http")) {
+                    Intent browserIntent = new Intent(Intent.ACTION_VIEW, Uri.parse(url));
+                    startActivity(browserIntent);
+                    return true;
+                } else {
                     url = url.replace("file:///android_asset/", "");
                     loadAsset(url);
                     return true;
                 }
-                return false;
             }
 
         });
diff --git a/app/src/main/java/de/danoeh/antennapod/activity/VideoplayerActivity.java b/app/src/main/java/de/danoeh/antennapod/activity/VideoplayerActivity.java
index fa5012b74..78cc15b2c 100644
--- a/app/src/main/java/de/danoeh/antennapod/activity/VideoplayerActivity.java
+++ b/app/src/main/java/de/danoeh/antennapod/activity/VideoplayerActivity.java
@@ -47,7 +47,6 @@ public class VideoplayerActivity extends MediaplayerActivity {
      */
     private boolean videoControlsShowing = true;
     private boolean videoSurfaceCreated = false;
-    private boolean playbackStoppedUponExitVideo = false;
     private boolean destroyingDueToReload = false;
 
     private VideoControlsHider videoControlsHider = new VideoControlsHider(this);
@@ -78,7 +77,6 @@ public class VideoplayerActivity extends MediaplayerActivity {
     @Override
     protected void onResume() {
         super.onResume();
-        playbackStoppedUponExitVideo = false;
         if (TextUtils.equals(getIntent().getAction(), Intent.ACTION_VIEW)) {
             playExternalMedia(getIntent(), MediaType.VIDEO);
         } else if (PlaybackService.isCasting()) {
@@ -93,32 +91,12 @@ public class VideoplayerActivity extends MediaplayerActivity {
 
     @Override
     protected void onStop() {
-        stopPlaybackIfUserPreferencesSpecified(); // MUST be called before super.onStop(), while it still has member variable controller
         super.onStop();
         if (!PictureInPictureUtil.isInPictureInPictureMode(this)) {
             videoControlsHider.stop();
         }
     }
 
-    void stopPlaybackIfUserPreferencesSpecified() {
-        // to avoid the method being called twice during leaving Videoplayer
-        // , which will double-pause the media
-        // (it is usually first called by surfaceHolderCallback.surfaceDestroyed(),
-        //  then VideoplayerActivity.onStop() , but sometimes VideoplayerActivity.onStop()
-        //  will first be invoked.)
-        if (playbackStoppedUponExitVideo) {
-            return;
-        }
-        playbackStoppedUponExitVideo = true;
-
-        if (controller != null && !destroyingDueToReload
-                && UserPreferences.getVideoBackgroundBehavior()
-                != UserPreferences.VideoBackgroundBehavior.CONTINUE_PLAYING) {
-            Log.v(TAG, "stop video playback per UserPreference");
-            controller.notifyVideoSurfaceAbandoned();
-        }
-    }
-
     @Override
     public void onUserLeaveHint () {
         if (!PictureInPictureUtil.isInPictureInPictureMode(this) && UserPreferences.getVideoBackgroundBehavior()
@@ -297,12 +275,13 @@ public class VideoplayerActivity extends MediaplayerActivity {
 
         @Override
         public void surfaceDestroyed(SurfaceHolder holder) {
-            Log.d(TAG, "Videosurface was destroyed." );
-            Log.v(TAG, "    hasController=" + (controller != null)
-                    + " , destroyingDueToReload=" + destroyingDueToReload
-                    + " , videoBackgroundBehavior=" + UserPreferences.getVideoBackgroundBehavior());
+            Log.d(TAG, "Videosurface was destroyed");
             videoSurfaceCreated = false;
-            stopPlaybackIfUserPreferencesSpecified();
+            if (controller != null && !destroyingDueToReload
+                    && UserPreferences.getVideoBackgroundBehavior()
+                    != UserPreferences.VideoBackgroundBehavior.CONTINUE_PLAYING) {
+                controller.notifyVideoSurfaceAbandoned();
+            }
         }
     };