commit 956f1f2da9635878cdaeda3794ff1b41ed4bdcf9
author John Reck <jreck@google.com> Fri Jan 21 10:33:23 2011 -0800
committer John Reck <jreck@google.com> Fri Jan 21 10:33:23 2011 -0800
tree b4a6c39d7a4aedb275efd047388b390e1f0fcf70
parent ee804cd408ebbaa75b8832784934531520e455e8

Fix "Most visited" history group flicker

 Bug: 3374382

Change-Id: Idedf9804fa2010c91ac163965fb1d7a0617d438e

src/com/android/browser/BrowserHistoryPage.java

diff --git a/src/com/android/browser/BrowserHistoryPage.java b/src/com/android/browser/BrowserHistoryPage.java
index 6ea2bff..44c26ec 100644
--- a/src/com/android/browser/BrowserHistoryPage.java
+++ b/src/com/android/browser/BrowserHistoryPage.java
@@ -557,13 +557,13 @@
 
         @Override
         public int getGroupCount() {
-            return super.getGroupCount() + (mMostVisited != null ? 1 : 0);
+            return super.getGroupCount() + (!isMostVisitedEmpty() ? 1 : 0);
         }
 
         @Override
         public int getChildrenCount(int groupPosition) {
             if (groupPosition >= super.getGroupCount()) {
-                if (mMostVisited == null) {
+                if (isMostVisitedEmpty()) {
                     return 0;
                 }
                 return mMostVisited.getCount();
@@ -576,6 +576,10 @@
             if (!super.isEmpty()) {
                 return false;
             }
+            return isMostVisitedEmpty();
+        }
+
+        private boolean isMostVisitedEmpty() {
             return mMostVisited == null
                     || mMostVisited.isClosed()
                     || mMostVisited.getCount() == 0;