From ff0f1c4e9c41ae9c0e087988f1e5d82cbe4c1324 Mon Sep 17 00:00:00 2001
From: ibaker <ibaker@google.com>
Date: Tue, 23 May 2023 11:57:43 +0100
Subject: [PATCH] Remove deprecated `DownloadManager` constructor

Use the constructor that takes an `Executor` instead.

#minor-release

PiperOrigin-RevId: 534370613
---
 RELEASENOTES.md                                 |  5 +++++
 .../exoplayer/offline/DownloadManager.java      | 17 -----------------
 2 files changed, 5 insertions(+), 17 deletions(-)

diff --git a/RELEASENOTES.md b/RELEASENOTES.md
index 34c0606a5b..edc0633d24 100644
--- a/RELEASENOTES.md
+++ b/RELEASENOTES.md
@@ -89,6 +89,11 @@
     *   Remove deprecated `ExoPlayer.retry()`, use `prepare()` instead.
     *   Remove deprecated zero-arg `DefaultTrackSelector` constructor, use
         `DefaultTrackSelector(Context)` instead.
+    *   Remove deprecated `OfflineLicenseHelper` constructor, use
+        `OfflineLicenseHelper(DefaultDrmSessionManager,
+        DrmSessionEventListener.EventDispatcher)` instead.
+    *   Remove deprecated `DownloadManager` constructor, use the constructor
+        that takes an `Executor` instead.
     *   Remove `ExoPlayer.setHandleWakeLock(boolean)`, use `setWakeMode(int)`
         instead.
     *   Remove deprecated
diff --git a/libraries/exoplayer/src/main/java/androidx/media3/exoplayer/offline/DownloadManager.java b/libraries/exoplayer/src/main/java/androidx/media3/exoplayer/offline/DownloadManager.java
index 74f36f804e..2284b936fa 100644
--- a/libraries/exoplayer/src/main/java/androidx/media3/exoplayer/offline/DownloadManager.java
+++ b/libraries/exoplayer/src/main/java/androidx/media3/exoplayer/offline/DownloadManager.java
@@ -193,23 +193,6 @@ public final class DownloadManager {
   private List<Download> downloads;
   private RequirementsWatcher requirementsWatcher;
 
-  /**
-   * Constructs a {@link DownloadManager}.
-   *
-   * @param context Any context.
-   * @param databaseProvider Provides the SQLite database in which downloads are persisted.
-   * @param cache A cache to be used to store downloaded data. The cache should be configured with
-   *     an {@link CacheEvictor} that will not evict downloaded content, for example {@link
-   *     NoOpCacheEvictor}.
-   * @param upstreamFactory A {@link Factory} for creating {@link DataSource}s for downloading data.
-   * @deprecated Use {@link #DownloadManager(Context, DatabaseProvider, Cache, Factory, Executor)}.
-   */
-  @Deprecated
-  public DownloadManager(
-      Context context, DatabaseProvider databaseProvider, Cache cache, Factory upstreamFactory) {
-    this(context, databaseProvider, cache, upstreamFactory, Runnable::run);
-  }
-
   /**
    * Constructs a {@link DownloadManager}.
    *