Skip to content
This repository has been archived by the owner on Feb 22, 2023. It is now read-only.

video_player: Allow audio playback in silent mode #624

Merged
merged 4 commits into from
Jun 26, 2018

Conversation

cbenhagen
Copy link
Contributor

@cbenhagen cbenhagen commented Jun 17, 2018

This is the default behaviour of video apps on iOS.
(fixes flutter/flutter#17911)

I considered setting the category back to AVAudioSessionCategorySoloAmbient on dispose but this led to race conditions when switching between videos.

Copy link
Contributor

@mravn-google mravn-google left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the contribution. Please bump the plugin version in pubspec.yaml and add a CHANGELOG.md entry.

@cbenhagen
Copy link
Contributor Author

@mravn-google done ;)

@mravn-google
Copy link
Contributor

This branch seems to need a rebase now. Sorry.

@cbenhagen
Copy link
Contributor Author

No worries. Branch has been rebased.

@mravn-google
Copy link
Contributor

Thanks. Of course your two PRs are now conflicting...

@cbenhagen
Copy link
Contributor Author

Was to be expected yes. Merged in the master branch again.

@cbenhagen
Copy link
Contributor Author

As the other PR has already been released I bumped the version again. Also fixed a small typo in the changelog.

Copy link
Contributor

@mravn-google mravn-google left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mravn-google mravn-google merged commit c7c3f1b into flutter:master Jun 26, 2018
@cbenhagen cbenhagen deleted the patch-2 branch June 26, 2018 13:19
najeira added a commit to najeira/flutter-video-player that referenced this pull request Aug 22, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
3 participants