From 956f1f2da9635878cdaeda3794ff1b41ed4bdcf9 Mon Sep 17 00:00:00 2001
From: John Reck <jreck@google.com>
Date: Fri, 21 Jan 2011 10:33:23 -0800
Subject: Fix "Most visited" history group flicker

 Bug: 3374382

Change-Id: Idedf9804fa2010c91ac163965fb1d7a0617d438e
---
 src/com/android/browser/BrowserHistoryPage.java | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

(limited to 'src/com/android/browser/BrowserHistoryPage.java')

diff --git a/src/com/android/browser/BrowserHistoryPage.java b/src/com/android/browser/BrowserHistoryPage.java
index 6ea2bffcd..44c26ecce 100644
--- a/src/com/android/browser/BrowserHistoryPage.java
+++ b/src/com/android/browser/BrowserHistoryPage.java
@@ -557,13 +557,13 @@ public class BrowserHistoryPage extends Fragment
 
         @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 @@ public class BrowserHistoryPage extends Fragment
             if (!super.isEmpty()) {
                 return false;
             }
+            return isMostVisitedEmpty();
+        }
+
+        private boolean isMostVisitedEmpty() {
             return mMostVisited == null
                     || mMostVisited.isClosed()
                     || mMostVisited.getCount() == 0;
-- 
cgit v1.2.3