commit 6c5e169de610b0dee24813e7126818d47fc6fc96
author Tony Wickham <twickham@google.com> Fri Apr 20 11:32:03 2018 -0700
committer Tony Wickham <twickham@google.com> Fri Apr 20 11:32:03 2018 -0700
tree d4c48d5a803782c3f18b9f264b5a45553b59f38c
parent f589eebbec1d6826f15d15b09eb712b8b8a30f43

Prevent unecessary invalidates

updateEmptyMessage() calls invalidate() whenever size has changed. We
were only updating mLastMeasuredSize when showing the empty message,
however, so we were invalidating whenever updateEmptyMessage was called.

Change-Id: Ic9046738e4f53d53aded6347a273522c53394991

quickstep/src/com/android/quickstep/views/RecentsView.java

diff --git a/quickstep/src/com/android/quickstep/views/RecentsView.java b/quickstep/src/com/android/quickstep/views/RecentsView.java
index 6a2df0b..2e45d83 100644
--- a/quickstep/src/com/android/quickstep/views/RecentsView.java
+++ b/quickstep/src/com/android/quickstep/views/RecentsView.java
@@ -943,6 +943,7 @@
         boolean hasValidSize = getWidth() > 0 && getHeight() > 0;
         if (sizeChanged && hasValidSize) {
             mEmptyTextLayout = null;
+            mLastMeasureSize.set(getWidth(), getHeight());
         }
 
         if (!mShowEmptyMessage) return;
@@ -953,7 +954,6 @@
         scrollTo(0, 0);
 
         if (hasValidSize && mEmptyTextLayout == null) {
-            mLastMeasureSize.set(getWidth(), getHeight());
             int availableWidth = mLastMeasureSize.x - mEmptyMessagePadding - mEmptyMessagePadding;
             mEmptyTextLayout = StaticLayout.Builder.obtain(mEmptyMessage, 0, mEmptyMessage.length(),
                     mEmptyMessagePaint, availableWidth)