[gnome-shell] screencast: Fix disabling screencasts via session mode



commit 9520e87a384cb5cfaa5099e05931f8a3b77e6d33
Author: Florian Müllner <fmuellner gnome org>
Date:   Mon Nov 4 14:06:33 2013 +0100

    screencast: Fix disabling screencasts via session mode
    
    If screencasts are disabled, we return a DBus error, but still start
    the recording happily - add early returns in that case.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=699752

 js/ui/screencast.js |    8 ++++++--
 1 files changed, 6 insertions(+), 2 deletions(-)
---
diff --git a/js/ui/screencast.js b/js/ui/screencast.js
index a1593b2..23d6a42 100644
--- a/js/ui/screencast.js
+++ b/js/ui/screencast.js
@@ -105,8 +105,10 @@ const ScreencastService = new Lang.Class({
 
     ScreencastAsync: function(params, invocation) {
         let returnValue = [false, ''];
-        if (!Main.sessionMode.allowScreencast)
+        if (!Main.sessionMode.allowScreencast) {
             invocation.return_value(GLib.Variant.new('(bs)', returnValue));
+            return;
+        }
 
         let sender = invocation.get_sender();
         let recorder = this._ensureRecorderForSender(sender);
@@ -124,8 +126,10 @@ const ScreencastService = new Lang.Class({
 
     ScreencastAreaAsync: function(params, invocation) {
         let returnValue = [false, ''];
-        if (!Main.sessionMode.allowScreencast)
+        if (!Main.sessionMode.allowScreencast) {
             invocation.return_value(GLib.Variant.new('(bs)', returnValue));
+            return;
+        }
 
         let sender = invocation.get_sender();
         let recorder = this._ensureRecorderForSender(sender);


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]