Uploaded image for project: 'Qt'
  1. Qt
  2. QTBUG-128100

Screensharing on wayland triggers multiple xdg portal dialogs

    XMLWordPrintable

Details

    • Bug
    • Resolution: Unresolved
    • P2: Important
    • None
    • 6.7, 6.8, 6.9
    • WebEngine
    • None
    • Linux/Wayland

    Description

      navigator.mediaDevices.getDisplayMedia triggers two portal dialogs on wayland one for screen selection and one for windows.
      This makes the DesktopMediaRequest api annoying to use.
      Users have to cancel these dialogs multiple times before using the actual qt specific api/dialog.
      It can be reproduced with the webrtc manual test and simplebrowser.

      Chromium shows/hides these portal dialogs based on the state of their own dialog's.
      Firefox triggers only one portal dialog that shows both screens and windows and does not show its own dialog.

      Proposed solutions:
       1. extend the DesktopMediaRequest api with show/hidePortalDialog so users can implement the chrome stlye dialog. It is easy to implement but can be complicated to use.
       2. do everything behind the scenes and provide a "generic" portal dialog same as firefox. It is harder to implement because we would need direct access to webrtc methods and currently the MediaRequest api is based on the DesktopMediaList structures which don't have this functionality.

      Attachments

        1. chrome.png
          chrome.png
          100 kB
        2. firefox.png
          firefox.png
          99 kB
        3. qtwebengine.png
          qtwebengine.png
          69 kB

        Issue Links

          For Gerrit Dashboard: QTBUG-128100
          # Subject Branch Project Status CR V

          Activity

            People

              mnegyokru Martin Negyokru
              mnegyokru Martin Negyokru
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:

                Gerrit Reviews

                  There are 2 open Gerrit changes