VideoPlayerWebOptions is not working

[TheCGDF]

Steps to reproduce

1. make a VideoPlayerController with webOptions.

VideoPlayerController.asset("assets/xxx.mp4",
        videoPlayerOptions: VideoPlayerOptions(
            webOptions: const VideoPlayerWebOptions(
                allowContextMenu: false, allowRemotePlayback: false)));

2. We should have such options on web now:

allowDownload: false,
allowFullscreen: false,
allowPlaybackRate: false,
allowPictureInPicture: false
allowContextMenu: false,
allowRemotePlayback: false

Expected results

Cannot download/fullscreen/... videos

Actual results

Nothing works. Videos can still be downloaded/...

Code sample

Code sample

import 'package:flutter/cupertino.dart';
import 'package:video_player/video_player.dart';
import 'package:video_player_platform_interface/video_player_platform_interface.dart';

class VideoBody extends StatefulWidget {
  @override
  State<StatefulWidget> createState() => VideoBodyState();
}

class VideoBodyState extends State<VideoBody> {
  late VideoPlayerController _backgroundPlayer;

  @override
  void initState() {
    _backgroundPlayer = VideoPlayerController.asset("assets/xxx.mp4",
        videoPlayerOptions: VideoPlayerOptions(
            webOptions: const VideoPlayerWebOptions(
                allowContextMenu: false,
                allowRemotePlayback: false)));
    _backgroundPlayer.setVolume(0);
    _backgroundPlayer.initialize().then((_) => _backgroundPlayer.play());
    super.initState();
  }

  @override
  Widget build(BuildContext context) => VideoPlayer(_backgroundPlayer);
}

Screenshots or Video

Screenshots / Video demonstration

Logs

Logs

No error.

Flutter Doctor output

Doctor output

[√] Flutter (Channel stable, 3.22.2, on Microsoft Windows [Version 10.0.22631.3737], locale en-US)
    • Flutter version 3.22.2 on channel stable at D:\Software\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 761747bfc5 (11 days ago), 2024-06-05 22:15:13 +0200
    • Engine revision edd8546116
    • Dart version 3.4.3
    • DevTools version 2.34.3

[√] Windows Version (Installed version of Windows is version 10 or higher)

[√] Android toolchain - develop for Android devices (Android SDK version 35.0.0-rc3)
    • Android SDK at D:\Software\AndroidSDK
    • Platform android-34, build-tools 35.0.0-rc3
    • Java binary at: D:\Software\Android Studio\jbr\bin\java
    • Java version OpenJDK Runtime Environment (build 17.0.10+0--11609105)
    • All Android licenses accepted.

[X] Chrome - develop for the web (Cannot find Chrome executable at .\Google\Chrome\Application\chrome.exe)
    ! Cannot find Chrome. Try setting CHROME_EXECUTABLE to a Chrome executable.

[√] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.10.0)
    • Visual Studio at D:\Software\Microsoft Visual Studio
    • Visual Studio Community 2022 version 17.10.34916.146
    • Windows 10 SDK version 10.0.22621.0

[√] Android Studio (version 2024.1)
    • Android Studio at D:\Software\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.10+0--11609105)

[√] Connected device (2 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.22631.3737]
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 125.0.2535.92

[√] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

[huycozy]

Thanks for the report. I can reproduce this issue as well. Current package versions: video_player: ^2.8.7, video_player_web: ^2.3.1.

flutter doctor -v

[✓] Flutter (Channel stable, 3.22.2, on macOS 14.1 23B74 darwin-x64, locale en-VN)
    • Flutter version 3.22.2 on channel stable at /Users/huynq/Documents/GitHub/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 761747bfc5 (31 hours ago), 2024-06-05 22:15:13 +0200
    • Engine revision edd8546116
    • Dart version 3.4.3
    • DevTools version 2.34.3

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/huynq/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/huynq/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.10+0-17.0.10b1087.21-11572160)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.3)
    • Xcode at /Applications/Xcode15.3.app/Contents/Developer
    • Build 15E204a
    • CocoaPods version 1.15.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2023.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • android-studio-dir = /Applications/Android Studio.app/
    • Java version OpenJDK Runtime Environment (build 17.0.10+0-17.0.10b1087.21-11572160)

[✓] VS Code (version 1.89.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.90.0

[✓] Connected device (3 available)
    • iPhone (mobile) • d9a94afe2b649fef56ba0bfeb052f0f2a7dae95e • ios            • iOS 15.8 19H370
    • macOS (desktop) • macos                                    • darwin-x64     • macOS 14.1 23B74 darwin-x64
    • Chrome (web)    • chrome                                   • web-javascript • Google Chrome 125.0.6422.142

[✓] Network resources
    • All expected network resources are available.

• No issues found!

[stuartmorgan-g]

The issue here is that flutter/packages#3259 never landed, but we landed the platform interface piece that was split out, so we have options that are defined, but not honored by the implementation package.

(This isn't supposed to happen because we're supposed to have full approval on the main PR first, and also in our more current designs we wouldn't have this options class in the platform interface package at all.)

Reverting the platform interface would be a breaking change. @ditman Any chance you could take over flutter/packages#3259 to get it over the line? If not, we should at least deprecate the options and document that they don't work.

[ditman]

Any chance you could take over flutter/packages#3259 to get it over the line? If not, we should at least deprecate the options and document that they don't work.

@stuartmorgan yes, let me assign the PR to myself; I'll try to land it next time I'm gardening the repo! (the week of June 24th)