cmsdk: fix visible check in hasChanged

Previously the visible check was comparing the newly set value to
itself. Now we cache the old value, set new value, and can do a proper
comparison.

Change-Id: Idbe921bfe9e311551d6bfbad432c647c129f376e
Signed-off-by: Roman Birg <roman@cyngn.com>

1 files changed, 3 insertions, 3 deletions

diff --git a/sdk/src/java/cyanogenmod/externalviews/ExternalViewProperties.java b/sdk/src/java/cyanogenmod/externalviews/ExternalViewProperties.java
index 2b4404d..e8adddf 100644
--- a/sdk/src/java/cyanogenmod/externalviews/ExternalViewProperties.java
+++ b/sdk/src/java/cyanogenmod/externalviews/ExternalViewProperties.java
@@ -84,12 +84,12 @@ public class ExternalViewProperties {
         if (mDecorView != null) {
             mDecorView.getHitRect(mHitRect);
         }
-        boolean visible = mView.getLocalVisibleRect(mHitRect);
-        mVisible = visible;
+        boolean wasVisible = mVisible;
+        mVisible = mView.getLocalVisibleRect(mHitRect);
 
         // Check if anything actually changed
         return previousX != newX || previousY != newY
                 || previousWidth != mWidth || previousHeight != mHeight
-                || mVisible != visible;
+                || mVisible != wasVisible;
     }
 }