Exoplayer cannot be constructed with a specification for "surface_type" programmatically? Why not?

[jonmdev]

Version

Media3 main branch

More version details

As noted here:

• How to construct ExoPlayer as TextureView rather than SurfaceView in C# .NET? #1360
• [BUG] MediaElement does not respect clipping of Views that contain it in Android, so videos show where they should not (works properly in Windows/iOS) CommunityToolkit/Maui#1730
• [BUG] ExoPlayer uses SurfaceView by default, needs way to optionally use TextureView to fix glitchy behavior and extend usage to transparent background CommunityToolkit/Maui#1773
• https://stackoverflow.com/questions/78446498/how-to-create-textureview-programmatically-with-exoplayer-in-net-maui
• https://stackoverflow.com/questions/78457606/how-to-create-iattributeset-from-xml-string-using-xmlreader-in-net-android

There exists no way to create an ExoPlayer programmatically while declaring the surface type in the constructor in C#/.NET. The only proposed way to do this in Android is explained here but even this is extremely clumsy and illogical.

No equivalent to this Android code exists in C#/.NET. No one is able to propose a method to make this possible in C#/.NET.

It seems obvious then the simplest solution is to simply add a constructor (which should exist either way) where we can declare the surface type on construction.

For example, we could have a simple enum:

enum ExoPlayerSurfaceType {
    surfaceView,
    textureView,
    none
}

We have the currently unhelpful constructor:

public unsafe StyledPlayerView (global::Android.Content.Context? context, global::Android.Util.IAttributeSet? attrs)

We could add the following option:

public unsafe StyledPlayerView (global::Android.Content.Context context, ExoPlayerSurfaceType surfaceType)

Or we could have an entire new class for ExoPlayerConstructorAttributes which we could create an object of and set the surface type inside (along with any other "only available on construction" parameters if they exist).

Why is this not the case already? If you agree this is a logical solution, how can we add it?

Thanks.

Devices that reproduce the issue

All ExoPlayer developments

Devices that do not reproduce the issue

n/a

Reproducible in the demo app?

Yes

Reproduction steps

See demo code and the problems this issue is causing in links above, or again, at:

• How to construct ExoPlayer as TextureView rather than SurfaceView in C# .NET? #1360
• [BUG] MediaElement does not respect clipping of Views that contain it in Android, so videos show where they should not (works properly in Windows/iOS) CommunityToolkit/Maui#1730
• [BUG] ExoPlayer uses SurfaceView by default, needs way to optionally use TextureView to fix glitchy behavior and extend usage to transparent background CommunityToolkit/Maui#1773
• https://stackoverflow.com/questions/78446498/how-to-create-textureview-programmatically-with-exoplayer-in-net-maui
• https://stackoverflow.com/questions/78457606/how-to-create-iattributeset-from-xml-string-using-xmlreader-in-net-android

Expected result

One should be able to easily create an ExoPlayer with any type of surface (TextureView, SurfaceView, none) easily from a simple constructor in Android or C#/.NET.

Actual result

It is currently very circuitous to do this in native Android code and completely impossible in C#/.NET. No solutions or answers are available anywhere.

Media

Open Visual Studio 2022. Go File > New > Android Application. Copy and paste the code here into MainActivity.cs:

protected override void OnCreate(Bundle? savedInstanceState) {
    base.OnCreate(savedInstanceState);
    
    string xmlString =
        "<?xml version=\"1.0\" encoding=\"utf-8\"?>\n" +
        "<Com.Google.Android.Exoplayer2.UI.PlayerView " +           
        "android:id=\"@+id/player_view\" " +
        "app:surface_type=\"texture_view\" " + //KEY NEEDED LINE FOR GOAL
        "android:layout_width=\"match_parent\" " +
        "android:layout_height=\"match_parent\"/>";
    
    XmlReader xmlReader = XmlReader.Create(new StringReader(xmlString));
    xmlReader.Read();
    System.Diagnostics.Debug.WriteLine("XML READER " + xmlReader.AttributeCount);
            
    Android.Util.IAttributeSet attributes = Android.Util.Xml.AsAttributeSet(xmlReader);
    System.Diagnostics.Debug.WriteLine("ATTRIBUTES " + attributes.AttributeCount);
    
    Com.Google.Android.Exoplayer2.UI.StyledPlayerView styledPlayerView = new(this, attributes);            

Receive errors as there is no current working workflow available to do this.

[andrewlewis]

Providing support for usage from C# is not really in scope for this tracker, but have you tried storing a layout file as a resource like what's described here, instead of trying to construct and read the XML file programmatically?

Is this issue a duplicate of #1360?

[jonmdev]

I have just realized the only Kotlin method to do this also does not programmatically create the XML but rather just loads from resources. It seems Android offers no way to allow one to programmatically do this at all.

One must save the XML to resources and then load it.

.NET "SOLUTION"

1. Create XML file named "test.xml" in Resources\layout
2. Copy paste into it:

<?xml version="1.0" encoding="utf-8" ?> 
<com.google.android.exoplayer2.ui.StyledPlayerView 
    xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:app="http://schemas.android.com/apk/res-auto"
    android:id="@+id/test"
    app:surface_type="texture_view"
    android:layout_width= "match_parent"
    android:layout_height= "match_parent" />

3. Run code in MainActivity.cs:

XmlReader xmlResource = this.Resources.GetXml(Resource.Layout.test);
xmlResource.Read();

Android.Util.IAttributeSet attributes = Android.Util.Xml.AsAttributeSet(xmlResource);

//BUILD EXOPLAYER
Com.Google.Android.Exoplayer2.UI.StyledPlayerView styledPlayerView = new(this, attributes); 

INTENDED DESIGN?

Is this the intended design of Android? Are we supposed to save tiny little snippets of silly XML in our Resources folder and then load them like this for usage when we need to construct something?

Why? Why not programmatically create the XML (if XML must be used as an intermediate) on the fly in the code?

[andrewlewis]

I think setting attributes in the XML file is a conventional way to specify properties for a custom view -- see Define custom attributes. There are a bunch of these for PlayerView defined around here.

As documented here the intended way to use this is to set them in your layout XML file.

After adding a .NET discussion (linked above) which has also been unhelpful, and reviewing further, I again believe this is an Android issue not a .NET issue.

Based on your reply just now, did you get this to work in the end? If not, please could you describe what didn't work in case that's useful to someone else in future?