Compose visibility fixes (#85)

* Fix sending up updates when not automanaging video

* Fix compose visibility

@@ -40,7 +40,7 @@ class RemoteTrackPublication(
                     }
                     }
                 }
                 }
 
 
-                if (value is RemoteVideoTrack) {
+                if (value is RemoteVideoTrack && value.autoManageVideo) {
                     handleVideoDimensionsChanged(value.lastDimensions)
                     handleVideoDimensionsChanged(value.lastDimensions)
                     handleVisibilityChanged(value.lastVisibility)
                     handleVisibilityChanged(value.lastVisibility)
                 }
                 }

@@ -33,33 +33,41 @@ abstract class VideoSinkVisibility : Observable() {
 }
 }
 
 
 class ComposeVisibility : VideoSinkVisibility() {
 class ComposeVisibility : VideoSinkVisibility() {
-    private var lastCoordinates: LayoutCoordinates? = null
+    private var coordinates: LayoutCoordinates? = null
 
 
+    private var lastVisible = isVisible()
+    private var lastSize = size()
     override fun isVisible(): Boolean {
     override fun isVisible(): Boolean {
-        return (lastCoordinates?.isAttached == true && lastCoordinates?.size?.width != 0 && lastCoordinates?.size?.height != 0)
+        return (coordinates?.isAttached == true &&
+                coordinates?.size?.width != 0 &&
+                coordinates?.size?.height != 0)
     }
     }
 
 
     override fun size(): Track.Dimensions {
     override fun size(): Track.Dimensions {
-        val width = lastCoordinates?.size?.width ?: 0
-        val height = lastCoordinates?.size?.height ?: 0
+        val width = coordinates?.size?.width ?: 0
+        val height = coordinates?.size?.height ?: 0
         return Track.Dimensions(width, height)
         return Track.Dimensions(width, height)
     }
     }
 
 
+    // Note, LayoutCoordinates are mutable and may be reused.
     fun onGloballyPositioned(layoutCoordinates: LayoutCoordinates) {
     fun onGloballyPositioned(layoutCoordinates: LayoutCoordinates) {
-        val lastVisible = isVisible()
-        val lastSize = size()
-        lastCoordinates = layoutCoordinates
+        coordinates = layoutCoordinates
+        val visible = isVisible()
+        val size = size()
 
 
-        if (lastVisible != isVisible() || lastSize != size()) {
+        if (lastVisible != visible || lastSize != size) {
             notifyChanged()
             notifyChanged()
         }
         }
+
+        lastVisible = visible
+        lastSize = size
     }
     }
 
 
     fun onDispose() {
     fun onDispose() {
-        if (lastCoordinates == null) {
+        if (coordinates == null) {
             return
             return
         }
         }
-        lastCoordinates = null
+        coordinates = null
         notifyChanged()
         notifyChanged()
     }
     }
 }
 }