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

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

Issue 2828353002: Android: Remove GetApplicationContext part 3 (Closed)
Patch Set: Fix android webview tests Created 3 years, 8 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 90534f1920cd7b967561c2e83a81e0fd1ce98e06..5fe243de8451248152d94ddd447f5252d6e62ca3 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
@@ -263,6 +263,7 @@ public class DownloadManagerService extends BroadcastReceiver implements
@VisibleForTesting
protected DownloadManagerService(Context context, DownloadNotifier downloadNotifier,
Handler handler, long updateDelayInMillis) {
+ // TODO(wnwen): Remove mContext since it is always the application context.
mContext = context;
mSharedPrefs = ContextUtils.getAppSharedPreferences();
// Clean up unused shared prefs. TODO(qinmin): remove this after M61.
@@ -1478,8 +1479,8 @@ public class DownloadManagerService extends BroadcastReceiver implements
*/
private void addAutoResumableDownload(String guid) {
if (mAutoResumableDownloadIds.isEmpty() && !sIsNetworkListenerDisabled) {
- mNetworkChangeNotifier = new NetworkChangeNotifierAutoDetect(this, mContext,
- new RegistrationPolicyAlwaysRegister());
+ mNetworkChangeNotifier = new NetworkChangeNotifierAutoDetect(
+ this, new RegistrationPolicyAlwaysRegister());
}
if (!mAutoResumableDownloadIds.contains(guid)) {
mAutoResumableDownloadIds.add(guid);

Powered by Google App Engine
This is Rietveld 408576698