From 36aa5e73686eadb4deef735ac6ea7978de8a2c3e Mon Sep 17 00:00:00 2001 From: Tommy Chow Date: Sat, 8 Aug 2026 15:43:35 -0700 Subject: [PATCH] fix(video): regenerate video store bindings MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 880a9c9 added the handleAndroidPipChanged and _setPipActive actions to VideoStoreBase without regenerating the mixin, so neither was wrapped by the MobX action controller. Both still ran, but their observable writes landed as separate notifications instead of one batched transaction — extra rebuilds on every PiP transition. Analyze and the test suite pass either way; only CI's generated-file diff check catches it. Co-Authored-By: Claude Opus 5 --- lib/screens/channel/video/video_store.g.dart | 24 ++++++++++++++++++++ 1 file changed, 24 insertions(+) diff --git a/lib/screens/channel/video/video_store.g.dart b/lib/screens/channel/video/video_store.g.dart index c12540e5..13e704fc 100644 --- a/lib/screens/channel/video/video_store.g.dart +++ b/lib/screens/channel/video/video_store.g.dart @@ -278,6 +278,30 @@ mixin _$VideoStore on VideoStoreBase, Store { } } + @override + void handleAndroidPipChanged(bool isInPip) { + final _$actionInfo = _$VideoStoreBaseActionController.startAction( + name: 'VideoStoreBase.handleAndroidPipChanged', + ); + try { + return super.handleAndroidPipChanged(isInPip); + } finally { + _$VideoStoreBaseActionController.endAction(_$actionInfo); + } + } + + @override + void _setPipActive(bool isInPip) { + final _$actionInfo = _$VideoStoreBaseActionController.startAction( + name: 'VideoStoreBase._setPipActive', + ); + try { + return super._setPipActive(isInPip); + } finally { + _$VideoStoreBaseActionController.endAction(_$actionInfo); + } + } + @override void handleAppResume() { final _$actionInfo = _$VideoStoreBaseActionController.startAction(