[Android] VideoPlayerActivity: remove the onStart method as its job can be done in the onResume method.

Adrien Maglo git at videolan.org
Tue Oct 29 15:23:22 CET 2013


vlc-ports/android | branch: master | Adrien Maglo <magsoft at videolan.org> | Tue Oct 29 15:15:23 2013 +0100| [907b736f73696bfd5f662abdcfa78de27683c9ff] | committer: Adrien Maglo

VideoPlayerActivity: remove the onStart method as its job can be done in the onResume method.
This commit also prevents the showOverlay method to be called two times.

> http://git.videolan.org/gitweb.cgi/vlc-ports/android.git/?a=commit;h=907b736f73696bfd5f662abdcfa78de27683c9ff
---

 .../src/org/videolan/vlc/gui/video/VideoPlayerActivity.java    |    8 +-------
 1 file changed, 1 insertion(+), 7 deletions(-)

diff --git a/vlc-android/src/org/videolan/vlc/gui/video/VideoPlayerActivity.java b/vlc-android/src/org/videolan/vlc/gui/video/VideoPlayerActivity.java
index 9e1ce0d..9b1162f 100644
--- a/vlc-android/src/org/videolan/vlc/gui/video/VideoPlayerActivity.java
+++ b/vlc-android/src/org/videolan/vlc/gui/video/VideoPlayerActivity.java
@@ -330,13 +330,6 @@ public class VideoPlayerActivity extends Activity implements IVideoPlayer {
     }
 
     @Override
-    protected void onStart() {
-        super.onStart();
-        showOverlay();
-        mSwitchingView = false;
-    }
-
-    @Override
     protected void onPause() {
         super.onPause();
 
@@ -420,6 +413,7 @@ public class VideoPlayerActivity extends Activity implements IVideoPlayer {
     @Override
     protected void onResume() {
         super.onResume();
+        mSwitchingView = false;
         AudioServiceController.getInstance().bindAudioService(this);
 
         load();



More information about the Android mailing list