From cb8f87e05e73d72ce7d2c405c4c5899dc31ade5e Mon Sep 17 00:00:00 2001
From: ibaker <ibaker@google.com>
Date: Thu, 20 Jun 2024 04:03:10 -0700
Subject: [PATCH] Remove direct `AspectRatioFrameLayout` usage from session
 demo

This class is a lower-level UI component that isn't directly needed if
apps are using `PlayerView` to handle their video output (it is used as
an implementation detail of `PlayerView`).

Removing its unecessary usages from this demo avoids developers copying
this as an example when building their own apps.

#cherrypick

PiperOrigin-RevId: 644972454
---
 .../src/main/res/layout/activity_player.xml   | 23 ++++++++-----------
 1 file changed, 9 insertions(+), 14 deletions(-)

diff --git a/demos/session/src/main/res/layout/activity_player.xml b/demos/session/src/main/res/layout/activity_player.xml
index a638d8aca6..9254d8b761 100644
--- a/demos/session/src/main/res/layout/activity_player.xml
+++ b/demos/session/src/main/res/layout/activity_player.xml
@@ -22,21 +22,16 @@
   android:background="@color/player_background"
   tools:context=".PlayerActivity">
 
-  <androidx.media3.ui.AspectRatioFrameLayout
-    android:layout_height="300dp"
+  <androidx.media3.ui.PlayerView
+    android:id="@+id/player_view"
+    android:background="@color/player_background"
     android:layout_width="match_parent"
-    >
-    <androidx.media3.ui.PlayerView
-      android:id="@+id/player_view"
-      android:background="@color/player_background"
-      android:layout_width="match_parent"
-      android:layout_height="match_parent"
-      app:artwork_display_mode="fill"
-      app:default_artwork="@drawable/artwork_placeholder"
-      app:repeat_toggle_modes="one|all"
-      app:show_shuffle_button="true"
-      app:shutter_background_color="@color/player_background" />
-  </androidx.media3.ui.AspectRatioFrameLayout>
+    android:layout_height="300dp"
+    app:artwork_display_mode="fill"
+    app:default_artwork="@drawable/artwork_placeholder"
+    app:repeat_toggle_modes="one|all"
+    app:show_shuffle_button="true"
+    app:shutter_background_color="@color/player_background" />
 
   <TextView
     android:id="@+id/media_artist"