gnome-screensaver-command --poke no longer inhibits screensaver
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
GNOME Screensaver |
Won't Fix
|
High
|
|||
Me TV |
Fix Released
|
Medium
|
Unassigned | ||
VLC media player |
Unknown
|
Unknown
|
|||
X.Org X server |
Fix Released
|
Medium
|
|||
gnome-screensaver (Debian) |
Confirmed
|
Unknown
|
|||
gnome-screensaver (Ubuntu) |
Fix Released
|
Medium
|
Chris Coulson | ||
Karmic |
Won't Fix
|
Medium
|
Unassigned | ||
mplayer (Ubuntu) |
Invalid
|
Undecided
|
Unassigned | ||
Karmic |
Invalid
|
Undecided
|
Unassigned | ||
mythtv (Ubuntu) |
Won't Fix
|
Undecided
|
Unassigned | ||
Karmic |
Won't Fix
|
Undecided
|
Unassigned | ||
vlc (Ubuntu) |
Invalid
|
High
|
Unassigned | ||
Karmic |
Invalid
|
High
|
Unassigned | ||
xorg-server (Ubuntu) |
Fix Released
|
High
|
Canonical Desktop Team | ||
Karmic |
Won't Fix
|
Medium
|
Unassigned |
Bug Description
Binary package hint: vlc
In karmic , while playing videos , VLC does not inhibit screensaver.
This is a regression from the behavior in jaunty.
ProblemType: Bug
Architecture: i386
CheckboxSubmission: 417990aadff2335
CheckboxSystem: 5484a8dd99f0061
Date: Sun Sep 13 19:58:49 2009
DistroRelease: Ubuntu 9.10
Package: vlc 1.0.1-2ubuntu1
ProcEnviron:
LANGUAGE=
LANG=en_US.UTF-8
SHELL=/bin/bash
ProcVersionSign
SourcePackage: vlc
Uname: Linux 2.6.31-10-generic i686
*******
**** SRU Justification ****
*******
** Impact of bug **
The legacy inhibit API was removed from gnome-screensaver, as the gnome-session inhibit API is the preferred way to inhibit the screensaver. The legacy API required applications to "poll" the screensaver regularly to simulate user activity. The new API requires applications to register inhibitors, which will exist until they are removed, or the application exits.
xdg-screensaver is used by a lot of desktop-neutral applications including games and VLC, and this wraps the legacy gnome-screensaver API in GNOME. This means that screensaver inhibiting is currently broken in GNOME for any application which hasn't been ported to the new inhibit API.
** Description of changes **
The existing legacy API still exists in gnome-screensaver, but is currently just a stub. I've added code to this which resets the IDLETIME counter, by simulating fake keypresses using XTEST. The code is already used in gnome-session and totem, and this method of resetting IDLETIME is also used in other applications.
Resetting IDLETIME will prevent gnome-session from setting the session idle (which prevents the screensaver from activating), and will also prevent gnome-power-manager from dimming the display.
** Regression potential **
Minimal. The additional functionality is quite self-contained, and the existing interface doesn't work at all anyway. The chosen method of resetting IDLETIME is already used.
** TEST CASE **
Using the following versions of gnome-screensaver in turn, perform the steps detailed below.
PLEASE MAKE SURE that you run "killall gnome-screensaver && gnome-screensaver" after installing the new version, to ensure that you are testing the new version.
- 2.28.0-0ubuntu3.1 (karmic-security)
- 2.28.0-0ubuntu3.2 (karmic-proposed)
1) Open gnome-screensav
2) Create a script in your home folder called "gss-test.sh", and containing the following:
#!/bin/sh
while true; do
sleep 30
done
3) Run the script (sh ~/gss-test.sh), and wait
With version 2.28.0-0ubuntu3.1 of gnome-screesaver, the screensaver will activate after 1 minute.
With version 2.28.0-0ubuntu3.2, the screensaver will not activate as long as the script is still running. If you run "gnome-
TI:21:57:13 TH:0x177e890 FI:gpm-idle.c FN:gpm_
- idletime reset
TI:21:57:13 TH:0x177e890 FI:gpm-idle.c FN:gpm_
- session_idle=0, session_
TI:21:57:13 TH:0x177e890 FI:gpm-idle.c FN:gpm_
- Doing a state transition: normal
TI:21:57:13 TH:0x177e890 FI:gpm-backlight.c FN:gpm_
- we have just been idle for 20.017084s
TI:21:57:13 TH:0x177e890 FI:gpm-backlight.c FN:gpm_
- changing powersave idle status to 0
TI:21:57:13 TH:0x177e890 FI:gpm-backlight.c FN:gpm_
- no dimming hardware
TI:21:57:13 TH:0x177e890 FI:gpm-idle.c FN:gpm_
- X not idle
4) Exit the script by pressing CTRL+C in the terminal
description: | updated |
Changed in vlc (Ubuntu): | |
importance: | Undecided → High |
Changed in vlc (Ubuntu): | |
status: | Incomplete → Confirmed |
Changed in gnome-screensaver (Debian): | |
status: | Unknown → New |
Changed in gnome-screensaver (Debian): | |
status: | Unknown → Confirmed |
summary: |
- Vlc does not inhibit screensaver + gnome-screensaver --poke functionality does no longer inhibit screen + blanking |
tags: |
added: regression-release removed: regression-potential |
Changed in me-tv: | |
status: | New → Incomplete |
Changed in gnome-screensaver (Ubuntu): | |
status: | Triaged → Fix Released |
Changed in gnome-screensaver (Ubuntu): | |
status: | Fix Released → Triaged |
Changed in vlc (Ubuntu): | |
status: | Invalid → Confirmed |
Changed in vlc (Ubuntu): | |
status: | Confirmed → Invalid |
summary: |
- gnome-screensaver --poke functionality does no longer inhibit screen - blanking + gnome-screensaver-command --poke no longer inhibits screensaver |
Changed in me-tv: | |
status: | Incomplete → Invalid |
Changed in gnome-screensaver (Ubuntu Karmic): | |
assignee: | Reinhard Tartler (siretart) → nobody |
Changed in gnome-screensaver (Ubuntu): | |
assignee: | Reinhard Tartler (siretart) → nobody |
Changed in vlc: | |
status: | Incomplete → Won't Fix |
Changed in vlc (Ubuntu): | |
status: | Invalid → Confirmed |
Changed in gnome-screensaver (Ubuntu): | |
status: | Triaged → Invalid |
Changed in gnome-screensaver (Ubuntu Karmic): | |
status: | Triaged → Invalid |
Changed in vlc (Ubuntu Karmic): | |
status: | Invalid → Confirmed |
Changed in vlc (Ubuntu): | |
status: | Confirmed → Invalid |
Changed in gnome-screensaver (Ubuntu): | |
status: | Invalid → Triaged |
Changed in gnome-screensaver (Ubuntu Karmic): | |
status: | Invalid → Triaged |
Changed in vlc (Ubuntu Karmic): | |
status: | Confirmed → Invalid |
Changed in gnome-screensaver (Ubuntu Karmic): | |
assignee: | nobody → Chris Coulson (chrisccoulson) |
status: | Triaged → In Progress |
Changed in gnome-screensaver (Ubuntu): | |
assignee: | nobody → Chris Coulson (chrisccoulson) |
status: | Triaged → In Progress |
milestone: | karmic-updates → none |
Changed in gnome-screensaver (Ubuntu): | |
status: | In Progress → Fix Released |
description: | updated |
description: | updated |
Changed in gnome-screensaver (Ubuntu Karmic): | |
status: | In Progress → Triaged |
Changed in gnome-screensaver (Ubuntu Karmic): | |
status: | Triaged → Fix Committed |
tags: | added: verification-needed |
tags: |
added: verification-done removed: verification-needed |
Changed in mplayer (Ubuntu): | |
status: | New → Invalid |
Changed in mplayer (Ubuntu Karmic): | |
status: | New → Invalid |
Changed in xorg-server: | |
status: | Unknown → Confirmed |
Changed in xorg-server: | |
status: | Confirmed → Fix Released |
Changed in xorg (Ubuntu): | |
importance: | Undecided → High |
status: | New → Triaged |
Changed in xorg (Ubuntu): | |
assignee: | nobody → Canonical Desktop Team (canonical-desktop-team) |
Changed in xorg-server (Ubuntu Karmic): | |
importance: | Undecided → Medium |
status: | New → Triaged |
Changed in me-tv: | |
status: | In Progress → Fix Committed |
Changed in me-tv: | |
milestone: | none → 1.2.2 |
Changed in me-tv: | |
status: | Fix Committed → Fix Released |
Changed in gnome-screensaver: | |
status: | Unknown → Confirmed |
Changed in xorg-server: | |
importance: | Unknown → Medium |
Changed in gnome-screensaver: | |
importance: | Unknown → High |
Changed in xorg-server: | |
importance: | Medium → Unknown |
Changed in xorg-server: | |
importance: | Unknown → Medium |
Changed in xorg-server (Ubuntu Karmic): | |
status: | Triaged → Won't Fix |
Changed in gnome-screensaver (Ubuntu Karmic): | |
assignee: | Chris Coulson (chrisccoulson) → nobody |
milestone: | karmic-updates → none |
status: | Triaged → Won't Fix |
Changed in gnome-screensaver: | |
status: | Confirmed → Won't Fix |
Changed in gnome-screensaver: | |
status: | Won't Fix → Confirmed |
Changed in vlc: | |
importance: | Undecided → Unknown |
status: | Won't Fix → Unknown |
Changed in vlc: | |
status: | Unknown → New |
Changed in vlc: | |
status: | New → Unknown |
Changed in gnome-screensaver: | |
status: | Confirmed → Won't Fix |
Changed in vlc: | |
status: | Unknown → New |
Changed in vlc: | |
status: | New → Unknown |
Confirmed, same behavior on up-to-date Karmic i386.