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

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

Issue 2271583006: [Android] Implement back key functionality in download manager (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@loading
Patch Set: fix test 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/DownloadManagerUi.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/download/ui/DownloadManagerUi.java b/chrome/android/java/src/org/chromium/chrome/browser/download/ui/DownloadManagerUi.java
index 7527007781ac621de15eda03326569125214ebe6..18298010c23f5f939816a8802e8e9eb06364f170 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/download/ui/DownloadManagerUi.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/download/ui/DownloadManagerUi.java
@@ -279,6 +279,21 @@ public class DownloadManagerUi implements OnMenuItemClickListener {
}
/**
+ * Adds a {@link DownloadUiObserver} to observe the changes in the download manager.
+ */
+ public void addObserver(DownloadUiObserver observer) {
+ mObservers.addObserver(observer);
+ }
+
+ /**
+ * Removes a {@link DownloadUiObserver} that were added in
+ * {@link #addObserver(DownloadUiObserver)}
+ */
+ public void removeObserver(DownloadUiObserver observer) {
+ mObservers.removeObserver(observer);
+ }
+
+ /**
* @see DrawerLayout#closeDrawer(int)
*/
void closeDrawer() {
@@ -317,21 +332,6 @@ public class DownloadManagerUi implements OnMenuItemClickListener {
DownloadFilter.FILTER_BOUNDARY);
}
- /**
- * Adds a {@link DownloadUiObserver} to observe the changes in the download manager.
- */
- void addObserver(DownloadUiObserver observer) {
- mObservers.addObserver(observer);
- }
-
- /**
- * Removes a {@link DownloadUiObserver} that were added in
- * {@link #addObserver(DownloadUiObserver)}
- */
- void removeObserver(DownloadUiObserver observer) {
- mObservers.removeObserver(observer);
- }
-
private void shareSelectedItems() {
List<DownloadHistoryItemWrapper> selectedItems =
mBackendProvider.getSelectionDelegate().getSelectedItems();

Powered by Google App Engine
This is Rietveld 408576698