Merge lp:~kamstrup/libunity/finished-signal into lp:libunity

[Mikkel Kamstrup Erlandsen]

Adds a finished() method on PlaceSearch that will emit com.canonical.Unity.PlaceEntry.SearchFinished() on the parent place entry once called

[Neil J. Patel (njpatel)]

Looks good, approved!

review: Approve

=== modified file 'src/unity-place.vala'
--- src/unity-place.vala	2011-02-17 13:05:33 +0000
+++ src/unity-place.vala	2011-03-21 11:32:29 +0000
@@ -155,6 +155,8 @@
 
     private string search;
     private HashTable<string,string> hints;
+    private weak PlaceEntryInfo? parent_entry;
+    private uint section;
 
     /*
      * Public API
@@ -165,6 +167,8 @@
       GLib.Object();
       this.search = search;
       this.hints = hints;
+      parent_entry = null;
+      section = 0;
     }
 
     public string get_search_string ()
@@ -222,6 +226,46 @@
       
       return true;
     }
+    
+    public void set_section (uint section)
+    {
+      this.section = section;
+    }
+    
+    public uint get_section ()
+    {
+      return section;
+    }
+    
+    public void set_parent_entry (PlaceEntryInfo entry)
+    {
+      parent_entry = entry;
+    }
+    
+    public unowned PlaceEntryInfo? get_parent_entry ()
+    {
+      return parent_entry;
+    }
+    
+    public void finished ()
+    {
+      if (parent_entry == null)
+        return;
+  
+      try {
+        Variant vhints = hints;
+        var params = new Variant ("(us@a{ss})", section, search, vhints);
+        var conn = Bus.get_sync(BusType.SESSION);      
+        conn.emit_signal (null,                                 /* dest name */
+                          parent_entry.dbus_path,               /* orig path */
+                          "com.canonical.Unity.PlaceEntry",     /* iface */
+                          "SearchFinished",                     /* member */
+                          params);                              /* params */
+      } catch (IOError e) {
+        warning ("Failed to emit com.canonical.Unity.PlaceEntry.SearchFinished: %s",
+                 e.message);
+      }
+    }
   }
 
   /**
@@ -485,6 +529,8 @@
     public signal void global_renderer_info_changed (_PlaceRendererInfo renderer_info);
     
     public signal void place_entry_info_changed (_PlaceEntryInfoData entry_info_data);
+    
+    public signal void search_finished (uint section, string search, HashTable<string,string> hints);
   }
 
   /**
@@ -728,6 +774,7 @@
                                    HashTable<string,string> hints)
     {
       var s = new PlaceSearch (search, hints);
+      s.set_parent_entry (_entry_info);
       
       if (!s.equals (this._entry_info.active_global_search))
         this._entry_info.active_global_search = s;
@@ -737,6 +784,8 @@
                             HashTable<string,string> hints)
     {
       var s = new PlaceSearch (search, hints);
+      s.set_section (_entry_info.active_section);
+      s.set_parent_entry (_entry_info);
       
       if (!s.equals (this._entry_info.active_search))
         this._entry_info.active_search = s;