Use better names

Author: crazytonyli

WordPress/Classes/ViewRelated/Aztec/Media/MediaProgressCoordinator.swift

@@ -34,8 +34,6 @@ public class MediaProgressCoordinator: NSObject {
 
     private static var mediaProgressObserverContext = 0
 
-    private lazy var bgTaskTracker: MediaUploadBackgroundTracker? = mediaUploadBackgroundTracker()
-
     deinit {
         mediaGlobalProgress?.removeObserver(self, forKeyPath: #keyPath(Progress.fractionCompleted))
     }
@@ -77,7 +75,7 @@ public class MediaProgressCoordinator: NSObject {
 
         let objectID = TaggedManagedObjectID(media)
         Task {
-            await bgTaskTracker?.track(progress: progress, media: objectID)
+            await mediaUploadBackgroundTaskScheduler()?.scheduleTask(for: objectID, progress: progress)
         }
     }
 

…Media/MediaUploadBackgroundTracker.swift …MediaUploadBackgroundTaskScheduler.swiftWordPress/Classes/ViewRelated/Aztec/Media/MediaUploadBackgroundTracker.swift renamed to WordPress/Classes/ViewRelated/Aztec/Media/MediaUploadBackgroundTaskScheduler.swift WordPress/Classes/ViewRelated/Aztec/Media/MediaUploadBackgroundTracker.swift renamed to WordPress/Classes/ViewRelated/Aztec/Media/MediaUploadBackgroundTaskScheduler.swift

@@ -4,23 +4,23 @@ import Combine
 import WordPressShared
 
 // This protocol is used to hide the `@available(iOS 26.0, *)` check.
-protocol MediaUploadBackgroundTracker {
+protocol MediaUploadBackgroundTaskScheduler {
 
-    func track(progress: Progress, media: TaggedManagedObjectID<Media>) async
+    func scheduleTask(for media: TaggedManagedObjectID<Media>, progress: Progress) async
 
 }
 
-func mediaUploadBackgroundTracker() -> MediaUploadBackgroundTracker? {
+func mediaUploadBackgroundTaskScheduler() -> MediaUploadBackgroundTaskScheduler? {
     if #available(iOS 26.0, *) {
-        ConcreteMediaUploadBackgroundTracker.shared
+        ConcreteMediaUploadBackgroundTaskScheduler.shared
     } else {
         nil
     }
 }
 
 @available(iOS 26.0, *)
 /// Utilize `BGContinuedProcessingTask` to show the uploading media activity.
-private actor ConcreteMediaUploadBackgroundTracker: MediaUploadBackgroundTracker {
+private actor ConcreteMediaUploadBackgroundTaskScheduler: MediaUploadBackgroundTaskScheduler {
     struct Item {
         // Please note: all media query needs to be done in the main context, due to the current upload media implementation.
         var media: TaggedManagedObjectID<Media>
@@ -47,7 +47,7 @@ private actor ConcreteMediaUploadBackgroundTracker: MediaUploadBackgroundTracker
     }
 
     // Since this type works with `BGTaskScheduler.shared`, it only makes sense for the type to also be a singleton.
-    static let shared = ConcreteMediaUploadBackgroundTracker()
+    static let shared = ConcreteMediaUploadBackgroundTaskScheduler()
 
     // We only use one `BGContinuedProcessingTask` for all uploads. When adding new media during uploading, the new ones
     // will be added to the existing task.
@@ -80,7 +80,7 @@ private actor ConcreteMediaUploadBackgroundTracker: MediaUploadBackgroundTracker
 
     }
 
-    func track(progress: Progress, media: TaggedManagedObjectID<Media>) async {
+    func scheduleTask(for media: TaggedManagedObjectID<Media>, progress: Progress) async {
         observeCoreDataChanges()
 
         let item = Item(media: media, progress: progress)