MUXに送信するパラメータを追加 #14

lefirea · 2025-07-25T06:21:51Z

MUXに送信するパラメータにviewerPlanStatusを追加し、視聴プラン（OneTimePlan or SubscriptionPlan）を記録できるようにした
https://linear.app/glucose/issue/SRS-576

App側の修正：https://github.com/glucoseinc/shirasu-app/pull/408

Android側は、mux-stats-sdk-exoplayerからは送れなかったので、mux-stats-sdk-media3に移行した。
それに伴い、exoplayer2からmedia3に移行した。
補足として、下記のように移行が推奨されていたのでいずれ必要なことではあった。

現在スタンドアロンの com.google.android.exoplayer2 ライブラリと androidx.media を使用しているアプリは、androidx.media3 に移行する必要があります。
https://developer.android.com/media/media3/exoplayer/migration-guide?hl=ja

media3への移行作業はflutter-packagesのフォーク元が既に対応していたので取り込むことで対応した。
その後、弊社の独自拡張分を手作業で修正した。
具体的な手順は以下。

# 1. 作業用ブランチを切る
git checkout -b media3

# 2. フォーク元から取り込みたい部分（`video_player_android/`）だけを取り込む
git checkout upstream/main -- packages/video_player/video_player_android

# 3. 弊社側の最新コミット（`5bb58fc`）でrebaseして、弊社側の追加分を加え直す
# ※ 2. でファイルが丸々上書きされて弊社分が消えるので、rebaseして加える必要がある
git rebase 5bb58fc

# (ここまででexoplayerからmedia3への移行は終了)

# 4. `mux-stats-sdk-exoplayer`から`mux-stats-sdk-media3`に移行する
手作業で当該パッケージ使用部分を書き換える

この取り込み作業のコミット：chore: exoplayerからmedia3に移行 f06b94f

この手順のあと、viewer_plan_statusを送信するコードを追加した。

MUXのパッケージ更新に伴い、videoData.setVideoCdnがdeprecatedになっていたので削除した。

iOS側はパッケージのバージョンアップ以外の作業は不要だった。

Copilot

Pull Request Overview

This PR adds a new viewerPlanStatus parameter to MUX configuration to track viewing plans (OneTimePlan or SubscriptionPlan). The Android implementation migrates from ExoPlayer2 to Media3 and updates the MUX SDK accordingly, while iOS only requires a package version update.

Adds viewerPlanStatus parameter to MUX configuration across all platform interfaces
Migrates Android video player from ExoPlayer2 to Media3 with updated MUX SDK
Updates iOS MUX Stats dependency to version 4.0

Reviewed Changes

Copilot reviewed 31 out of 31 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
`video_player_platform_interface.dart`	Adds `viewerPlanStatus` field to MuxConfig class
`video_player_avfoundation/pigeons/messages.dart`	Adds `viewerPlanStatus` to MuxConfigMessage pigeon definition
`video_player_avfoundation/lib/src/messages.g.dart`	Generated pigeon code updates for new field
`video_player_avfoundation/lib/src/avfoundation_video_player.dart`	Maps `viewerPlanStatus` to MUX message
`video_player_avfoundation/ios/video_player_avfoundation.podspec`	Updates MUX Stats dependency to 4.0
iOS native files	Updates pigeon-generated code and implements MUX viewer data
Android files	Comprehensive migration from ExoPlayer2 to Media3 with MUX SDK updates

Copilot · 2025-08-12T06:16:27Z

...o_player_android/android/src/main/java/io/flutter/plugins/videoplayer/VideoPlayerPlugin.java


-    if (arg.getVideoCdn() != null)
-      videoData.setVideoCdn(arg.getVideoCdn());
+    // deprecatedになったが、iOS版との統一のために残しておく


[nitpick] The comment contains mixed languages (Japanese and English). Consider using consistent language in comments for better code maintainability.

Suggested change

// deprecatedになったが、iOS版との統一のために残しておく

// This is deprecated, but kept for consistency with the iOS version.

Copilot · 2025-08-12T06:16:27Z

...r/video_player_android/android/src/main/java/io/flutter/plugins/videoplayer/VideoPlayer.java

+    PendingIntent sessionActivity = buildSessionActivityPendingIntent(context);
+
+    MediaSession.Builder builder = new MediaSession.Builder(context, exoPlayer).setId("VideoPlayer");
+    if(sessionActivity != null){


Missing space after if keyword. Should be if (sessionActivity != null) { to follow Java formatting conventions.

Suggested change

if(sessionActivity != null){

if (sessionActivity != null) {

Copilot

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copilot

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

ryo0511 · 2025-08-18T11:07:13Z

...ayer/video_player_android/android/src/main/java/io/flutter/plugins/videoplayer/Messages.java

      PlaybackSpeedMessage pigeonResult = new PlaybackSpeedMessage();
      Object textureId = list.get(0);
-      pigeonResult.setTextureId((textureId == null) ? null : ((textureId instanceof Integer) ? (Integer) textureId : (Long) textureId));
+      pigeonResult.setTextureId(


lintが崩れてるので直してください

ryo0511 · 2025-08-21T08:44:38Z

...ayer/video_player_android/android/src/main/java/io/flutter/plugins/videoplayer/Messages.java

                  wrapped.add(0, null);
-                }
- catch (Throwable exception) {
+                } catch (Throwable exception) {


他の箇所もlint崩れてるので直してください

一通り直しました
これで大丈夫だと思います

ryo0511 · 2025-08-22T11:10:56Z

この変更は以降スクリプトによるものですか？
https://developer.android.com/media/media3/exoplayer/migration-guide?hl=ja#usingscript

　media3への移行作業はflutter-packagesのフォーク元が既に対応していたので取り込むことで対応した。

これはどの部分がそうですか？
取り込んだ部分と自前で手を加えた部分とがコミットでわかるようにするか、欲を言えば別PRにしてもらった方が経緯が追えて助かります

lefirea · 2025-08-25T02:54:14Z

@ryo0511
取り込み手順をPR本文に追記しました
ご確認ください。

フォーマットが変わってる部分については、この取り込み手順で発生した部分です。
フォーク元でそのフォーマットになってます。

スクリプトについては、フォーク元がスクリプトを使って移行しており、今回の作業ではその移行後を取り込んでます

fix: 送信するパラメータを追加

0c26ea7

github-actions bot added p: video_player platform-ios labels Jul 25, 2025

漆原宏丞 added 2 commits August 12, 2025 13:04

chore: exoplayerからmedia3に移行

f06b94f

feat: Androidからもviewer_plan_statusを送れるようにした

7505a83

github-actions bot added the platform-android label Aug 12, 2025

refactor: デバッグコードを削除

c3c2cae

lefirea marked this pull request as ready for review August 12, 2025 05:55

lefirea requested a review from Copilot August 12, 2025 06:03

This comment was marked as outdated.

Sign in to view

refactor: 不要なコードの削除

1f6bd0f

lefirea requested a review from Copilot August 12, 2025 06:15

Copilot AI reviewed Aug 12, 2025

View reviewed changes

refactor: 不要なコードを削除

56915cf

lefirea requested a review from Copilot August 12, 2025 07:18

Copilot AI reviewed Aug 12, 2025

View reviewed changes

lefirea requested review from Copilot, hagihara1483 and ryo0511 August 12, 2025 07:28

Copilot AI reviewed Aug 12, 2025

View reviewed changes

ryo0511 requested changes Aug 18, 2025

View reviewed changes

refactor: lint崩れを修正

26db06f

lefirea requested a review from ryo0511 August 19, 2025 05:58

ryo0511 reviewed Aug 21, 2025

View reviewed changes

refactor: reformat

f3de554

lefirea requested a review from ryo0511 August 21, 2025 09:43

ryo0511 approved these changes Aug 25, 2025

View reviewed changes

hagihara1483 approved these changes Aug 26, 2025

View reviewed changes

ryo0511 merged commit 82cdfde into glucose-hosted Aug 26, 2025
1 check passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

MUXに送信するパラメータを追加 #14

MUXに送信するパラメータを追加 #14

Uh oh!

lefirea commented Jul 25, 2025 •

edited

Loading

Uh oh!

This comment was marked as outdated.

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Aug 12, 2025

Uh oh!

Copilot AI Aug 12, 2025

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI left a comment

Uh oh!

ryo0511 Aug 18, 2025

Uh oh!

ryo0511 Aug 21, 2025

Uh oh!

lefirea Aug 21, 2025

Uh oh!

ryo0511 commented Aug 22, 2025

Uh oh!

lefirea commented Aug 25, 2025 •

edited

Loading

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

	// deprecatedになったが、iOS版との統一のために残しておく
	// This is deprecated, but kept for consistency with the iOS version.

MUXに送信するパラメータを追加 #14

MUXに送信するパラメータを追加 #14

Uh oh!

Conversation

lefirea commented Jul 25, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

This comment was marked as outdated.

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Copilot AI Aug 12, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Aug 12, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

ryo0511 Aug 18, 2025

Choose a reason for hiding this comment

Uh oh!

ryo0511 Aug 21, 2025

Choose a reason for hiding this comment

Uh oh!

lefirea Aug 21, 2025

Choose a reason for hiding this comment

Uh oh!

ryo0511 commented Aug 22, 2025

Uh oh!

lefirea commented Aug 25, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

lefirea commented Jul 25, 2025 •

edited

Loading

lefirea commented Aug 25, 2025 •

edited

Loading