Merge pull request flutter-webrtc#216 from ycherniavskyi/fix-odd-renderer-state-changed-handler-nulling

cloudwebrtc · web-flow · commit bc8dc677af22 · 2020-04-04T13:24:59.000+08:00
Use proper state lifecycle method for nulling renderer state changed handler
diff --git a/lib/rtc_video_view.dart b/lib/rtc_video_view.dart
@@ -128,8 +128,8 @@ class _RTCVideoViewState extends State<RTCVideoView> {
   }
 
   @override
-  void deactivate() {
-    super.deactivate();
+  void dispose() {
+    super.dispose();
     widget._renderer.onStateChanged = null;
   }
 
diff --git a/lib/web/rtc_video_view.dart b/lib/web/rtc_video_view.dart
@@ -171,8 +171,8 @@ class _RTCVideoViewState extends State<RTCVideoView> {
   }
 
   @override
-  void deactivate() {
-    super.deactivate();
+  void dispose() {
+    super.dispose();
     _renderer.onStateChanged = null;
   }
 

Original file line number	Diff line number	Diff line change
`@@ -128,8 +128,8 @@ class _RTCVideoViewState extends State<RTCVideoView> {`
`128`	`128`	`}`
`1 BDD3 29`	`129`
`130`	`130`	`@override`
`131`		`- void deactivate() {`
`132`		`- super.deactivate();`
	`131`	`+ void dispose() {`
	`132`	`+ super.dispose();`
`133`	`133`	`widget._renderer.onStateChanged = null;`
`134`	`134`	`}`
`135`	`135`
Original file line number	Diff line number	Diff line change
`@@ -171,8 +171,8 @@ class _RTCVideoViewState extends State<RTCVideoView> {`
`171`	`171`	`}`
`172`	`172`
`173`	`173`	`@override`
`174`		`- void deactivate() {`
`175`		`- super.deactivate();`
	`174`	`+ void dispose() {`
	`175`	`+ super.dispose();`
`176`	`176`	`_renderer.onStateChanged = null;`
`177`	`177`	`}`
`178`	`178`