diff options
| author | John Reck <jreck@google.com> | 2011-01-21 10:33:23 -0800 |
|---|---|---|
| committer | John Reck <jreck@google.com> | 2011-01-21 10:33:23 -0800 |
| commit | 956f1f2da9635878cdaeda3794ff1b41ed4bdcf9 (patch) | |
| tree | b4a6c39d7a4aedb275efd047388b390e1f0fcf70 /src/com/android/browser | |
| parent | ee804cd408ebbaa75b8832784934531520e455e8 (diff) | |
| download | packages_apps_Browser-956f1f2da9635878cdaeda3794ff1b41ed4bdcf9.tar.gz packages_apps_Browser-956f1f2da9635878cdaeda3794ff1b41ed4bdcf9.tar.bz2 packages_apps_Browser-956f1f2da9635878cdaeda3794ff1b41ed4bdcf9.zip | |
Fix "Most visited" history group flicker
Bug: 3374382
Change-Id: Idedf9804fa2010c91ac163965fb1d7a0617d438e
Diffstat (limited to 'src/com/android/browser')
| -rw-r--r-- | src/com/android/browser/BrowserHistoryPage.java | 8 |
1 files changed, 6 insertions, 2 deletions
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; |
