Fixed unnecessary starts

This commit is contained in:
Nite 2020-07-11 11:15:16 +02:00
parent 3d61dde83f
commit 4b0809f905
No known key found for this signature in database
GPG Key ID: 1D1AD59B1C6386C1
2 changed files with 17 additions and 28 deletions

View File

@ -176,28 +176,6 @@ public class MediaPlayerControllerImpl implements MediaPlayerController
}); });
} }
@Override
public synchronized void seekTo(final int position)
{
MediaPlayerService.executeOnStartedMediaPlayerService(context, new Consumer<MediaPlayerService>() {
@Override
public void accept(MediaPlayerService mediaPlayerService) {
mediaPlayerService.seekTo(position);
}
});
}
@Override
public synchronized void pause()
{
MediaPlayerService.executeOnStartedMediaPlayerService(context, new Consumer<MediaPlayerService>() {
@Override
public void accept(MediaPlayerService mediaPlayerService) {
mediaPlayerService.pause();
}
});
}
@Override @Override
public synchronized void start() public synchronized void start()
{ {
@ -209,15 +187,25 @@ public class MediaPlayerControllerImpl implements MediaPlayerController
}); });
} }
@Override
public synchronized void seekTo(final int position)
{
MediaPlayerService mediaPlayerService = MediaPlayerService.getRunningInstance();
if (mediaPlayerService != null) mediaPlayerService.seekTo(position);
}
@Override
public synchronized void pause()
{
MediaPlayerService mediaPlayerService = MediaPlayerService.getRunningInstance();
if (mediaPlayerService != null) mediaPlayerService.pause();
}
@Override @Override
public synchronized void stop() public synchronized void stop()
{ {
MediaPlayerService.executeOnStartedMediaPlayerService(context, new Consumer<MediaPlayerService>() { MediaPlayerService mediaPlayerService = MediaPlayerService.getRunningInstance();
@Override if (mediaPlayerService != null) mediaPlayerService.stop();
public void accept(MediaPlayerService mediaPlayerService) {
mediaPlayerService.stop();
}
});
} }
@Override @Override

View File

@ -269,6 +269,7 @@ public class MediaPlayerLifecycleSupport
mediaPlayerController.togglePlayPause(); mediaPlayerController.togglePlayPause();
break; break;
case Constants.CMD_STOP: case Constants.CMD_STOP:
// TODO: There is a stop() function, shouldn't we use that?
mediaPlayerController.pause(); mediaPlayerController.pause();
mediaPlayerController.seekTo(0); mediaPlayerController.seekTo(0);
break; break;