muxinc / videojs-mux-kit

MIT License
33 stars 11 forks source link

Update logic for Native vs. Hls.js engines #40

Closed erikpena closed 3 years ago

erikpena commented 3 years ago
erikpena commented 3 years ago

Using the changes in this PR, I ran through a set of manual tests against different platforms -> browsers. Here are the results—

| Platform | Browser    | Result         |
==========================================
| iOS      | Safari     | Native, played |
| iOS      | Chrome     | It played      |
| iOS      | Firefox    | It played      |
| iOS      | DuckDuckGo | It played      |
| Android  | Chrome     | It played      |
| Android  | Samsung    | It played      |
| Android  | Firefox    | It played      |
| Android  | DuckDuckGo | It played      |
| Android  | Opera      | It played      |
| Android  | Edge       | It played      |
| macOS    | Safari     | Native, played |
| macOS    | Chrome     | Hls.js, played |
| macOS    | Firefox    | Hls.js, played |
| Windows  | Edge       | Hls.js, played |
| Windows  | Chrome     | Hls.js, played |
| Windows  | Firefox    | Hls.js, played |

"It played" means that it worked but I had no DevTools to confirm what code path it took (native or hls.js)

dylanjha commented 3 years ago

closes #39