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

WebGLでのビデオ再生をJavaScriptに委譲できるようにする #9

Merged
merged 12 commits into from Feb 22, 2024

Conversation

sus-taguchi-t
Copy link
Contributor

@sus-taguchi-t sus-taguchi-t commented Feb 6, 2024

何の変更を加えましたか?

  • WebGLでのビデオ再生をJavaScriptに委譲できるようにしました
    • プラットフォームに応じてVideoPlayerを扱うクラスEVideoPlayerを作成しました
    • 動画を扱うときにJavaScriptで使用するヘルパー関数GL、GLctxをjslibファイルに追加しました

何を確認しましたか?

実装

  • Frameworkの誤った使い方にすぐに気づけるように、無効な引数や無効なメソッド呼び出しに対するチェックが入っていることを確認しました
  • Framework実行時の動きが分かるように、ログ(Error/Warn/Info/Debug)を出力していることを確認しました
  • 静的解析で問題が見つからないことを確認しました
  • フレームワーク利用者が使うAPI(主にprivate以外)に C# ドキュメントを記述しました

テスト

  • 全ての自動テストが成功することを確認しました
    • MVS2 を用いた手動テストにて確認
  • テストカバレッジが100%になることを確認しました
  • サンプルがあるものはサンプルが動作することを確認しました
    • MVS2

変更影響

ガイドの PR

  • GuideのReleaseページに変更内容が追加されることを確認しました
  • GuideのModuleページ(機能ページ)に変更が反映されることを確認しました
  • GuideのLearningページに変更が反映されることを確認しました
    • 影響なし
  • Sample Applicationに変更が反映されることを確認しました
    • 影響なし

レビュアーへのメッセージ

  • 名前空間およびクラス名で悩んでいるので良い命名があれば教えて欲しいです
    • どちらも VideoPlayer だと Unity の VideoPlayer と衝突してコーディングが大変になります

Runtime/Video/IVideoPlayer.cs Outdated Show resolved Hide resolved
Runtime/Video/IVideoPlayer.cs Outdated Show resolved Hide resolved
WebScripts~/src/Video/VideoPlayerAdopter.ts Outdated Show resolved Hide resolved
WebScripts~/src/index.ts Show resolved Hide resolved
WebScripts~/src/Video/VideoPlayerAdopter.ts Outdated Show resolved Hide resolved
@kiyohome kiyohome merged commit 215f874 into develop Feb 22, 2024
1 check passed
@kiyohome kiyohome deleted the feature/video-player branch February 22, 2024 08:58
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.

None yet

3 participants