Comment 74 for bug 595555

Revision history for this message
In , tim.liim (tim.liim-redhat-bugs) wrote :

Created attachment 460990
proof-of concept patch v3. (need to undo previous patch first, then apply this patch)

I found some issue with previous patch in Comment #57, Comment #58, so
here is a second try. The issue: when g-p-m (gnome-power-manager)
doubles its timeout to be beyond that of g-ss (eg. g-ss has 60 sec
timeout, and g-p-m has 80 sec timeout; see bug566351 Comment #1),
pIdleTimeValueLess was not updated properly. To test this scenario
without much waiting, do
    k=/apps/gnome-power-manager/backlight/idle_dim_time
    gconftool-2 -g $k # note down this number
    gconftool-2 -s $k -t int 80 # remember to set it back when done

Remember to undo previous patch before applying this one. This patch
is also based on Comment #49.

Adam,
Chris,
James,
Would you please review this patch as well?
Thanks.