rpms/xorg-x11-server/F-11 xserver-1.6.99-dpms.patch, NONE, 1.1 xorg-x11-server.spec, 1.456, 1.457

[Peter Hutterer]

Author: whot

Update of /cvs/pkgs/rpms/xorg-x11-server/F-11
In directory cvs1.fedora.phx.redhat.com:/tmp/cvs-serv14846

Modified Files:
	xorg-x11-server.spec 
Added Files:
	xserver-1.6.99-dpms.patch 
Log Message:
* Mon Aug 17 2009 Peter Hutterer <peter.hutterer at redhat.com> 1.6.3-3
- xserver-1.6.99-dpms.patch: don't reset last event time on DPMS changes.


xserver-1.6.99-dpms.patch:
 dpms.c |   18 ++++--------------
 1 file changed, 4 insertions(+), 14 deletions(-)

--- NEW FILE xserver-1.6.99-dpms.patch ---
>From c1d901d723c3bee523736eacc15b44a7dff484fe Mon Sep 17 00:00:00 2001
From: Richard Hughes <richard at hughsie.com>
Date: Fri, 14 Aug 2009 11:44:35 +0100
Subject: [PATCH] Don't reset the lastDeviceEventTime when doing DPMS actions

When we change the DPMS mode, don't play games with the last event time as
this breaks applications using IDLETIME to turn the backlight off after a
preset time.

This patch fixes gnome-power-manager and xfce-power-manager

Signed-off-by: Peter Hutterer <peter.hutterer at who-t.net>
---
 Xext/dpms.c |   17 ++++-------------
 1 files changed, 4 insertions(+), 13 deletions(-)

diff --git a/Xext/dpms.c b/Xext/dpms.c
index 21ba923..df63a8b 100644
--- a/Xext/dpms.c
+++ b/Xext/dpms.c
@@ -177,19 +177,10 @@ ProcDPMSForceLevel(ClientPtr client)
     if (!DPMSEnabled)
 	return BadMatch;
 
-    if (stuff->level == DPMSModeOn) {
-      lastDeviceEventTime.milliseconds =
-          GetTimeInMillis();
-    } else if (stuff->level == DPMSModeStandby) {
-      lastDeviceEventTime.milliseconds =
-          GetTimeInMillis() -  DPMSStandbyTime;
-    } else if (stuff->level == DPMSModeSuspend) {
-      lastDeviceEventTime.milliseconds =
-          GetTimeInMillis() -  DPMSSuspendTime;
-    } else if (stuff->level == DPMSModeOff) {
-      lastDeviceEventTime.milliseconds =
-          GetTimeInMillis() -  DPMSOffTime;
-    } else {
+    if (stuff->level != DPMSModeOn &&
+        stuff->level != DPMSModeStandby &&
+        stuff->level != DPMSModeSuspend &&
+        stuff->level != DPMSModeOff) {
 	client->errorValue = stuff->level;
 	return BadValue;
     }
-- 
1.6.3.rc1.2.g0164.dirty



Index: xorg-x11-server.spec
===================================================================
RCS file: /cvs/pkgs/rpms/xorg-x11-server/F-11/xorg-x11-server.spec,v
retrieving revision 1.456
retrieving revision 1.457
diff -u -p -r1.456 -r1.457
--- xorg-x11-server.spec	13 Aug 2009 21:56:11 -0000	1.456
+++ xorg-x11-server.spec	17 Aug 2009 04:38:08 -0000	1.457
@@ -19,7 +19,7 @@
 Summary:   X.Org X11 X server
 Name:      xorg-x11-server
 Version:   1.6.3
-Release:   2%{?dist}
+Release:   3%{?dist}
 URL:       http://www.x.org
 License:   MIT
 Group:     User Interface/X
@@ -113,6 +113,8 @@ Patch6047: xserver-1.6.2-vboxvideo.patch
 
 # 501601, upstream, nominated for 1.6.4
 Patch6048: xserver-1.6.3-sync-conditions.patch
+# 501601, part 2.
+Patch6049: xserver-1.6.99-dpms.patch
 
 %define moduledir	%{_libdir}/xorg/modules
 %define drimoduledir	%{_libdir}/dri
@@ -530,6 +532,9 @@ rm -rf $RPM_BUILD_ROOT
 
 
 %changelog
+* Mon Aug 17 2009 Peter Hutterer <peter.hutterer at redhat.com> 1.6.3-3
+- xserver-1.6.99-dpms.patch: don't reset last event time on DPMS changes.
+
 * Thu Aug 13 2009 Peter Hutterer <peter.hutterer at redhat.com> 1.6.3-2
 - xserver-1.6.3-sync-conditions.patch: Fix conditions when checking
   negative transition triggers in the sync extension (#501601)