Merge lp:~mterry/unity-greeter/dbus-interface into lp:unity-greeter

[Michael Terry]

Expose a dbus session interface for setting active users.

Expose a dbus session interface for setting active users. This could be useful for custom indicators (which can only happen after https://code.launchpad.net/~mterry/unity-greeter/indicator-gsettings/+merge/151573 )

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:776
http://jenkins.qa.ubuntu.com/job/unity-greeter-ci/6/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-greeter-ci/./label=precise/6/console

Click here to trigger a rebuild:
http://jenkins.qa.ubuntu.com/job/unity-greeter-ci/6//rebuild/?

review: Needs Fixing (continuous-integration)

[Michael Terry (mterry)]

Wait, I'm just now realizing you said elsewhere: "there could potentially be more than one greeter running at the same time so the bus address would need to be unique"

When does LightDM spawn more than one? I can rework patch to fit.

[lp:~mterry/unity-greeter/dbus-interface]

777. By Michael Terry on 2013-03-04

Merge indicator-gsettings branch

778. By Michael Terry on 2013-03-04

Use unique dbus name, rather than claiming a global one

Unmerged revisions

=== modified file 'data/com.canonical.unity-greeter.gschema.xml'
--- data/com.canonical.unity-greeter.gschema.xml	2012-09-11 17:52:41 +0000
+++ data/com.canonical.unity-greeter.gschema.xml	2013-03-04 19:12:20 +0000
@@ -82,5 +82,9 @@
       <default>true</default>
       <summary>Whether to play sound when greeter is ready</summary>
     </key>
+    <key name="indicators" type="as">
+      <default>['ug-keyboard', 'ug-accessibility', 'session', 'datetime', 'power', 'soundmenu', 'application']</default>
+      <summary>Which indicators to load</summary>
+    </key>
   </schema>
 </schemalist>

=== modified file 'src/greeter-list.vala'
--- src/greeter-list.vala	2013-01-03 15:15:54 +0000
+++ src/greeter-list.vala	2013-03-04 19:12:20 +0000
@@ -24,6 +24,22 @@
     return (int) (size % grid_size) / 2;
 }
 
+[DBus (name="com.canonical.UnityGreeter.List")]
+public class ListDBusInterface : Object
+{
+    private GreeterList list;
+
+    public ListDBusInterface (GreeterList list)
+    {
+        this.list = list;
+    }
+
+    public void set_active_entry (string entry_name)
+    {
+        list.set_active_entry (entry_name);
+    }
+}
+
 public abstract class GreeterList : FadableBox
 {
     public Background background { get; construct; }
@@ -49,6 +65,7 @@
 
     protected List<PromptBox> entries = null;
 
+    private ListDBusInterface dbus_object;
 
     private double scroll_target_location;
     private double scroll_start_location;
@@ -152,6 +169,29 @@
 
         scroll_timer = new AnimateTimer (AnimateTimer.ease_out_quint, AnimateTimer.FAST);
         scroll_timer.animate.connect (animate_scrolling);
+
+        try
+        {
+            Bus.get.begin (BusType.SESSION, null, on_bus_acquired);
+        }
+        catch (IOError e)
+        {
+            debug ("Error getting session bus: %s", e.message);
+        }
+    }
+
+    private void on_bus_acquired (Object? obj, AsyncResult res)
+    {
+        try
+        {
+            var conn = Bus.get.end (res);
+            this.dbus_object = new ListDBusInterface (this);
+            conn.register_object ("/list", this.dbus_object);
+        }
+        catch (IOError e)
+        {
+            debug ("Error registering user list dbus object: %s", e.message);
+        }
     }
 
     public enum ScrollTarget
@@ -354,7 +394,15 @@
     {
         var e = find_entry (name);
         if (e != null)
-            select_entry (e, 1.0);
+        {
+            var direction = 1.0;
+            if (selected_entry != null &&
+                entries.index (selected_entry) > entries.index (e))
+            {
+                direction = -1.0;
+            }
+            select_entry (e, direction);
+        }
     }
 
     public void set_active_first_entry_with_prefix (string prefix)

=== modified file 'src/menubar.vala'
--- src/menubar.vala	2013-02-20 16:17:42 +0000
+++ src/menubar.vala	2013-03-04 19:12:20 +0000
@@ -322,6 +322,42 @@
         return keyboard_item;
     }
 
+    private void load_indicator (string indicator_name)
+    {
+        if (indicator_name == "ug-keyboard")
+        {
+            var keyboard_item = make_keyboard_indicator ();
+            insert (keyboard_item, (int) get_children ().length () - 1);
+        }
+        else if (indicator_name == "ug-accessibility")
+        {
+            var a11y_item = make_a11y_indicator ();
+            insert (a11y_item, (int) get_children ().length () - 1);
+        }
+        else
+        {
+            // Find file, if it exists
+            string[] names_to_try = {"lib" + indicator_name + ".so",
+                                     indicator_name + ".so",
+                                     indicator_name};
+            foreach (var filename in names_to_try)
+            {
+                var full_path = Path.build_filename (Config.INDICATORDIR, filename);
+                var io = new Indicator.Object.from_file (full_path);
+                if (io == null)
+                    continue;
+
+                indicator_objects.append (io);
+                io.entry_added.connect (indicator_added_cb);
+                io.entry_removed.connect (indicator_removed_cb);
+                foreach (var entry in io.get_entries ())
+                    indicator_added_cb (io, entry);
+
+                break;
+            }
+        }
+    }
+
     private void setup_indicators ()
     {
         /* Set indicators to run with reduced functionality */
@@ -334,30 +370,23 @@
         /* Hint to have gnome-settings-daemon run in greeter mode */
         greeter_set_env ("RUNNING_UNDER_GDM", "1");
 
-        var keyboard_item = make_keyboard_indicator ();
-        insert (keyboard_item, (int) get_children ().length () - 1);
-
-        var a11y_item = make_a11y_indicator ();
-        insert (a11y_item, (int) get_children ().length () - 1);
+        /* Let indicators know about our unique dbus name */
+        try
+        {
+            var conn = Bus.get_sync (BusType.SESSION);
+            greeter_set_env ("UNITY_GREETER_DBUS_NAME", conn.get_unique_name ());
+        }
+        catch (IOError e)
+        {
+            debug ("Could not set DBUS_NAME: %s", e.message);
+        }
 
         debug ("LANG=%s LANGUAGE=%s", Environment.get_variable ("LANG"), Environment.get_variable ("LANGUAGE"));
-        string[] filenames = { Path.build_filename (Config.INDICATORDIR, "libsession.so"),
-                               Path.build_filename (Config.INDICATORDIR, "libdatetime.so"),
-                               Path.build_filename (Config.INDICATORDIR, "libpower.so"),
-                               Path.build_filename (Config.INDICATORDIR, "libsoundmenu.so"),
-                               Path.build_filename (Config.INDICATORDIR, "libapplication.so") };
-        foreach (var filename in filenames)
-        {
-            var io = new Indicator.Object.from_file (filename);
-            if (io == null)
-                continue;
-
-            indicator_objects.append (io);
-            io.entry_added.connect (indicator_added_cb);
-            io.entry_removed.connect (indicator_removed_cb);
-            foreach (var entry in io.get_entries ())
-                indicator_added_cb (io, entry);
-        }
+
+        var indicator_list = UGSettings.get_strv(UGSettings.KEY_INDICATORS);
+        foreach (var indicator in indicator_list)
+            load_indicator(indicator);
+
         debug ("LANG=%s LANGUAGE=%s", Environment.get_variable ("LANG"), Environment.get_variable ("LANGUAGE"));
     }
 

=== modified file 'src/settings.vala'
--- src/settings.vala	2012-09-11 17:52:41 +0000
+++ src/settings.vala	2013-03-04 19:12:20 +0000
@@ -37,6 +37,7 @@
     public static const string KEY_HIGH_CONTRAST = "high-contrast";
     public static const string KEY_SCREEN_READER = "screen-reader";
     public static const string KEY_PLAY_READY_SOUND = "play-ready-sound";
+    public static const string KEY_INDICATORS = "indicators";
 
     public static bool get_boolean (string key)
     {
@@ -75,5 +76,11 @@
         return gsettings.set_boolean (key, value);
     }
 
+    public static string[] get_strv (string key)
+    {
+        var gsettings = new Settings (SCHEMA);
+        return gsettings.get_strv (key);
+    }
+
     private static const string SCHEMA = "com.canonical.unity-greeter";
 }