tree 03e5d2f1b870b7f6143d2e9d4e60bd9633eadcbc
parent 29b637301dfbafe7f84c0c6166f2b5c4dd186697
author stuartmorgan <stuartmorgan@google.com> 1639942860 -0500
committer GitHub <noreply@github.com> 1639942860 -0800
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsBcBAABCAAQBQJhv4rMCRBK7hj4Ov3rIwAAudEIAIOtvJeW4qHknfdZqYqliilN
 S+PodGiBVhvRQJAViI2ROzspGBCre0Ptg40Fc1CzKJcFXJ8DBKySdOYIyAHYl4Wd
 P8PUagbi7WWt3bT17zSRc8YbABXLV9Vj0FvPQ0LYDr9u1zT8dbVpfKGa2CH7LYLt
 APdWFcApAYSN+E5M9VR347G+XWmPLSViVs1DN96JoEmp/RqMIDmFcuetjpqbbnaW
 Zq+UXCWS7kxCgx2VYE2riADmrTfptAoDm3oRyDBNmagrfa/R+f79tLkRn64lLTNB
 24GXM1nqkp3txZleCJcJYaD9C5V5fKi0QRudd8f7yKcKZVi1peKfPubPUXSS5Ns=
 =lm1w
 -----END PGP SIGNATURE-----
 

[video_player] Remove test code from platform interface (#4589)

This is a breaking change to video_player_platform_interface to remove Pigeon mocks from the public interface. These should never have been there, as tests in packages outside of the platform interface should be mocking the platform interface rather than the method channel, but the app-facing tests weren't rewritten when they should have been so they ended up published to support the legacy tests. This creates an unwanted non-dev dependency on `flutter_test`, in addition to promoting an anti-pattern.

While making the breaking change, this also adopts `PlatformInterface`, removing `isMock` in favor of the standard mixin pattern provided by the base class.

Part of https://github.com/flutter/flutter/issues/83562