-Removed duplicate dialog open instances in service player activity.

This commit is contained in:
John Zhen Mo 2018-03-22 18:44:03 -07:00
parent 72eaff148c
commit 02f48ccc7f

View file

@ -433,12 +433,10 @@ public abstract class ServicePlayerActivity extends AppCompatActivity
player.onShuffleClicked();
} else if (view.getId() == playbackSpeedButton.getId()) {
PlaybackParameterDialog.newInstance(player.getPlaybackSpeed(),
player.getPlaybackPitch()).show(getSupportFragmentManager(), getTag());
openPlaybackParameterDialog();
} else if (view.getId() == playbackPitchButton.getId()) {
PlaybackParameterDialog.newInstance(player.getPlaybackSpeed(),
player.getPlaybackPitch()).show(getSupportFragmentManager(), getTag());
openPlaybackParameterDialog();
} else if (view.getId() == metadata.getId()) {
scrollToSelected();
@ -450,9 +448,15 @@ public abstract class ServicePlayerActivity extends AppCompatActivity
}
////////////////////////////////////////////////////////////////////////////
// Playback Parameters Listener
// Playback Parameters
////////////////////////////////////////////////////////////////////////////
private void openPlaybackParameterDialog() {
if (player == null) return;
PlaybackParameterDialog.newInstance(player.getPlaybackSpeed(),
player.getPlaybackPitch()).show(getSupportFragmentManager(), getTag());
}
@Override
public void onPlaybackParameterChanged(float playbackTempo, float playbackPitch) {
if (player != null) player.setPlaybackParameters(playbackTempo, playbackPitch);