Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • gnome-shell gnome-shell
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 2,040
    • Issues 2,040
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 150
    • Merge requests 150
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • GNOMEGNOME
  • gnome-shellgnome-shell
  • Merge requests
  • !211

Fix slow visual feedback on screenshot service

  • Review changes

  • Download
  • Email patches
  • Plain diff
Closed Brion Vibber (WMF) requested to merge bvibber/gnome-shell:fast-flash into master Aug 31, 2018
  • Overview 55
  • Commits 3
  • Pipelines 12
  • Changes 3

Fix slow audiovisual feedback on screenshot service

Adds a "screenshot-taken" signal from the screenshot service's internal C implementation to the JS side, and uses that to trigger the camera flash visual effect and the click, allowing them to run in parallel with the PNG compression instead of waiting until the file is complete to start.

This significantly improves perceived latency on high res setups such as 4K, 5K, or dual 4K screens.

See #512 (closed)

For dual 4K I measured save times in excess of 1.25s on old first-generation Core (Nehalem) 2.26 GHz workstation, and expect about 0.5s on more modern high end workstations, still noticeable for keyboard event feedback.

Unredirect disabling is also reduced from the entire time of grab and compression to just during the screen grab, improving performance of 3d games during a screenshot.

Companion patches for gnome-settings-daemon and gnome-screenshot are required to avoid double sound effects:

gnome-settings-daemon!44 (closed) gnome-screenshot!3 (closed)

Note this moves some strings from gnome-settings-daemon.

Edited Feb 09, 2021 by Jonas Dreßler
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: fast-flash