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

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

Issue 2341643008: Defaulting all downloads to go through Chrome network stack (Closed)
Patch Set: remove extern Created 4 years, 3 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/DownloadManagerService.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/download/DownloadManagerService.java b/chrome/android/java/src/org/chromium/chrome/browser/download/DownloadManagerService.java
index e9c2b80eb17b0a7d724a2805166e74e33c546fd2..6a8a495cc80a04c09829c709b446a775d56f138d 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/download/DownloadManagerService.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/download/DownloadManagerService.java
@@ -34,7 +34,6 @@ import org.chromium.base.annotations.SuppressFBWarnings;
import org.chromium.base.metrics.RecordHistogram;
import org.chromium.chrome.R;
import org.chromium.chrome.browser.ChromeActivity;
-import org.chromium.chrome.browser.ChromeFeatureList;
import org.chromium.chrome.browser.download.ui.BackendProvider;
import org.chromium.chrome.browser.download.ui.DownloadHistoryAdapter;
import org.chromium.chrome.browser.externalnav.ExternalNavigationDelegateImpl;
@@ -1063,17 +1062,15 @@ public class DownloadManagerService extends BroadcastReceiver implements
* @param context Application context
*/
protected static void openDownloadsPage(Context context) {
- if (!ChromeFeatureList.isEnabled(ChromeFeatureList.SYSTEM_DOWNLOAD_MANAGER)) {
- // Try to open Download Home.
- Activity lastActivity = ApplicationStatus.getLastTrackedFocusedActivity();
- if (lastActivity instanceof ChromeActivity) {
- int state = ApplicationStatus.getStateForActivity(lastActivity);
- if (state >= ActivityState.CREATED && state <= ActivityState.RESUMED) {
- ChromeActivity chromeActivity = (ChromeActivity) lastActivity;
- DownloadUtils.showDownloadManager(
- lastActivity, chromeActivity.getActivityTab());
- return;
- }
+ // Try to open Download Home.
+ Activity lastActivity = ApplicationStatus.getLastTrackedFocusedActivity();
+ if (lastActivity instanceof ChromeActivity) {
+ int state = ApplicationStatus.getStateForActivity(lastActivity);
+ if (state >= ActivityState.CREATED && state <= ActivityState.RESUMED) {
+ ChromeActivity chromeActivity = (ChromeActivity) lastActivity;
+ DownloadUtils.showDownloadManager(
+ lastActivity, chromeActivity.getActivityTab());
+ return;
}
}

Powered by Google App Engine
This is Rietveld 408576698