Skip to content

Conversation

@t-miya
Copy link
Contributor

@t-miya t-miya commented Jan 21, 2026

背景

現状の実装ではカメラ有効の設定での開始時は必ずカメラキャプチャが生成されるため、カメラキャプチャの生成遅延により
開始時カメラミュートを行うことができない。
ちなみにカメラ無効で開始した場合は MediaChannel.setVideHardMute による解除をすることができない。

修正内容

接続設定に initialCameraEnabled を追加することにより、カメラキャプチャの生成を遅延できるようにする。
また、VideoHardMuteActor. setMute(false) でカメラキャプチャが未生成の場合は生成するようにする。

@t-miya
Copy link
Contributor Author

t-miya commented Jan 21, 2026

!haiku レビューして

@github-actions

This comment was marked as resolved.

@t-miya
Copy link
Contributor Author

t-miya commented Jan 21, 2026

!haiku 修正したので再度レビューして
CHANGS の並びは現状で問題ないかと思われる

@github-actions

This comment was marked as resolved.

@t-miya
Copy link
Contributor Author

t-miya commented Jan 21, 2026

!sonnet haiku のレビュー内容も踏まえてダブルチェックして

@github-actions

This comment was marked as resolved.

@t-miya t-miya requested a review from zztkm January 21, 2026 03:57
Copy link
Contributor

@zztkm zztkm left a comment

Choose a reason for hiding this comment

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

コード中のコメントなどはわかりやすいと思いました 👍🏼

@t-miya
Copy link
Contributor Author

t-miya commented Jan 21, 2026

!haiku 修正箇所についてレビューして

@github-actions

This comment was marked as resolved.

@t-miya
Copy link
Contributor Author

t-miya commented Jan 21, 2026

!haiku
setVideoHardMute(false) 側はまず currentCameraVideoCapturer()(CameraVideoCapturer.current)を見て、既に起動中なら何もしないので、self.capturer == nil でも重複 start にならない想定

@github-actions

This comment was marked as resolved.

@t-miya
Copy link
Contributor Author

t-miya commented Jan 21, 2026

!haiku

  • CHANGES の順番は問題ない状態になっているため順序に関する指摘は以後不要
  • startCameraVideoCapture で開始した capturer を VideoHardMuteActor.capturer(self.capturer)には保持していないため、次に setVideoHardMute(true) する際は currentCameraVideoCapturer()(CameraVideoCapturer.current)から実行中キャプチャラーを取得して stop する設計としている

@github-actions

This comment was marked as resolved.

@t-miya
Copy link
Contributor Author

t-miya commented Jan 21, 2026

!haiku VideoHardMuteActor の self.capturer は restart 用に保持している、という意図がわかりやすくなるように変数名等を修正した

@github-actions

This comment was marked as resolved.

Copy link
Contributor

@zztkm zztkm left a comment

Choose a reason for hiding this comment

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

LGTM です !

@t-miya t-miya merged commit 496959a into develop Jan 22, 2026
5 checks passed
@t-miya t-miya deleted the feature/initial-camera-enabled branch January 22, 2026 01:09
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.

3 participants