atori74 / streamsync

Sync timing of streaming simultaneously viewed by host and clients
1 stars 0 forks source link

popupの状態遷移がpopupのアクティブ時でないとうまく動作しない #32

Closed atori74 closed 3 years ago

atori74 commented 3 years ago

popupに表示しているログは、backgroundからmessageを受け取ってappendしていくが、 popup.jsに書いているので、popupがそもそもアクティブじゃないときはmessageのハンドリングも実行されない

ログだけでなく、popupの状態を決めるstorageのstatusデータの書き換え処理の最中などにpopupが閉じられてしまうと 実際のroom参加状態とpopupからの状態の見え方に不整合がでてしまう可能性がある。

実際、Clientとしてroom参加する際、ws serverからjoinSuccessのメッセージを受け取って popup.jsでuserLogに"joined"と書き込むが、 同じタイミングでbackground側から動画ページを別タブで開く処理が走るため、popupが閉じられてしまい うまくロギングできていない

atori74 commented 3 years ago

popup.js上ではログの書き込みやstatusの書き換えなど重要なことはさせず、 backgroundにmessageを送るだけにとどめ、 background側でメインの処理を行うようにしたほうが良さそう。