Video player state interface

[duongphuhiep]

Motivation

Allow consumer to mock the VideoPlayer in all kind of situations (for Previewing or testing). For example

• Preview (or instrumented test) when the video is Buffering -> assert that the slider turns gray
• Preview (or instrumented test) when the currentTime > 1min -> assert that the Ads modal pop up, slider turns blue..
• Preview (or instrumented test) when state contains an error -> assert that the error is displayed in the right place
• etc..

Solution

Turn VideoPlayerState to interface, consumer provides their own (mock) Player State for previewing (or testing).

Breaking changes

VideoPlayerState() becomes createVideoPlayerState()

Note: this breaking change won't impact most consumers, because they usually use rememberVideoPlayerState() to instantiate a new VideoPlayerState

[duongphuhiep]

I amended my "rush" commit.
Now we can have this

[kdroidFilter]

Thank you so much, I really like it, it’s super convenient!
I’d prefer to merge it into master instead. Would you be able to resolve the conflicts? I just finished my PR to re-add Compose Web support.

[duongphuhiep]

I will try,

but I won't be able to test on Mac, Windows, iOS, I only have a Linux laptop

[duongphuhiep]

Hi, I solved all the conflicts and tested on

• Android (Google TV Emulator old Android 12)
• Linux Desktop (my Ubuntu laptop)
• Web (Google Chrome)

For other platform Win, Mac, iOS, I think that it compiles, but I don't have MAC or iPhone to test.