Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(7314)

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/download/ui/DownloadManagerToolbar.java

Issue 2266543003: [Download Home] Use OfflinePage titles & bolster tests (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@titles
Patch Set: setup Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/android/java/src/org/chromium/chrome/browser/download/ui/DownloadManagerToolbar.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/download/ui/DownloadManagerToolbar.java b/chrome/android/java/src/org/chromium/chrome/browser/download/ui/DownloadManagerToolbar.java
index bc6bfdc249774cd779af9d57fabfb92628585d31..bf62f9a7db7e1935e4fe0b7e9cce933db5175ae5 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/download/ui/DownloadManagerToolbar.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/download/ui/DownloadManagerToolbar.java
@@ -5,10 +5,13 @@
package org.chromium.chrome.browser.download.ui;
import android.content.Context;
+import android.support.annotation.Nullable;
+import android.support.v4.widget.DrawerLayout;
import android.util.AttributeSet;
import org.chromium.chrome.R;
import org.chromium.chrome.browser.download.ui.DownloadManagerUi.DownloadUiObserver;
+import org.chromium.chrome.browser.widget.selection.SelectionDelegate;
import org.chromium.chrome.browser.widget.selection.SelectionToolbar;
import org.chromium.ui.base.DeviceFormFactor;
@@ -27,11 +30,13 @@ public class DownloadManagerToolbar extends SelectionToolbar<DownloadHistoryItem
}
@Override
- public void initialize(DownloadManagerUi manager) {
- manager.addObserver(this);
+ public void initialize(SelectionDelegate<DownloadHistoryItemWrapper> delegate, int titleResId,
+ @Nullable DrawerLayout drawerLayout, int normalGroupResId, int selectedGroupResId) {
if (DeviceFormFactor.isTablet(getContext())) {
getMenu().findItem(R.id.close_menu_id).setVisible(false);
}
+
+ super.initialize(delegate, titleResId, drawerLayout, normalGroupResId, selectedGroupResId);
}
@Override
@@ -47,8 +52,7 @@ public class DownloadManagerToolbar extends SelectionToolbar<DownloadHistoryItem
}
@Override
- public void onManagerDestroyed(DownloadManagerUi manager) {
- }
+ public void onManagerDestroyed() { }
private void updateTitle() {
if (mFilter == DownloadFilter.FILTER_ALL) {

Powered by Google App Engine
This is Rietveld 408576698