Chromium Code Reviews| Index: chrome/android/java/src/org/chromium/chrome/browser/download/ui/DownloadHistoryAdapter.java |
| diff --git a/chrome/android/java/src/org/chromium/chrome/browser/download/ui/DownloadHistoryAdapter.java b/chrome/android/java/src/org/chromium/chrome/browser/download/ui/DownloadHistoryAdapter.java |
| index ffd9d6e79b86b559e22d447801a4e1fed71e8ff7..059ee7525dbb375e0cec68decfe41212c97e5e3e 100644 |
| --- a/chrome/android/java/src/org/chromium/chrome/browser/download/ui/DownloadHistoryAdapter.java |
| +++ b/chrome/android/java/src/org/chromium/chrome/browser/download/ui/DownloadHistoryAdapter.java |
| @@ -161,6 +161,7 @@ public class DownloadHistoryAdapter extends DateDividedAdapter |
| private String mSearchQuery = EMPTY_QUERY; |
| private SpaceDisplay mSpaceDisplay; |
| private boolean mIsSearching; |
| + private boolean mShowSpaceDisplayHeader; |
|
Theresa
2017/05/09 23:32:23
nit: mShowingSpaceDisplayHeader or mShouldShowSpac
shaktisahu
2017/05/11 19:55:47
Done.
|
| @Nullable // This may be null during tests. |
| private UiConfig mUiConfig; |
| @@ -531,6 +532,16 @@ public class DownloadHistoryAdapter extends DateDividedAdapter |
| filter(mFilter); |
| } |
| + /** |
| + * Toggles the visibility of the space usage info header. |
| + * @return Whether or not we are showing the header after the toggle. |
| + */ |
| + boolean toggleSpaceDisplayHeader() { |
| + mShowSpaceDisplayHeader = !mShowSpaceDisplayHeader; |
| + filter(mFilter); |
| + return mShowSpaceDisplayHeader; |
| + } |
| + |
| private DownloadDelegate getDownloadDelegate() { |
| return mBackendProvider.getDownloadDelegate(); |
| } |
| @@ -560,7 +571,7 @@ public class DownloadHistoryAdapter extends DateDividedAdapter |
| } |
| clear(false); |
| - if (!filteredTimedItems.isEmpty() && !mIsSearching) addHeader(); |
| + if (!filteredTimedItems.isEmpty() && !mIsSearching && mShowSpaceDisplayHeader) addHeader(); |
| loadItems(filteredTimedItems); |
| } |