[gnome-shell] shell-util: Remove shell_util_icon_from_string

[Jasper St. Pierre <jstpierre src gnome org>]

commit 786cfbd3976b4077e8a24737fc86a85e21f2c219
Author: Jasper St. Pierre <jstpierre mecheye net>
Date:   Fri Oct 7 17:35:38 2011 -0400

    shell-util: Remove shell_util_icon_from_string
    
    GJS doesn't need to be able to represent interfaces for you to be able to
    access an interface method, so Gio.icon_new_for_string works fine.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=661231

 js/ui/status/power.js    |    4 ++--
 js/ui/telepathyClient.js |    6 +++---
 src/shell-util.c         |   16 ----------------
 src/shell-util.h         |    2 --
 4 files changed, 5 insertions(+), 23 deletions(-)
---
diff --git a/js/ui/status/power.js b/js/ui/status/power.js
index a48aac5..72e47a1 100644
--- a/js/ui/status/power.js
+++ b/js/ui/status/power.js
@@ -155,7 +155,7 @@ Indicator.prototype = {
     _devicesChanged: function() {
         this._proxy.GetRemote('Icon', Lang.bind(this, function(icon, error) {
             if (icon) {
-                let gicon = Shell.util_icon_from_string (icon);
+                let gicon = Gio.icon_new_for_string(icon);
                 this.setGIcon(gicon);
                 this.actor.show();
             } else {
@@ -183,7 +183,7 @@ DeviceItem.prototype = {
         this._box = new St.BoxLayout({ style_class: 'popup-device-menu-item' });
         this._label = new St.Label({ text: this._deviceTypeToString(device_type) });
 
-        this._icon = new St.Icon({ gicon: Shell.util_icon_from_string(icon),
+        this._icon = new St.Icon({ gicon: Gio.icon_new_for_string(icon),
                                    icon_type: St.IconType.SYMBOLIC,
                                    style_class: 'popup-menu-icon' });
 
diff --git a/js/ui/telepathyClient.js b/js/ui/telepathyClient.js
index 3315366..7578d5a 100644
--- a/js/ui/telepathyClient.js
+++ b/js/ui/telepathyClient.js
@@ -257,7 +257,7 @@ Client.prototype = {
         // FIXME: We don't have a 'chat room' icon (bgo #653737) use
         // system-users for now as Empathy does.
         let source = new ApproverSource(dispatchOp, _("Invitation"),
-                                        Shell.util_icon_from_string('system-users'));
+                                        Gio.icon_new_for_string('system-users'));
         Main.messageTray.add(source);
 
         let notif = new RoomInviteNotification(source, dispatchOp, channel, contacts[0]);
@@ -325,8 +325,8 @@ Client.prototype = {
 
         // We got the TpContact
         let source = new ApproverSource(dispatchOp, _("Call"), isVideo ?
-                                        Shell.util_icon_from_string('camera-web') :
-                                        Shell.util_icon_from_string('audio-input-microphone'));
+                                        Gio.icon_new_for_string('camera-web') :
+                                        Gio.icon_new_for_string('audio-input-microphone'));
         Main.messageTray.add(source);
 
         let notif = new AudioVideoNotification(source, dispatchOp, channel, contacts[0], isVideo);
diff --git a/src/shell-util.c b/src/shell-util.c
index a06f99a..2354995 100644
--- a/src/shell-util.c
+++ b/src/shell-util.c
@@ -395,22 +395,6 @@ shell_util_get_icon_for_uri (const char *text_uri)
   return g_themed_icon_new ("gtk-file");
 }
 
-/**
- * shell_util_icon_from_string:
- * @string: a stringified #GIcon
- *
- * A static method equivalent to g_icon_new_for_string, workaround
- * for GJS not being able to represent Gio.Icon (which is an interface,
- * not a class).
- *
- * Returns: (transfer full): the icon which is represented by @string
- */
-GIcon *
-shell_util_icon_from_string (const char *string, GError **error)
-{
-  return g_icon_new_for_string (string, error);
-}
-
 static void
 stop_pick (ClutterActor       *actor,
            const ClutterColor *color)
diff --git a/src/shell-util.h b/src/shell-util.h
index c0d4567..b0752ae 100644
--- a/src/shell-util.h
+++ b/src/shell-util.h
@@ -11,8 +11,6 @@ G_BEGIN_DECLS
 
 char    *shell_util_get_label_for_uri          (const char       *text_uri);
 GIcon   *shell_util_get_icon_for_uri           (const char       *text_uri);
-GIcon   *shell_util_icon_from_string           (const char       *string,
-                                                GError          **error);
 
 void     shell_util_set_hidden_from_pick       (ClutterActor     *actor,
                                                 gboolean          hidden);