Release notes and formatting

Author: tonihei

RELEASENOTES.md

@@ -201,6 +201,9 @@
     *   Bump IMA dependency to 3.37.0 which requires enabling core library
         desugaring. This must also be enabled by dependent apps too. See IMA's
         [config notes](https://developers.google.com/interactive-media-ads/docs/sdks/android/client-side/get-started#2.-add-the-ima-sdk-to-your-project).
+    *   Fix issue where content preparation error for content after an ad would
+        be wrongly reported as an ad playback error
+        ([#2656](https://github.com/androidx/media/issues/2656)).
 *   Session:
     *   Add new parameter to `MediaSession.Callback.onPlaybackResumption` to
         indicate if the call happens to gather information only or to start

libraries/exoplayer_ima/src/main/java/androidx/media3/exoplayer/ima/AdTagLoader.java

@@ -391,18 +391,15 @@ public void activate(Player player) {
   /** Deactivates playback. */
   public void deactivate() {
     Player player = checkNotNull(this.player);
-    // Post release of listener behind any already queued Player.Listener events to ensure that
-    // any pending events are processed before the player is deferred.
-    handler.post(
-        () -> {
-          deactivateInternal(player);
-        });
+    // Post deactivation behind any already queued Player.Listener events to ensure that
+    // any pending events are processed before the listener is removed and the ads manager paused.
+    handler.post(() -> deactivateInternal(player));
   }
 
   /**
    * Deactivates playback internally, after the Listener.onEvents() cycle completes so the complete
-   * state change picture is clear. For example, if an error caused the deactivation, need to
-   * determine whether the error was for an ad or content.
+   * state change picture is clear. For example, if an error caused the deactivation, the error
+   * callback can be handled first.
    */
   private void deactivateInternal(Player player) {
     if (!AdPlaybackState.NONE.equals(adPlaybackState)