Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[video_player_avplay] Add video_player_avplay package #620

Merged
merged 16 commits into from
Nov 9, 2023

Conversation

xiaowei-guan
Copy link
Contributor

@xiaowei-guan xiaowei-guan commented Sep 25, 2023

@xiaowei-guan xiaowei-guan marked this pull request as ready for review October 7, 2023 02:29
Copy link
Member

@JSUYA JSUYA left a comment

Choose a reason for hiding this comment

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

I left a few comments. I haven't seen all code yet. Please check comments

packages/video_player_avplayer/tizen/src/plus_player.cc Outdated Show resolved Hide resolved
packages/video_player_avplayer/pigeons/messages.dart Outdated Show resolved Hide resolved
packages/video_player_avplayer/tizen/src/plus_player.h Outdated Show resolved Hide resolved
packages/video_player_avplayer/tizen/src/plus_player.cc Outdated Show resolved Hide resolved
packages/video_player_avplayer/tizen/src/plus_player.cc Outdated Show resolved Hide resolved
packages/video_player_avplayer/tizen/src/plus_player.cc Outdated Show resolved Hide resolved
packages/video_player_avplayer/tizen/src/plus_player.cc Outdated Show resolved Hide resolved
packages/video_player_avplayer/tizen/src/plus_player.cc Outdated Show resolved Hide resolved
* Update plusplayer header file

* Fix cannot play live stream issue
README.md Outdated Show resolved Hide resolved
packages/video_player_avplayer/lib/video_player.dart Outdated Show resolved Hide resolved
packages/video_player_avplayer/tizen/src/plus_player.cc Outdated Show resolved Hide resolved
packages/video_player_avplayer/tizen/src/plus_player.cc Outdated Show resolved Hide resolved
packages/video_player_avplayer/tizen/src/plus_player.cc Outdated Show resolved Hide resolved
packages/video_player_avplayer/tizen/src/plus_player.cc Outdated Show resolved Hide resolved
packages/video_player_avplayer/tizen/src/plus_player.cc Outdated Show resolved Hide resolved
packages/video_player_avplayer/tizen/src/plus_player.cc Outdated Show resolved Hide resolved
* Codes cleanup

* Change message channel Api name

* Refactor track type

* Rename package name

* Update labeler.yml and recipe.yaml

* Not return -1

* Return FlutterError if failed
…#7)

* Remove ffi and implement request license through method channel

* Extract a new method InstallKey for installing eme key

* drm_manager codes cleanup

* Minor codes cleanup
@xiaowei-guan xiaowei-guan changed the title [video_player_avplayer] Add video_player_avplayer package [video_player_avplay] Add video_player_avplayer package Nov 1, 2023
@hyue7 hyue7 changed the title [video_player_avplay] Add video_player_avplayer package [video_player_avplay] Add video_player_avplay package Nov 3, 2023
* Use plusplayer-wrapper to replace plusplayer header files

* Add license and rename ifndef

* Update plusplayer-wrapper for adding GetStreamingProperty Api

* Update README.md
@xiaowei-guan xiaowei-guan merged commit 1b720d0 into flutter-tizen:master Nov 9, 2023
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants