Review Request 125910: Fix for Bug 334525 - Gwenview hangs when switching from normal to full screen mode

[Luigi Toscano]

> On Dic. 30, 2015, 3:48 p.m., Luigi Toscano wrote:
> > For the record: this was submitted to the knotifications frameworks, not to kdelibs.
> 
> Martin Klapetek wrote:
>     Yes, sorry, I wrote it as a message here but now I see that it didn't get through; I still have "Publish" here :S
>     
>     There are no more kdelibs releases planned and the same bug happens in frameworks, so I pushed it to frameworks.

Afaik we are still publishing patch releases of kdelibs 4.14 branch, so I would add the fix there too.


- Luigi


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125910/#review90339
-----------------------------------------------------------


On Dic. 29, 2015, 9:54 p.m., Johannes Stefan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125910/
> -----------------------------------------------------------
> 
> (Updated Dic. 29, 2015, 9:54 p.m.)
> 
> 
> Review request for kdelibs and Martin Klapetek.
> 
> 
> Bugs: 334525
>     http://bugs.kde.org/show_bug.cgi?id=334525
> 
> 
> Repository: kdelibs
> 
> 
> Description
> -------
> 
> Setting default reason for going into fullscreen mode
> 
> 
> Diffs
> -----
> 
>   kdeui/notifications/knotificationrestrictions.cpp 818edea 
> 
> Diff: https://git.reviewboard.kde.org/r/125910/diff/
> 
> 
> Testing
> -------
> 
> Compiled an tested - DBus Log as expected:
> 
> ##### begin DBUS-log #####
> method call sender=:1.223 -> dest=org.freedesktop.ScreenSaver serial=56 path=/ScreenSaver; interface=org.freedesktop.ScreenSaver; member=Inhibit
>    string "Gwenview"
>    string "no_reason_specified"
> method call sender=:1.6 -> dest=:1.3 serial=588 path=/org/gnome/SessionManager; interface=org.gnome.SessionManager; member=Inhibit
>    string "Gwenview"
>    uint32 0
>    string "no_reason_specified"
>    uint32 8
> signal sender=:1.3 -> dest=(null destination) serial=351 path=/org/gnome/SessionManager; interface=org.freedesktop.DBus.Properties; member=PropertiesChanged
>    string "org.gnome.SessionManager"
>    array [
>       dict entry(
>          string "InhibitedActions"
>          variant             uint32 8
>       )
>    ]
>    array [
>    ]
> signal sender=:1.3 -> dest=(null destination) serial=352 path=/org/gnome/SessionManager; interface=org.gnome.SessionManager; member=InhibitorAdded
>    object path "/org/gnome/SessionManager/Inhibitor29"
> method return sender=:1.3 -> dest=:1.6 reply_serial=588
>    uint32 1169992534
> method call sender=:1.4 -> dest=:1.21 serial=56 path=/org/gnome/Mutter/IdleMonitor/Core; interface=org.gnome.Mutter.IdleMonitor; member=RemoveWatch
>    uint32 35
> method call sender=:1.6 -> dest=org.freedesktop.DBus serial=589 path=/org/freedesktop/DBus; interface=org.freedesktop.DBus; member=AddMatch
>    string "type='signal',sender='org.freedesktop.DBus',interface='org.freedesktop.DBus',member='NameOwnerChanged',path='/org/freedesktop/DBus',arg0=':1.223'"
> method call sender=:1.6 -> dest=org.freedesktop.DBus serial=590 path=/org/freedesktop/DBus; interface=org.freedesktop.DBus; member=GetNameOwner
>    string ":1.223"
> method return sender=:1.6 -> dest=:1.223 reply_serial=56
>    uint32 1169992534
> method call sender=:1.6 -> dest=:1.21 serial=592 path=/org/gnome/Mutter/DisplayConfig; interface=org.freedesktop.DBus.Properties; member=Set
>    string "org.gnome.Mutter.DisplayConfig"
>    string "PowerSaveMode"
>    variant       int32 0
> method call sender=:1.21 -> dest=:1.3 serial=1073 path=/org/gnome/SessionManager; interface=org.gnome.SessionManager; member=IsInhibited
>    uint32 16
> method call sender=:1.21 -> dest=org.freedesktop.DBus serial=1074 path=/org/freedesktop/DBus; interface=org.freedesktop.DBus; member=RemoveMatch
>    string "type='signal',sender='org.freedesktop.DBus',interface='org.freedesktop.DBus',member='NameOwnerChanged',path='/org/freedesktop/DBus',arg0=':1.4'"
> method return sender=:1.21 -> dest=:1.4 reply_serial=56
> method return sender=:1.3 -> dest=:1.21 reply_serial=1073
>    boolean false
> ##### end of DBUS-log #####
> 
> 
> Thanks,
> 
> Johannes Stefan
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20151230/a69fed52/attachment.htm>