-
Notifications
You must be signed in to change notification settings - Fork 54
[video_player_videohole] Migrating from Platform Channels to Dart FFI #1073
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
Open
gin7773
wants to merge
45
commits into
flutter-tizen:main
Choose a base branch
from
gin7773:videohole/ffi_migration
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
45 commits
Select commit
Hold shift + click to select a range
2f0fd5d
FFI initialize and create
gin7773 19c4bd1
Migration of methods other than (de)activate
gin7773 51571e3
remove BasicMessageChannel and VideoPlayerVideoholeApi
gin7773 643a04d
code format
gin7773 1815a22
replace EventChannel with ffi
gin7773 93de4a2
fixed dart analyze issue
gin7773 3a558d6
Code refactoring
gin7773 46dee32
Ensure minimal changes
gin7773 7efc252
remove VideoPlayerTizenPlugin
gin7773 c6c0570
code format
gin7773 29d58da
fixed UI not refreshing issue and code structure updated
gin7773 619dc62
code format
gin7773 45eafa5
Complete the registration and destruction of playerEventPort, and res…
gin7773 f5dac1b
P1-1: Use nlohmann/json library for JSON parsing
gin7773 486d9e8
P1-2: Add FFI port lifecycle cleanup for hot restart
gin7773 16a4f83
code format
gin7773 b4dbc98
Fixed the potential deadlock issue and unescaped JSON issue
gin7773 11a01a4
Fix: FFI event port binding and Restore playback issues
gin7773 534df84
remove unused log
gin7773 4530daf
fixed some review issues
gin7773 2fb3939
refactor Restore()
gin7773 06b983b
remove unused code
gin7773 9996e07
fix crash issue
gin7773 09c5443
update version
gin7773 e08ba75
remove unused code
gin7773 daa0dd8
add ffi
gin7773 9866b28
Refactor: Extract FFI implementation to ffi_messages.cc
gin7773 6e9dd49
Simplify ffi_messages.h internal API
gin7773 97da2d7
Fix: Address PR review comments
gin7773 6582664
Fix PR #1073 review comments
gin7773 3cff139
Add TODO for ecore_wl2 thread safety issue
gin7773 0387b7c
Fixed review issues
gin7773 45342b6
Unified code architecture
gin7773 fd3e299
Corrected the state check in StopAndDestroy()
gin7773 1beebbe
fixed some review issues
gin7773 6b74187
Clean up FFI event port initialization
gin7773 bb93681
Fix seekTo completion
gin7773 630e3d4
Fix pending seekTo handing
gin7773 4db5a67
Update CHANGELOG
gin7773 c808844
Add native seek guard
gin7773 2b3d8ce
Merge branch 'main' into videohole/ffi_migration
gin7773 86d413f
Add LICENSE and remove unnecessary code
gin7773 bd3c6fb
Merge branch 'main' into videohole/ffi_migration
gin7773 806c17f
Rename manually maintained FFI bindings file and remove redundant code
gin7773 29b02a9
Merge branch 'main' into videohole/ffi_migration
gin7773 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You must not delete the 0.5.11 changelog.