]> git.0d.be Git - empathy.git/blobdiff - libempathy-gtk/empathy-avatar-chooser.c
GNOME Goal: Update icon names
[empathy.git] / libempathy-gtk / empathy-avatar-chooser.c
index 78535b439a98e610235d3c1dffb4ef07c149683a..4c58a1aa58094aeae7338edd0dbe279d1a3cadb0 100644 (file)
@@ -1,4 +1,3 @@
-/* -*- Mode: C; tab-width: 8; indent-tabs-mode: t; c-basic-offset: 8 -*- */
 /*
  * Copyright (C) 2006-2007 Imendio AB.
  * Copyright (C) 2007-2008 Collabora Ltd.
  *
  * You should have received a copy of the GNU General Public
  * License along with this library; if not, write to the
- * Free Software Foundation, Inc., 59 Temple Place - Suite 330,
- * Boston, MA 02111-1307, USA.
+ * Free Software Foundation, Inc., 51 Franklin St, Fifth Floor,
+ * Boston, MA  02110-1301  USA
  *
  * Authors: Based on Novell's e-image-chooser.
  *          Xavier Claessens <xclaesse@gmail.com>
  */
 
 #include "config.h"
+#include "empathy-avatar-chooser.h"
 
-#include <string.h>
+#include <glib/gi18n-lib.h>
 
-#include <glib/gi18n.h>
-#include <gtk/gtk.h>
-#include <gio/gio.h>
+#ifdef HAVE_CHEESE
+#include <cheese-avatar-chooser.h>
+#endif /* HAVE_CHEESE */
 
-#include <libempathy/empathy-utils.h>
-#include "empathy-avatar-chooser.h"
-#include "empathy-conf.h"
+#include "empathy-camera-monitor.h"
+#include "empathy-gsettings.h"
+#include "empathy-images.h"
 #include "empathy-ui-utils.h"
+#include "empathy-utils.h"
 
 #define DEBUG_FLAG EMPATHY_DEBUG_OTHER
-#include <libempathy/empathy-debug.h>
+#include "empathy-debug.h"
+
+/**
+ * SECTION:empathy-avatar-chooser
+ * @title: EmpathyAvatarChooser
+ * @short_description: A widget used to change avatar
+ * @include: libempathy-gtk/empathy-avatar-chooser.h
+ *
+ * #EmpathyAvatarChooser is a widget which extends #GtkButton to
+ * provide a way of changing avatar.
+ */
+
+/**
+ * EmpathyAvatarChooser:
+ * @parent: parent object
+ *
+ * Widget which extends #GtkButton to provide a way of changing avatar.
+ */
 
 #define AVATAR_SIZE_SAVE 96
 #define AVATAR_SIZE_VIEW 64
 #define DEFAULT_DIR DATADIR"/pixmaps/faces"
 
-#define GET_PRIV(obj) EMPATHY_GET_PRIV (obj, EmpathyAvatarChooser)
-typedef struct {
-       EmpathyContactFactory   *contact_factory;
-       McAccount               *account;
-       EmpathyTpContactFactory *tp_contact_factory;
-
-       gulong ready_handler_id;
-
-       EmpathyAvatar *avatar;
-} EmpathyAvatarChooserPriv;
-
-static void       avatar_chooser_finalize              (GObject              *object);
-static void       avatar_chooser_set_account           (EmpathyAvatarChooser *self,
-                                                       McAccount            *account);
-static void       avatar_chooser_set_image             (EmpathyAvatarChooser *chooser,
-                                                       EmpathyAvatar        *avatar,
-                                                       GdkPixbuf            *pixbuf,
-                                                       gboolean              set_locally);
-static gboolean   avatar_chooser_drag_motion_cb        (GtkWidget            *widget,
-                                                       GdkDragContext       *context,
-                                                       gint                  x,
-                                                       gint                  y,
-                                                       guint                 time,
-                                                       EmpathyAvatarChooser *chooser);
-static void       avatar_chooser_drag_leave_cb         (GtkWidget            *widget,
-                                                       GdkDragContext       *context,
-                                                       guint                 time,
-                                                       EmpathyAvatarChooser *chooser);
-static gboolean   avatar_chooser_drag_drop_cb          (GtkWidget            *widget,
-                                                       GdkDragContext       *context,
-                                                       gint                  x,
-                                                       gint                  y,
-                                                       guint                 time,
-                                                       EmpathyAvatarChooser *chooser);
-static void       avatar_chooser_drag_data_received_cb (GtkWidget            *widget,
-                                                       GdkDragContext       *context,
-                                                       gint                  x,
-                                                       gint                  y,
-                                                       GtkSelectionData     *selection_data,
-                                                       guint                 info,
-                                                       guint                 time,
-                                                       EmpathyAvatarChooser *chooser);
-static void       avatar_chooser_clicked_cb            (GtkWidget            *button,
-                                                       EmpathyAvatarChooser *chooser);
-
-enum {
-       CHANGED,
-       LAST_SIGNAL
-};
+#ifdef HAVE_CHEESE
+/*
+ * A custom GtkResponseType used when the user presses the
+ * "Camera Picture" button. Any positive value would be sufficient.
+ */
+#define EMPATHY_AVATAR_CHOOSER_RESPONSE_WEBCAM   10
+#endif
+#define EMPATHY_AVATAR_CHOOSER_RESPONSE_NO_IMAGE GTK_RESPONSE_NO
+#define EMPATHY_AVATAR_CHOOSER_RESPONSE_CANCEL   GTK_RESPONSE_CANCEL
+#define EMPATHY_AVATAR_CHOOSER_RESPONSE_FILE     GTK_RESPONSE_OK
 
-enum {
-       PROP_0,
-       PROP_ACCOUNT
+struct _EmpathyAvatarChooserPrivate
+{
+  TpAccount *account;
+
+  GArray *avatar;
+  gchar *mime_type;
+  gboolean changed;
+
+  GtkFileChooser *chooser_dialog;
+  GSettings *gsettings_ui;
 };
 
-static guint signals [LAST_SIGNAL];
+enum
+{
+  PROP_0,
+  PROP_ACCOUNT
+};
 
 G_DEFINE_TYPE (EmpathyAvatarChooser, empathy_avatar_chooser, GTK_TYPE_BUTTON);
 
@@ -105,890 +95,1128 @@ G_DEFINE_TYPE (EmpathyAvatarChooser, empathy_avatar_chooser, GTK_TYPE_BUTTON);
  */
 #define URI_LIST_TYPE "text/uri-list"
 
-enum DndTargetType {
-       DND_TARGET_TYPE_URI_LIST
+enum DndTargetType
+{
+  DND_TARGET_TYPE_URI_LIST
 };
 
-static const GtkTargetEntry drop_types[] = {
-       { URI_LIST_TYPE, 0, DND_TARGET_TYPE_URI_LIST },
+static const GtkTargetEntry drop_types[] =
+{
+  { URI_LIST_TYPE, 0, DND_TARGET_TYPE_URI_LIST },
 };
 
-static void
-avatar_chooser_get_property (GObject    *object,
-                            guint       param_id,
-                            GValue     *value,
-                            GParamSpec *pspec)
-{
-       EmpathyAvatarChooserPriv *priv = GET_PRIV (object);
-
-       switch (param_id) {
-       case PROP_ACCOUNT:
-               g_value_set_object (value, priv->account);
-               break;
-       default:
-               G_OBJECT_WARN_INVALID_PROPERTY_ID (object, param_id, pspec);
-               break;
-       }
-}
+static void avatar_chooser_set_image (EmpathyAvatarChooser *self,
+    GArray *avatar,
+    gchar *mime_type,
+    GdkPixbuf *pixbuf,
+    gboolean maybe_convert);
+static void avatar_chooser_clear_image (EmpathyAvatarChooser *self);
 
 static void
-avatar_chooser_set_property (GObject      *object,
-                            guint         param_id,
-                            const GValue *value,
-                            GParamSpec   *pspec)
-{
-       EmpathyAvatarChooser *self = EMPATHY_AVATAR_CHOOSER (object);
-
-       switch (param_id) {
-       case PROP_ACCOUNT:
-               avatar_chooser_set_account (self, g_value_get_object (value));
-               break;
-       default:
-               G_OBJECT_WARN_INVALID_PROPERTY_ID (object, param_id, pspec);
-               break;
-       }
+get_avatar_cb (GObject *source,
+    GAsyncResult *result,
+    gpointer user_data)
+{
+  TpWeakRef *wr = user_data;
+  EmpathyAvatarChooser *self = tp_weak_ref_dup_object (wr);
+  const GArray *avatar;
+  GdkPixbuf *pixbuf;
+  gchar *mime_type;
+  GError *error = NULL;
+
+  if (self == NULL)
+    {
+      tp_weak_ref_destroy (wr);
+      return;
+    }
+
+  avatar = tp_account_get_avatar_finish (self->priv->account, result, &error);
+  if (avatar == NULL)
+    {
+      DEBUG ("Error getting account's avatar: %s", error->message);
+      g_clear_error (&error);
+      goto out;
+    }
+
+  if (avatar->len == 0)
+    {
+      avatar_chooser_clear_image (self);
+      goto out;
+    }
+
+  pixbuf = empathy_pixbuf_from_data_and_mime ((gchar *) avatar->data,
+      avatar->len, &mime_type);
+  if (pixbuf == NULL)
+    {
+      DEBUG ("couldn't make a pixbuf from avatar; giving up");
+      goto out;
+    }
+
+  avatar_chooser_set_image (self, (GArray *) avatar, mime_type, pixbuf, FALSE);
+  g_free (mime_type);
+
+  self->priv->changed = FALSE;
+
+out:
+  tp_weak_ref_destroy (wr);
+  g_object_unref (self);
 }
 
 static void
-empathy_avatar_chooser_class_init (EmpathyAvatarChooserClass *klass)
+avatar_changed_cb (TpAccount *account,
+    gpointer user_data,
+    GObject *weak_object)
 {
-       GObjectClass *object_class = G_OBJECT_CLASS (klass);
-       GParamSpec *param_spec;
-
-       object_class->finalize = avatar_chooser_finalize;
-       object_class->get_property = avatar_chooser_get_property;
-       object_class->set_property = avatar_chooser_set_property;
-
-       signals[CHANGED] =
-               g_signal_new ("changed",
-                             G_TYPE_FROM_CLASS (klass),
-                             G_SIGNAL_RUN_LAST,
-                             0,
-                             NULL, NULL,
-                             g_cclosure_marshal_VOID__VOID,
-                             G_TYPE_NONE, 0);
-
-       param_spec = g_param_spec_object ("account",
-                                         "McAccount",
-                                         "McAccount whose avatar should be "
-                                         "shown and modified by this widget",
-                                         MC_TYPE_ACCOUNT,
-                                         G_PARAM_READWRITE |
-                                         G_PARAM_STATIC_STRINGS);
-       g_object_class_install_property (object_class,
-                                        PROP_ACCOUNT,
-                                        param_spec);
-
-       g_type_class_add_private (object_class, sizeof (EmpathyAvatarChooserPriv));
-}
+  EmpathyAvatarChooser *self = (EmpathyAvatarChooser *) weak_object;
 
-static void
-empathy_avatar_chooser_init (EmpathyAvatarChooser *chooser)
-{
-       EmpathyAvatarChooserPriv *priv = G_TYPE_INSTANCE_GET_PRIVATE (chooser,
-               EMPATHY_TYPE_AVATAR_CHOOSER, EmpathyAvatarChooserPriv);
-
-       chooser->priv = priv;
-       gtk_drag_dest_set (GTK_WIDGET (chooser),
-                          GTK_DEST_DEFAULT_ALL,
-                          drop_types,
-                          G_N_ELEMENTS (drop_types),
-                          GDK_ACTION_COPY);
-
-       g_signal_connect (chooser, "drag-motion",
-                         G_CALLBACK (avatar_chooser_drag_motion_cb),
-                         chooser);
-       g_signal_connect (chooser, "drag-leave",
-                         G_CALLBACK (avatar_chooser_drag_leave_cb),
-                         chooser);
-       g_signal_connect (chooser, "drag-drop",
-                         G_CALLBACK (avatar_chooser_drag_drop_cb),
-                         chooser);
-       g_signal_connect (chooser, "drag-data-received",
-                         G_CALLBACK (avatar_chooser_drag_data_received_cb),
-                         chooser);
-       g_signal_connect (chooser, "clicked",
-                         G_CALLBACK (avatar_chooser_clicked_cb),
-                         chooser);
-
-       priv->contact_factory = empathy_contact_factory_new ();
-
-       empathy_avatar_chooser_set (chooser, NULL);
+  tp_account_get_avatar_async (self->priv->account,
+      get_avatar_cb, tp_weak_ref_new (self, NULL, NULL));
 }
 
 static void
-avatar_chooser_finalize (GObject *object)
+avatar_chooser_constructed (GObject *object)
 {
-       EmpathyAvatarChooserPriv *priv;
+  EmpathyAvatarChooser *self = (EmpathyAvatarChooser *) object;
 
-       priv = GET_PRIV (object);
+  G_OBJECT_CLASS (empathy_avatar_chooser_parent_class)->constructed (object);
 
-       avatar_chooser_set_account (EMPATHY_AVATAR_CHOOSER (object), NULL);
-       g_assert (priv->account == NULL);
-       g_assert (priv->tp_contact_factory == NULL);
+  tp_account_get_avatar_async (self->priv->account,
+      get_avatar_cb, tp_weak_ref_new (self, NULL, NULL));
 
-       g_object_unref (priv->contact_factory);
+  /* FIXME: no signal on TpAccount, yet.
+   * See https://bugs.freedesktop.org/show_bug.cgi?id=52938 */
+  tp_cli_account_interface_avatar_connect_to_avatar_changed (
+      self->priv->account, avatar_changed_cb, NULL, NULL, (GObject *) self,
+      NULL);
+}
 
-       if (priv->avatar != NULL) {
-               empathy_avatar_unref (priv->avatar);
-       }
+static void
+avatar_chooser_get_property (GObject *object,
+    guint param_id,
+    GValue *value,
+    GParamSpec *pspec)
+{
+  EmpathyAvatarChooser *self = (EmpathyAvatarChooser *) object;
+
+  switch (param_id)
+    {
+      case PROP_ACCOUNT:
+        g_value_set_object (value, self->priv->account);
+        break;
+      default:
+        G_OBJECT_WARN_INVALID_PROPERTY_ID (object, param_id, pspec);
+        break;
+    }
+}
 
-       G_OBJECT_CLASS (empathy_avatar_chooser_parent_class)->finalize (object);
+static void
+avatar_chooser_set_property (GObject *object,
+    guint param_id,
+    const GValue *value,
+    GParamSpec *pspec)
+{
+  EmpathyAvatarChooser *self = EMPATHY_AVATAR_CHOOSER (object);
+
+  switch (param_id)
+    {
+      case PROP_ACCOUNT:
+        g_assert (self->priv->account == NULL); /* construct-only */
+        self->priv->account = g_value_dup_object (value);
+        break;
+      default:
+        G_OBJECT_WARN_INVALID_PROPERTY_ID (object, param_id, pspec);
+        break;
+    }
 }
 
 static void
-avatar_chooser_tp_cf_ready_cb (EmpathyTpContactFactory *tp_cf,
-                              GParamSpec              *unused,
-                              EmpathyAvatarChooser    *self)
+avatar_chooser_dispose (GObject *object)
 {
-       EmpathyAvatarChooserPriv *priv = GET_PRIV (self);
-       gboolean ready;
+  EmpathyAvatarChooser *self = (EmpathyAvatarChooser *) object;
 
-       /* sanity check that we're listening on the right ETpCF */
-       g_assert (priv->tp_contact_factory == tp_cf);
+  tp_clear_object (&self->priv->account);
+  tp_clear_pointer (&self->priv->avatar, g_array_unref);
+  tp_clear_pointer (&self->priv->mime_type, g_free);
+  tp_clear_object (&self->priv->gsettings_ui);
 
-       ready = empathy_tp_contact_factory_is_ready (tp_cf);
-       gtk_widget_set_sensitive (GTK_WIDGET (self), ready);
+  G_OBJECT_CLASS (empathy_avatar_chooser_parent_class)->dispose (object);
 }
 
 static void
-avatar_chooser_set_account (EmpathyAvatarChooser *self,
-                           McAccount            *account)
+empathy_avatar_chooser_class_init (EmpathyAvatarChooserClass *klass)
 {
-       EmpathyAvatarChooserPriv *priv = GET_PRIV (self);
+  GObjectClass *object_class = G_OBJECT_CLASS (klass);
+  GParamSpec *param_spec;
+
+  object_class->constructed = avatar_chooser_constructed;
+  object_class->dispose = avatar_chooser_dispose;
+  object_class->get_property = avatar_chooser_get_property;
+  object_class->set_property = avatar_chooser_set_property;
+
+  /**
+   * EmpathyAvatarChooser:account:
+   *
+   * The #TpAccount whose avatar should be shown and modified by
+   * the #EmpathyAvatarChooser instance.
+   */
+  param_spec = g_param_spec_object ("account",
+            "TpAccount",
+            "TpAccount whose avatar should be "
+            "shown and modified by this widget",
+            TP_TYPE_ACCOUNT,
+            G_PARAM_READWRITE |
+            G_PARAM_CONSTRUCT_ONLY |
+            G_PARAM_STATIC_STRINGS);
+  g_object_class_install_property (object_class,
+           PROP_ACCOUNT,
+           param_spec);
+
+  g_type_class_add_private (object_class, sizeof (EmpathyAvatarChooserPrivate));
+}
 
-       if (priv->account != NULL) {
-               g_object_unref (priv->account);
-               priv->account = NULL;
+static gboolean
+avatar_chooser_drag_motion_cb (GtkWidget *widget,
+    GdkDragContext *context,
+    gint x,
+    gint y,
+    guint time_,
+    EmpathyAvatarChooser *self)
+{
+  GList *p;
 
-               g_assert (priv->tp_contact_factory != NULL);
+  for (p = gdk_drag_context_list_targets (context); p != NULL;
+       p = p->next)
+    {
+      gchar *possible_type;
 
-               g_signal_handler_disconnect (priv->tp_contact_factory,
-                       priv->ready_handler_id);
-               priv->ready_handler_id = 0;
+      possible_type = gdk_atom_name (GDK_POINTER_TO_ATOM (p->data));
 
-               g_object_unref (priv->tp_contact_factory);
-               priv->tp_contact_factory = NULL;
-       }
+      if (!strcmp (possible_type, URI_LIST_TYPE))
+        {
+          g_free (possible_type);
+          gdk_drag_status (context, GDK_ACTION_COPY, time_);
 
-       if (account != NULL) {
-               priv->account = g_object_ref (account);
-               priv->tp_contact_factory = g_object_ref (
-                       empathy_contact_factory_get_tp_factory (
-                               priv->contact_factory, priv->account));
+          return TRUE;
+        }
 
-               priv->ready_handler_id = g_signal_connect (
-                       priv->tp_contact_factory, "notify::ready",
-                       G_CALLBACK (avatar_chooser_tp_cf_ready_cb), self);
-               avatar_chooser_tp_cf_ready_cb (priv->tp_contact_factory, NULL,
-                       self);
-       }
-}
+      g_free (possible_type);
+    }
 
-static gboolean
-str_in_strv (gchar  *str,
-            gchar **strv)
-{
-       if (strv == NULL) {
-               return FALSE;
-       }
-
-       while (*strv != NULL) {
-               if (g_str_equal (str, *strv)) {
-                       return TRUE;
-               }
-               strv++;
-       }
-       return FALSE;
+  return FALSE;
 }
 
-/* The caller must free the strings stored in satisfactory_format_name and
- * satisfactory_mime_type.
- */
 static gboolean
-can_satisfy_mime_type_requirements (gchar **accepted_mime_types,
-                                   gchar **satisfactory_format_name,
-                                   gchar **satisfactory_mime_type)
-{
-       gchar   *name = NULL,
-               *type = NULL;
-       gboolean done = FALSE;
-
-       if (accepted_mime_types == NULL || *accepted_mime_types == NULL) {
-               return FALSE;
-       }
-
-       g_assert (satisfactory_format_name != NULL);
-       g_assert (satisfactory_mime_type != NULL);
-
-       /* Special-case png and jpeg to avoid accidentally saving to something
-        * uncompressed like bmp. This assumes that we can write image/png and
-        * image/jpeg; if this isn't true then something's really wrong with
-        * GdkPixbuf.
-        */
-       if (str_in_strv ("image/png", accepted_mime_types)) {
-               name = g_strdup ("png");
-               type = g_strdup ("image/png");
-               done = TRUE;
-       } else if (str_in_strv ("image/jpeg", accepted_mime_types)) {
-               name = g_strdup ("jpeg");
-               type = g_strdup ("image/jpeg");
-               done = TRUE;
-       } else {
-               GSList  *formats = gdk_pixbuf_get_formats ();
-               GSList  *l;
-               gchar  **strv;
-
-               for (l = formats; !done && l != NULL; l = l->next) {
-                       GdkPixbufFormat *format = l->data;
-                       gchar **format_mime_types;
-
-                       if (!gdk_pixbuf_format_is_writable (format)) {
-                               continue;
-                       }
-
-                       format_mime_types = gdk_pixbuf_format_get_mime_types (format);
-                       for (strv = format_mime_types; *strv != NULL; strv++) {
-                               if (str_in_strv (*strv, accepted_mime_types)) {
-                                       name = gdk_pixbuf_format_get_name (format);
-                                       type = g_strdup (*strv);
-                                       done = TRUE;
-                                       break;
-                               }
-                       }
-                       g_strfreev (format_mime_types);
-               }
-
-               g_slist_free (formats);
-       }
-
-       if (done) {
-               *satisfactory_format_name = name;
-               *satisfactory_mime_type = type;
-               return TRUE;
-       } else {
-               /* check we're not leaking. */
-               g_assert (name == NULL);
-               g_assert (type == NULL);
-               return FALSE;
-       }
-}
+avatar_chooser_drag_drop_cb (GtkWidget *widget,
+    GdkDragContext *context,
+    gint x,
+    gint y,
+    guint time_,
+    EmpathyAvatarChooser *self)
+{
+  GList *p;
 
-static EmpathyAvatar *
-avatar_chooser_convert (EmpathyAvatarChooser *chooser,
-                       GdkPixbuf            *pixbuf_scaled,
-                       gchar               **mime_types,
-                       gsize                 max_size)
-{
-       gchar         *format_name = NULL, *new_mime_type = NULL;
-       gchar         *converted_image_data = NULL;
-       gsize          converted_image_size = 0;
-       EmpathyAvatar *converted_avatar = NULL;
-       gboolean       saved;
-       GError        *error = NULL;
-
-       if (!can_satisfy_mime_type_requirements (mime_types, &format_name,
-                                                &new_mime_type)) {
-               DEBUG ("Mon dieu! Can't convert to any acceptable format!");
-               return NULL;
-       }
-
-       saved = gdk_pixbuf_save_to_buffer (pixbuf_scaled, &converted_image_data,
-               &converted_image_size, format_name, &error, NULL);
-       g_free (format_name);
-
-       if (!saved) {
-               DEBUG ("Couldn't convert image: %s", error->message);
-               g_error_free (error);
-
-               g_free (new_mime_type);
-               return NULL;
-       }
-
-       /* Takes ownership of new_mime_type */
-       converted_avatar = empathy_avatar_new (converted_image_data,
-               converted_image_size, new_mime_type, NULL);
-
-       if (max_size > 0 && converted_avatar->len > max_size) {
-               /* FIXME: We could try converting to a different format; in
-                *        particular, try converting to jpeg with increasingly
-                *        high compression (if jpeg is supported). Not sure how
-                *        much we care.
-                */
-               DEBUG ("Converted the image, but the new filesize is too big");
-
-               empathy_avatar_unref (converted_avatar);
-               converted_avatar = NULL;
-       }
-
-       return converted_avatar;
-}
+  if (gdk_drag_context_list_targets (context) == NULL)
+    return FALSE;
+
+  for (p = gdk_drag_context_list_targets (context);
+       p != NULL; p = p->next)
+    {
+      char *possible_type;
+
+      possible_type = gdk_atom_name (GDK_POINTER_TO_ATOM (p->data));
+      if (!strcmp (possible_type, URI_LIST_TYPE))
+        {
+          g_free (possible_type);
+          gtk_drag_get_data (widget, context,
+                 GDK_POINTER_TO_ATOM (p->data),
+                 time_);
+
+          return TRUE;
+        }
 
-static EmpathyAvatar *
-avatar_chooser_maybe_convert_and_scale (EmpathyAvatarChooser *chooser,
-                                       GdkPixbuf            *pixbuf,
-                                       EmpathyAvatar        *avatar)
-{
-       EmpathyAvatarChooserPriv *priv = GET_PRIV (chooser);
-       EmpathyTpContactFactory  *tp_cf = priv->tp_contact_factory;
-       gint                      max_width = 0, max_height = 0, max_size = 0;
-       gchar                   **mime_types = NULL;
-       gboolean                  needs_conversion = FALSE;
-       GdkPixbuf                *pixbuf_scaled = NULL;
-
-       /* This should only be called if the user is setting a new avatar,
-        * which should only be allowed once the avatar requirements have been
-        * discovered.
-        */
-       g_return_val_if_fail (tp_cf != NULL, NULL);
-       g_return_val_if_fail (empathy_tp_contact_factory_is_ready (tp_cf),
-               NULL);
-
-       g_object_get (tp_cf,
-               "avatar-mime-types", &mime_types, /* Needs g_strfreev-ing */
-               "avatar-max-width", &max_width,
-               "avatar-max-height", &max_height,
-               "avatar-max-size", &max_size,
-               NULL);
-
-       /* If the avatar's not already the right type, it needs converting. */
-       if (!str_in_strv (avatar->format, mime_types)) {
-               needs_conversion = TRUE;
-       }
-
-       /* If _scale_down_if_necessary yields a new pixbuf, then it needed
-        * scaling.  If it's the same pixbuf, it did not, and did no extra work
-        * beyond checking the dimensions.  Hence, we call it even if
-        * needs_conversion is already TRUE, rather than duplicating the
-        * dimension checking code here, as it's no more expensive than
-        * checking the dimensions first in the case where no scaling is
-        * necessary.
-        */
-       pixbuf_scaled = empathy_pixbuf_scale_down_if_necessary (pixbuf,
-               MIN(max_width, max_height));
-       if (pixbuf_scaled != pixbuf) {
-               needs_conversion = TRUE;
-       }
-
-       if (max_size > 0 && avatar->len > max_size) {
-               needs_conversion = TRUE;
-       }
-
-       if (needs_conversion) {
-               avatar = avatar_chooser_convert (chooser, pixbuf_scaled,
-                       mime_types, max_size);
-       } else {
-               /* Just return another reference to the avatar passed in. */
-               avatar = empathy_avatar_ref (avatar);
-       }
-
-       g_object_unref (pixbuf_scaled);
-       g_strfreev (mime_types);
-       return avatar;
+      g_free (possible_type);
+    }
+
+  return FALSE;
 }
 
 static void
-avatar_chooser_clear_image (EmpathyAvatarChooser *chooser)
+avatar_chooser_clear_image (EmpathyAvatarChooser *self)
 {
-       EmpathyAvatarChooserPriv *priv = GET_PRIV (chooser);
-       GtkWidget *image;
-
-       if (priv->avatar == NULL) {
-               return;
-       }
+  GtkWidget *image;
 
-       empathy_avatar_unref (priv->avatar);
-       priv->avatar = NULL;
+  tp_clear_pointer (&self->priv->avatar, g_array_unref);
+  tp_clear_pointer (&self->priv->mime_type, g_free);
+  self->priv->changed = TRUE;
 
-       image = gtk_image_new_from_icon_name ("stock_person", GTK_ICON_SIZE_DIALOG);
-       gtk_button_set_image (GTK_BUTTON (chooser), image);
-       g_signal_emit (chooser, signals[CHANGED], 0);
+  image = gtk_image_new_from_icon_name (EMPATHY_IMAGE_AVATAR_DEFAULT,
+    GTK_ICON_SIZE_DIALOG);
+  gtk_button_set_image (GTK_BUTTON (self), image);
 }
 
-static void
-avatar_chooser_set_image_from_data (EmpathyAvatarChooser *chooser,
-                                   gchar                *data,
-                                   gsize                 size,
-                                   gboolean              set_locally)
-{
-       GdkPixbuf     *pixbuf;
-       EmpathyAvatar *avatar = NULL;
-       gchar         *mime_type = NULL;
-
-       if (data == NULL) {
-               avatar_chooser_clear_image (chooser);
-               g_free (data);
-               return;
-       }
-
-       pixbuf = empathy_pixbuf_from_data (data, size, &mime_type);
-       if (pixbuf == NULL) {
-               g_free (data);
-               return;
-       }
-
-       /* avatar takes ownership of data and mime_type */
-       avatar = empathy_avatar_new (data, size, mime_type, NULL);
-
-       avatar_chooser_set_image (chooser, avatar, pixbuf, set_locally);
+static gboolean
+str_in_strv (const gchar  *str,
+    gchar **strv)
+{
+  if (strv == NULL)
+    return FALSE;
+
+  while (*strv != NULL)
+    {
+      if (g_str_equal (str, *strv))
+        return TRUE;
+
+      strv++;
+    }
+
+  return FALSE;
 }
 
-static void
-avatar_chooser_set_image_from_avatar (EmpathyAvatarChooser *chooser,
-                                     EmpathyAvatar        *avatar,
-                                     gboolean              set_locally)
-{
-       GdkPixbuf *pixbuf;
-       gchar     *mime_type = NULL;
-
-       g_assert (avatar != NULL);
-
-       pixbuf = empathy_pixbuf_from_data (avatar->data, avatar->len, &mime_type);
-       if (pixbuf == NULL) {
-               DEBUG ("couldn't make a pixbuf from avatar; giving up");
-               return;
-       }
-
-       if (avatar->format == NULL) {
-               avatar->format = mime_type;
-       } else if (strcmp (mime_type, avatar->format)) {
-               DEBUG ("avatar had incorrect format! correcting");
-               g_free (avatar->format);
-               avatar->format = mime_type;
-       } else {
-               g_free (mime_type);
-       }
-
-       empathy_avatar_ref (avatar);
-
-       avatar_chooser_set_image (chooser, avatar, pixbuf, set_locally);
+/* The caller must free the strings stored in satisfactory_format_name and
+ * satisfactory_mime_type.
+ */
+static gboolean
+avatar_chooser_need_mime_type_conversion (const gchar *current_mime_type,
+    gchar **accepted_mime_types,
+    gchar **satisfactory_format_name,
+    gchar **satisfactory_mime_type)
+{
+  gchar *good_mime_types[] = {"image/jpeg", "image/png", NULL};
+  guint i;
+  GSList *formats, *l;
+  gboolean found = FALSE;
+
+  *satisfactory_format_name = NULL;
+  *satisfactory_mime_type = NULL;
+
+  /* If there is no accepted format there is nothing we can do */
+  if (accepted_mime_types == NULL || *accepted_mime_types == NULL)
+    return TRUE;
+
+  /* If the current mime type is good and accepted, don't change it!
+   * jpeg is compress better pictures, but png is better for logos and
+   * could have an alpha layer. */
+  if (str_in_strv (current_mime_type, good_mime_types) &&
+      str_in_strv (current_mime_type, accepted_mime_types))
+    {
+      *satisfactory_mime_type = g_strdup (current_mime_type);
+      *satisfactory_format_name = g_strdup (current_mime_type +
+                    strlen ("image/"));
+      return FALSE;
+    }
+
+  /* The current mime type is either not accepted or not good to use.
+   * Check if one of the good format is supported... */
+  for (i = 0; good_mime_types[i] != NULL;  i++)
+    {
+      if (str_in_strv (good_mime_types[i], accepted_mime_types))
+        {
+          *satisfactory_mime_type = g_strdup (good_mime_types[i]);
+          *satisfactory_format_name = g_strdup (good_mime_types[i] +
+              strlen ("image/"));
+          return TRUE;
+        }
+    }
+
+  /* Pick the first supported format we can write */
+  formats = gdk_pixbuf_get_formats ();
+  for (l = formats; !found && l != NULL; l = l->next)
+    {
+      GdkPixbufFormat *format = l->data;
+      gchar **format_mime_types;
+      gchar **iter;
+
+      if (!gdk_pixbuf_format_is_writable (format))
+        continue;
+
+      format_mime_types = gdk_pixbuf_format_get_mime_types (format);
+      for (iter = format_mime_types; *iter != NULL; iter++)
+        {
+          if (str_in_strv (*iter, accepted_mime_types))
+            {
+              *satisfactory_format_name = gdk_pixbuf_format_get_name (format);
+              *satisfactory_mime_type = g_strdup (*iter);
+              found = TRUE;
+              break;
+            }
+        }
+      g_strfreev (format_mime_types);
+    }
+  g_slist_free (formats);
+
+  return TRUE;
 }
 
 static void
-avatar_chooser_set_image (EmpathyAvatarChooser *chooser,
-                         EmpathyAvatar        *avatar,
-                         GdkPixbuf            *pixbuf,
-                         gboolean              set_locally)
+avatar_chooser_error_show (EmpathyAvatarChooser *self,
+    const gchar *primary_text,
+    const gchar *secondary_text)
 {
-       EmpathyAvatarChooserPriv *priv = GET_PRIV (chooser);
-       GdkPixbuf                *pixbuf_view;
-       GtkWidget                *image;
+  GtkWidget *parent;
+  GtkWidget *dialog;
 
-       g_assert (avatar != NULL);
-       g_assert (pixbuf != NULL);
+  parent = gtk_widget_get_toplevel (GTK_WIDGET (self));
+  if (!GTK_IS_WINDOW (parent))
+    parent = NULL;
 
-       if (set_locally) {
-               EmpathyAvatar *conv = avatar_chooser_maybe_convert_and_scale (
-                       chooser, pixbuf, avatar);
-               empathy_avatar_unref (avatar);
+  dialog = gtk_message_dialog_new (parent ? GTK_WINDOW (parent) : NULL,
+      GTK_DIALOG_MODAL,
+      GTK_MESSAGE_WARNING,
+      GTK_BUTTONS_CLOSE,
+      "%s", primary_text);
 
-               if (conv == NULL) {
-                       /* An error occured; don't change the avatar. */
-                       return;
-               }
+  if (secondary_text != NULL)
+    {
+      gtk_message_dialog_format_secondary_text (GTK_MESSAGE_DIALOG (dialog),
+          "%s", secondary_text);
+    }
 
-               avatar = conv;
-       }
+  g_signal_connect (dialog, "response",
+        G_CALLBACK (gtk_widget_destroy), NULL);
+  gtk_widget_show (dialog);
 
-       if (priv->avatar != NULL) {
-               empathy_avatar_unref (priv->avatar);
-       }
-       priv->avatar = avatar;
+}
+
+static TpAvatarRequirements *
+get_requirements (EmpathyAvatarChooser *self)
+{
+  TpConnection *connection;
 
-       pixbuf_view = empathy_pixbuf_scale_down_if_necessary (pixbuf, AVATAR_SIZE_VIEW);
-       image = gtk_image_new_from_pixbuf (pixbuf_view);
+  /* FIXME: Should get on TpProtocol if account is offline */
+  connection = tp_account_get_connection (self->priv->account);
+  return tp_connection_get_avatar_requirements (connection);
+}
 
-       gtk_button_set_image (GTK_BUTTON (chooser), image);
-       g_signal_emit (chooser, signals[CHANGED], 0);
 
-       g_object_unref (pixbuf_view);
-       g_object_unref (pixbuf);
+static gboolean
+avatar_chooser_maybe_convert_and_scale (EmpathyAvatarChooser *self,
+    GdkPixbuf *pixbuf,
+    GArray *avatar,
+    gchar *mime_type,
+    GArray **ret_avatar,
+    gchar **ret_mime_type)
+{
+  TpAvatarRequirements *req;
+  gboolean needs_conversion = FALSE;
+  guint width, height;
+  gchar *new_format_name = NULL;
+  gchar *new_mime_type = NULL;
+  gdouble min_factor, max_factor;
+  gdouble factor;
+  gchar *best_image_data = NULL;
+  gsize best_image_size = 0;
+  guint count = 0;
+
+  g_assert (ret_avatar != NULL);
+  g_assert (ret_mime_type != NULL);
+
+  req = get_requirements (self);
+  if (req == NULL)
+    {
+      DEBUG ("Avatar requirements not ready");
+      return FALSE;
+    }
+
+  /* Smaller is the factor, smaller will be the image.
+   * 0 is an empty image, 1 is the full size. */
+  min_factor = 0;
+  max_factor = 1;
+  factor = 1;
+
+  /* Check if we need to convert to another image format */
+  if (avatar_chooser_need_mime_type_conversion (mime_type,
+        req->supported_mime_types, &new_format_name, &new_mime_type))
+    {
+      DEBUG ("Format conversion needed, we'll use mime type '%s' "
+             "and format name '%s'. Current mime type is '%s'",
+             new_mime_type, new_format_name, mime_type);
+      needs_conversion = TRUE;
+    }
+
+  /* If there is no format we can use, report error to the user. */
+  if (new_mime_type == NULL || new_format_name == NULL)
+    {
+      avatar_chooser_error_show (self, _("Couldn't convert image"),
+          _("None of the accepted image formats are "
+            "supported on your system"));
+      return FALSE;
+    }
+
+  /* If width or height are too big, it needs converting. */
+  width = gdk_pixbuf_get_width (pixbuf);
+  height = gdk_pixbuf_get_height (pixbuf);
+  if ((req->maximum_width > 0 && width > req->maximum_width) ||
+      (req->maximum_height > 0 && height > req->maximum_height))
+    {
+      gdouble h_factor, v_factor;
+
+      h_factor = (gdouble) req->maximum_width / width;
+      v_factor = (gdouble) req->maximum_height / height;
+      factor = max_factor = MIN (h_factor, v_factor);
+
+      DEBUG ("Image dimensions (%dx%d) are too big. Max is %dx%d.",
+          width, height, req->maximum_width, req->maximum_height);
+
+      needs_conversion = TRUE;
+    }
+
+  /* If the data len is too big and no other conversion is needed,
+   * try with a lower factor. */
+  if (req->maximum_bytes > 0 && avatar->len > req->maximum_bytes &&
+      !needs_conversion)
+    {
+      DEBUG ("Image data (%u bytes) is too big "
+             "(max is %u bytes), conversion needed.",
+             avatar->len, req->maximum_bytes);
+
+      factor = 0.5;
+      needs_conversion = TRUE;
+    }
+
+  /* If no conversion is needed, return the avatar */
+  if (!needs_conversion)
+    {
+      *ret_avatar = g_array_ref (avatar);
+      *ret_mime_type = g_strdup (mime_type);
+      return TRUE;
+    }
+
+  do
+    {
+      GdkPixbuf *pixbuf_scaled = NULL;
+      gboolean saved;
+      gint new_width, new_height;
+      gchar *converted_image_data;
+      gsize converted_image_size;
+      GError *error = NULL;
+
+      if (factor != 1)
+        {
+          new_width = width * factor;
+          new_height = height * factor;
+          pixbuf_scaled = gdk_pixbuf_scale_simple (pixbuf,
+                     new_width,
+                     new_height,
+                     GDK_INTERP_HYPER);
+        }
+      else
+        {
+          new_width = width;
+          new_height = height;
+          pixbuf_scaled = g_object_ref (pixbuf);
+        }
+
+      DEBUG ("Trying with factor %f (%dx%d) and format %s...", factor,
+        new_width, new_height, new_format_name);
+
+      saved = gdk_pixbuf_save_to_buffer (pixbuf_scaled,
+          &converted_image_data,
+          &converted_image_size,
+          new_format_name,
+          &error, NULL);
+      g_object_unref (pixbuf_scaled);
+
+      if (!saved)
+        {
+          g_free (new_format_name);
+          g_free (new_mime_type);
+          avatar_chooser_error_show (self,
+            _("Couldn't convert image"),
+            error ? error->message : NULL);
+          g_clear_error (&error);
+          return FALSE;
+        }
+
+      DEBUG ("Produced an image data of %"G_GSIZE_FORMAT" bytes.",
+        converted_image_size);
+
+      /* If the new image satisfy the req, keep it as current best */
+      if (req->maximum_bytes == 0 || converted_image_size <= req->maximum_bytes)
+        {
+          g_free (best_image_data);
+
+          best_image_data = converted_image_data;
+          best_image_size = converted_image_size;
+
+          /* If this image is close enough to the optimal size,
+           * stop searching */
+          if (req->maximum_bytes == 0 ||
+              req->maximum_bytes - converted_image_size <= 1024)
+            break;
+        }
+      else
+        {
+          g_free (converted_image_data);
+        }
+
+    /* Make a binary search for the bigest factor that produce
+     * an image data size less than max_size */
+    if (converted_image_size > req->maximum_bytes)
+      max_factor = factor;
+    if (converted_image_size < req->maximum_bytes)
+      min_factor = factor;
+    factor = (min_factor + max_factor)/2;
+
+    if ((int) (width * factor) == new_width ||
+        (int) (height * factor) == new_height)
+      {
+        /* min_factor and max_factor are too close, so the new
+         * factor will produce the same image as previous
+         * iteration. No need to continue, we already found
+         * the optimal size. */
+        break;
+      }
+
+    /* Do 10 iterations in the worst case */
+  } while (++count < 10);
+
+  g_free (new_format_name);
+
+  /* FIXME: there is no way to create a GArray with zero copy? */
+  *ret_avatar = g_array_sized_new (FALSE, FALSE, sizeof (gchar),
+      best_image_size);
+  g_array_append_vals (*ret_avatar, best_image_data, best_image_size);
+  g_free (best_image_data);
+
+  *ret_mime_type = new_mime_type;
+
+  return TRUE;
 }
 
+/* Take ownership of @pixbuf */
 static void
-avatar_chooser_set_image_from_file (EmpathyAvatarChooser *chooser,
-                                   const gchar          *filename)
+avatar_chooser_set_image (EmpathyAvatarChooser *self,
+    GArray *avatar,
+    gchar *mime_type,
+    GdkPixbuf *pixbuf,
+    gboolean maybe_convert)
 {
-       gchar  *image_data = NULL;
-       gsize   image_size = 0;
-       GError *error = NULL;
+  GdkPixbuf *pixbuf_view;
+  GtkWidget *image;
 
-       if (!g_file_get_contents (filename, &image_data, &image_size, &error)) {
-               DEBUG ("Failed to load image from '%s': %s", filename,
-                       error ? error->message : "No error given");
+  g_assert (avatar != NULL);
+  g_assert (pixbuf != NULL);
 
-               g_clear_error (&error);
-               return;
-       }
+  if (maybe_convert)
+    {
+      GArray *conv_avatar = NULL;
+      gchar *conv_mime_type = NULL;
 
-       avatar_chooser_set_image_from_data (chooser, image_data, image_size, TRUE);
-}
+      if (!avatar_chooser_maybe_convert_and_scale (self,
+              pixbuf, avatar, mime_type, &conv_avatar, &conv_mime_type))
+        return;
 
-static gboolean
-avatar_chooser_drag_motion_cb (GtkWidget          *widget,
-                             GdkDragContext     *context,
-                             gint                x,
-                             gint                y,
-                             guint               time,
-                             EmpathyAvatarChooser *chooser)
-{
-       EmpathyAvatarChooserPriv *priv;
-       GList                  *p;
+      /* Transfer ownership */
+      tp_clear_pointer (&self->priv->avatar, g_array_unref);
+      self->priv->avatar = conv_avatar;
 
-       priv = GET_PRIV (chooser);
+      g_free (self->priv->mime_type);
+      self->priv->mime_type = conv_mime_type;
+    }
+  else
+    {
+      tp_clear_pointer (&self->priv->avatar, g_array_unref);
+      self->priv->avatar = g_array_ref (avatar);
 
-       for (p = context->targets; p != NULL; p = p->next) {
-               gchar *possible_type;
+      g_free (self->priv->mime_type);
+      self->priv->mime_type = g_strdup (mime_type);
+    }
 
-               possible_type = gdk_atom_name (GDK_POINTER_TO_ATOM (p->data));
+  self->priv->changed = TRUE;
 
-               if (!strcmp (possible_type, URI_LIST_TYPE)) {
-                       g_free (possible_type);
-                       gdk_drag_status (context, GDK_ACTION_COPY, time);
+  pixbuf_view = empathy_pixbuf_scale_down_if_necessary (pixbuf,
+      AVATAR_SIZE_VIEW);
+  image = gtk_image_new_from_pixbuf (pixbuf_view);
 
-                       return TRUE;
-               }
+  gtk_button_set_image (GTK_BUTTON (self), image);
 
-               g_free (possible_type);
-       }
+  g_object_unref (pixbuf_view);
+  g_object_unref (pixbuf);
+}
 
-       return FALSE;
+/* takes ownership of @data */
+static void
+avatar_chooser_set_image_from_data (EmpathyAvatarChooser *self,
+    gchar *data,
+    gsize size)
+{
+  GdkPixbuf *pixbuf;
+  GArray *avatar;
+  gchar *mime_type = NULL;
+
+  if (data == NULL)
+    {
+      avatar_chooser_clear_image (self);
+      return;
+    }
+
+  pixbuf = empathy_pixbuf_from_data_and_mime (data, size, &mime_type);
+  if (pixbuf == NULL)
+    {
+      g_free (data);
+      return;
+    }
+
+  /* FIXME: there is no way to create a GArray with zero copy? */
+  avatar = g_array_sized_new (FALSE, FALSE, sizeof (gchar), size);
+  g_array_append_vals (avatar, data, size);
+
+  avatar_chooser_set_image (self, avatar, mime_type, pixbuf, TRUE);
+
+  g_free (mime_type);
+  g_array_unref (avatar);
+  g_free (data);
 }
 
 static void
-avatar_chooser_drag_leave_cb (GtkWidget          *widget,
-                            GdkDragContext     *context,
-                            guint               time,
-                            EmpathyAvatarChooser *chooser)
+avatar_chooser_drag_data_received_cb (GtkWidget          *widget,
+    GdkDragContext *context,
+    gint x,
+    gint y,
+    GtkSelectionData *selection_data,
+    guint info,
+    guint time_,
+    EmpathyAvatarChooser *self)
 {
+  gchar *target_type;
+  gboolean handled = FALSE;
+
+  target_type = gdk_atom_name (gtk_selection_data_get_target (selection_data));
+  if (!strcmp (target_type, URI_LIST_TYPE))
+    {
+      GFile *file;
+      gchar *nl;
+      gchar *data = NULL;
+      gsize bytes_read;
+
+      nl = strstr ((gchar *) gtk_selection_data_get_data (selection_data),
+              "\r\n");
+      if (nl != NULL)
+        {
+          gchar *uri;
+
+          uri = g_strndup (
+              (gchar *) gtk_selection_data_get_data (selection_data),
+              nl - (gchar *) gtk_selection_data_get_data (selection_data));
+
+          file = g_file_new_for_uri (uri);
+          g_free (uri);
+        }
+      else
+        {
+          file = g_file_new_for_uri ((gchar *) gtk_selection_data_get_data (
+                selection_data));
+        }
+
+      handled = g_file_load_contents (file, NULL, &data, &bytes_read,
+              NULL, NULL);
+
+      if (handled)
+        {
+          /* pass data to the avatar_chooser_set_image_from_data */
+          avatar_chooser_set_image_from_data (self, data, bytes_read);
+        }
+
+      g_object_unref (file);
+    }
+
+  gtk_drag_finish (context, handled, FALSE, time_);
 }
 
-static gboolean
-avatar_chooser_drag_drop_cb (GtkWidget          *widget,
-                           GdkDragContext     *context,
-                           gint                x,
-                           gint                y,
-                           guint               time,
-                           EmpathyAvatarChooser *chooser)
+static void
+avatar_chooser_update_preview_cb (GtkFileChooser *file_chooser,
+    EmpathyAvatarChooser *self)
 {
-       EmpathyAvatarChooserPriv *priv;
-       GList                  *p;
+  gchar *filename;
 
-       priv = GET_PRIV (chooser);
+  filename = gtk_file_chooser_get_preview_filename (file_chooser);
 
-       if (context->targets == NULL) {
-               return FALSE;
-       }
+  if (filename != NULL)
+    {
+      GtkWidget *image;
+      GdkPixbuf *pixbuf = NULL;
+      GdkPixbuf *scaled_pixbuf;
 
-       for (p = context->targets; p != NULL; p = p->next) {
-               char *possible_type;
+      pixbuf = gdk_pixbuf_new_from_file (filename, NULL);
 
-               possible_type = gdk_atom_name (GDK_POINTER_TO_ATOM (p->data));
-               if (!strcmp (possible_type, URI_LIST_TYPE)) {
-                       g_free (possible_type);
-                       gtk_drag_get_data (widget, context,
-                                          GDK_POINTER_TO_ATOM (p->data),
-                                          time);
+      image = gtk_file_chooser_get_preview_widget (file_chooser);
 
-                       return TRUE;
-               }
+      if (pixbuf != NULL)
+        {
+          scaled_pixbuf = empathy_pixbuf_scale_down_if_necessary (pixbuf,
+              AVATAR_SIZE_SAVE);
 
-               g_free (possible_type);
-       }
+          gtk_image_set_from_pixbuf (GTK_IMAGE (image), scaled_pixbuf);
+          g_object_unref (scaled_pixbuf);
+          g_object_unref (pixbuf);
+        }
+      else
+        {
+          gtk_image_set_from_stock (GTK_IMAGE (image),
+                  "dialog-question",
+                  GTK_ICON_SIZE_DIALOG);
+        }
 
-       return FALSE;
+      g_free (filename);
+    }
+
+  gtk_file_chooser_set_preview_widget_active (file_chooser, TRUE);
 }
 
 static void
-avatar_chooser_drag_data_received_cb (GtkWidget          *widget,
-                                    GdkDragContext     *context,
-                                    gint                x,
-                                    gint                y,
-                                    GtkSelectionData   *selection_data,
-                                    guint               info,
-                                    guint               time,
-                                    EmpathyAvatarChooser *chooser)
-{
-       gchar    *target_type;
-       gboolean  handled = FALSE;
-
-       target_type = gdk_atom_name (selection_data->target);
-       if (!strcmp (target_type, URI_LIST_TYPE)) {
-               GFile            *file;
-               GFileInputStream *input_stream;
-               gchar            *nl;
-               gchar            *data = NULL;
-
-               nl = strstr (selection_data->data, "\r\n");
-               if (nl) {
-                       gchar *uri;
-
-                       uri = g_strndup (selection_data->data,
-                                        nl - (gchar*) selection_data->data);
-
-                       file = g_file_new_for_uri (uri);
-                       g_free (uri);
-               } else {
-                       file = g_file_new_for_uri (selection_data->data);
-               }
-
-               input_stream = g_file_read (file, NULL, NULL);
-
-               if (input_stream != NULL) {
-                       GFileInfo *info;
-                       
-                       info = g_file_query_info (file,
-                                                 G_FILE_ATTRIBUTE_STANDARD_SIZE,
-                                                 0, NULL, NULL);
-                       if (info != NULL) {
-                               goffset size;
-                               gssize bytes_read;
-                               
-                               size = g_file_info_get_size (info);
-                               data = g_malloc (size);
-
-                               bytes_read = g_input_stream_read (G_INPUT_STREAM (input_stream),
-                                                                 data, size,
-                                                                 NULL, NULL);
-                               if (bytes_read != -1) {
-                                       avatar_chooser_set_image_from_data (
-                                               chooser, data,
-                                               (gsize) bytes_read,
-                                               TRUE);
-                                       handled = TRUE;
-                               }
-
-                               g_free (data);
-                               g_object_unref (info);
-                       }
-
-                       g_object_unref (input_stream);
-               }
-               
-               g_object_unref (file);
-       }
-
-       gtk_drag_finish (context, handled, FALSE, time);
+avatar_chooser_set_image_from_file (EmpathyAvatarChooser *self,
+    const gchar *filename)
+{
+  gchar *image_data = NULL;
+  gsize  image_size = 0;
+  GError *error = NULL;
+
+  if (!g_file_get_contents (filename, &image_data, &image_size, &error))
+    {
+      DEBUG ("Failed to load image from '%s': %s", filename,
+        error ? error->message : "No error given");
+
+      g_clear_error (&error);
+      return;
+    }
+
+  /* pass image_data to the avatar_chooser_set_image_from_data */
+  avatar_chooser_set_image_from_data (self, image_data, image_size);
 }
 
+#ifdef HAVE_CHEESE
 static void
-avatar_chooser_update_preview_cb (GtkFileChooser       *file_chooser,
-                                 EmpathyAvatarChooser *chooser)
+avatar_chooser_set_avatar_from_pixbuf (EmpathyAvatarChooser *self,
+               GdkPixbuf *pb)
 {
-       gchar *filename;
-
-       filename = gtk_file_chooser_get_preview_filename (file_chooser);
+  gsize size;
+  gchar *buf;
+  GArray *avatar;
+  GError *error = NULL;
+
+  if (!gdk_pixbuf_save_to_buffer (pb, &buf, &size, "png", &error, NULL))
+    {
+      avatar_chooser_error_show (self,
+        _("Couldn't save picture to file"),
+        error ? error->message : NULL);
+      g_clear_error (&error);
+      return;
+    }
+
+  /* FIXME: there is no way to create a GArray with zero copy? */
+  avatar = g_array_sized_new (FALSE, FALSE, sizeof (gchar), size);
+  g_array_append_vals (avatar, buf, size);
+
+  avatar_chooser_set_image (self, avatar, "image/png", pb, TRUE);
+
+  g_free (buf);
+  g_array_unref (avatar);
+}
 
-       if (filename) {
-               GtkWidget *image;
-               GdkPixbuf *pixbuf = NULL;
-               GdkPixbuf *scaled_pixbuf;
+static gboolean
+destroy_chooser (GtkWidget *self)
+{
+  gtk_widget_destroy (self);
+  return FALSE;
+}
 
-               pixbuf = gdk_pixbuf_new_from_file (filename, NULL);
+static void
+webcam_response_cb (GtkDialog *dialog,
+        int response,
+        EmpathyAvatarChooser *self)
+{
+  if (response == GTK_RESPONSE_ACCEPT)
+    {
+      GdkPixbuf *pb;
+      CheeseAvatarChooser *cheese_chooser;
+
+      cheese_chooser = CHEESE_AVATAR_CHOOSER (dialog);
+      pb = cheese_avatar_chooser_get_picture (cheese_chooser);
+      avatar_chooser_set_avatar_from_pixbuf (self, pb);
+    }
+
+  if (response != GTK_RESPONSE_DELETE_EVENT &&
+      response != GTK_RESPONSE_NONE)
+    g_idle_add ((GSourceFunc) destroy_chooser, dialog);
+}
 
-               image = gtk_file_chooser_get_preview_widget (file_chooser);
+static void
+choose_avatar_from_webcam (GtkWidget *widget,
+    EmpathyAvatarChooser *self)
+{
+  GtkWidget *window;
 
-               if (pixbuf) {
-                       scaled_pixbuf = empathy_pixbuf_scale_down_if_necessary (pixbuf, AVATAR_SIZE_SAVE);
-                       gtk_image_set_from_pixbuf (GTK_IMAGE (image), scaled_pixbuf);
-                       g_object_unref (scaled_pixbuf);
-                       g_object_unref (pixbuf);
-               } else {
-                       gtk_image_set_from_stock (GTK_IMAGE (image),
-                                                 "gtk-dialog-question",
-                                                 GTK_ICON_SIZE_DIALOG);
-               }
-       }
+  window = cheese_avatar_chooser_new ();
 
-       gtk_file_chooser_set_preview_widget_active (file_chooser, TRUE);
+  gtk_window_set_transient_for (GTK_WINDOW (window),
+      GTK_WINDOW (empathy_get_toplevel_window (GTK_WIDGET (self))));
+  gtk_window_set_modal (GTK_WINDOW (window), TRUE);
+  g_signal_connect (G_OBJECT (window), "response",
+      G_CALLBACK (webcam_response_cb), self);
+  gtk_widget_show (window);
 }
+#endif /* HAVE_CHEESE */
 
 static void
-avatar_chooser_response_cb (GtkWidget            *widget,
-                           gint                  response,
-                           EmpathyAvatarChooser *chooser)
-{
-       EmpathyAvatarChooserPriv *priv = GET_PRIV (chooser);
-
-       if (response == GTK_RESPONSE_CANCEL) {
-               goto out;
-       }
-
-       /* Check if we went non-ready since displaying the dialog. */
-       if (!empathy_tp_contact_factory_is_ready (priv->tp_contact_factory)) {
-               DEBUG ("Can't set avatar when contact factory isn't ready.");
-               goto out;
-       }
-
-       if (response == GTK_RESPONSE_OK) {
-               gchar *filename;
-               gchar *path;
-
-               filename = gtk_file_chooser_get_filename (GTK_FILE_CHOOSER (widget));
-               avatar_chooser_set_image_from_file (chooser, filename);
-               g_free (filename);
-
-               path = gtk_file_chooser_get_current_folder (GTK_FILE_CHOOSER (widget));
-               if (path) {
-                       empathy_conf_set_string (empathy_conf_get (),
-                                                EMPATHY_PREFS_UI_AVATAR_DIRECTORY,
-                                                path);
-                       g_free (path);
-               }
-       }
-       else if (response == GTK_RESPONSE_NO) {
-               /* This corresponds to "No Image", not to "Cancel" */
-               avatar_chooser_clear_image (chooser);
-       }
+avatar_chooser_response_cb (GtkWidget *widget,
+    gint response,
+    EmpathyAvatarChooser *self)
+{
+  self->priv->chooser_dialog = NULL;
+
+  if (response == EMPATHY_AVATAR_CHOOSER_RESPONSE_FILE)
+    {
+      gchar *filename;
+      gchar *path;
+
+      filename = gtk_file_chooser_get_filename (GTK_FILE_CHOOSER (widget));
+      avatar_chooser_set_image_from_file (self, filename);
+      g_free (filename);
+
+      path = gtk_file_chooser_get_current_folder (GTK_FILE_CHOOSER (widget));
+      if (path != NULL)
+        {
+          g_settings_set_string (self->priv->gsettings_ui,
+                     EMPATHY_PREFS_UI_AVATAR_DIRECTORY,
+                     path);
+
+          g_free (path);
+        }
+    }
+  else if (response == EMPATHY_AVATAR_CHOOSER_RESPONSE_NO_IMAGE)
+    {
+      /* This corresponds to "No Image", not to "Cancel" */
+      avatar_chooser_clear_image (self);
+    }
+  #ifdef HAVE_CHEESE
+  else if (response == EMPATHY_AVATAR_CHOOSER_RESPONSE_WEBCAM)
+    {
+      /* This corresponds to "Camera Picture" */
+      choose_avatar_from_webcam (widget, self);
+    }
+  #endif
+
+  gtk_widget_destroy (widget);
+}
 
-out:
-       gtk_widget_destroy (widget);
+static void
+avatar_chooser_clicked_cb (GtkWidget *button,
+    EmpathyAvatarChooser *self)
+{
+  GtkFileChooser *chooser_dialog;
+  GtkWidget *image;
+  gchar *saved_dir = NULL;
+  const gchar *default_dir = DEFAULT_DIR;
+  const gchar *pics_dir;
+  GtkFileFilter *filter;
+#ifdef HAVE_CHEESE
+  GtkWidget *picture_button;
+  EmpathyCameraMonitor *monitor;
+#endif
+
+  if (self->priv->chooser_dialog != NULL)
+    {
+      gtk_window_present (GTK_WINDOW (self->priv->chooser_dialog));
+      return;
+    }
+
+  self->priv->chooser_dialog = GTK_FILE_CHOOSER (
+      gtk_file_chooser_dialog_new (_("Select Your Avatar Image"),
+        empathy_get_toplevel_window (GTK_WIDGET (self)),
+        GTK_FILE_CHOOSER_ACTION_OPEN,
+        NULL, NULL));
+
+#ifdef HAVE_CHEESE
+  picture_button = gtk_dialog_add_button (
+      GTK_DIALOG (self->priv->chooser_dialog),
+      _("Take a picture..."), EMPATHY_AVATAR_CHOOSER_RESPONSE_WEBCAM);
+
+  /* Button is sensitive only if there is one camera connected */
+  monitor = empathy_camera_monitor_dup_singleton ();
+
+  g_object_set_data_full (G_OBJECT (picture_button),
+      "monitor", monitor, g_object_unref);
+
+  g_object_bind_property (monitor, "available", picture_button, "sensitive",
+      G_BINDING_SYNC_CREATE);
+#endif
+
+  gtk_dialog_add_buttons (GTK_DIALOG (self->priv->chooser_dialog),
+      _("No Image"), EMPATHY_AVATAR_CHOOSER_RESPONSE_NO_IMAGE,
+      GTK_STOCK_CANCEL, EMPATHY_AVATAR_CHOOSER_RESPONSE_CANCEL,
+      GTK_STOCK_OPEN, EMPATHY_AVATAR_CHOOSER_RESPONSE_FILE,
+      NULL);
+
+  chooser_dialog = self->priv->chooser_dialog;
+  gtk_window_set_destroy_with_parent (GTK_WINDOW (chooser_dialog), TRUE);
+
+  /* Get special dirs */
+  saved_dir = g_settings_get_string (self->priv->gsettings_ui,
+             EMPATHY_PREFS_UI_AVATAR_DIRECTORY);
+
+  if (saved_dir != NULL &&
+      !g_file_test (saved_dir, G_FILE_TEST_IS_DIR))
+    {
+      g_free (saved_dir);
+      saved_dir = NULL;
+    }
+
+  if (!g_file_test (default_dir, G_FILE_TEST_IS_DIR))
+    default_dir = NULL;
+
+  pics_dir = g_get_user_special_dir (G_USER_DIRECTORY_PICTURES);
+  if (pics_dir != NULL && !g_file_test (pics_dir, G_FILE_TEST_IS_DIR))
+    pics_dir = NULL;
+
+  /* Set current dir to the last one or to DEFAULT_DIR or to home */
+  if (saved_dir != NULL)
+    gtk_file_chooser_set_current_folder (chooser_dialog, saved_dir);
+  else if (pics_dir != NULL)
+    gtk_file_chooser_set_current_folder (chooser_dialog, pics_dir);
+  else if (default_dir != NULL)
+    gtk_file_chooser_set_current_folder (chooser_dialog, default_dir);
+  else
+    gtk_file_chooser_set_current_folder (chooser_dialog, g_get_home_dir ());
+
+  /* Add shortcuts to special dirs */
+  if (saved_dir)
+    gtk_file_chooser_add_shortcut_folder (chooser_dialog, saved_dir, NULL);
+  else if (pics_dir)
+    gtk_file_chooser_add_shortcut_folder (chooser_dialog, pics_dir, NULL);
+
+  if (default_dir != NULL)
+    gtk_file_chooser_add_shortcut_folder (chooser_dialog, default_dir, NULL);
+
+  /* Setup preview image */
+  image = gtk_image_new ();
+  gtk_file_chooser_set_preview_widget (chooser_dialog, image);
+  gtk_widget_set_size_request (image, AVATAR_SIZE_SAVE, AVATAR_SIZE_SAVE);
+  gtk_widget_show (image);
+  gtk_file_chooser_set_use_preview_label (chooser_dialog, FALSE);
+  g_signal_connect (chooser_dialog, "update-preview",
+      G_CALLBACK (avatar_chooser_update_preview_cb),
+      self);
+
+  /* Setup filers */
+  filter = gtk_file_filter_new ();
+  gtk_file_filter_set_name (filter, _("Images"));
+  gtk_file_filter_add_pixbuf_formats (filter);
+  gtk_file_chooser_add_filter (chooser_dialog, filter);
+  filter = gtk_file_filter_new ();
+  gtk_file_filter_set_name (filter, _("All Files"));
+  gtk_file_filter_add_pattern (filter, "*");
+  gtk_file_chooser_add_filter (chooser_dialog, filter);
+
+  /* Setup response */
+  gtk_dialog_set_default_response (GTK_DIALOG (chooser_dialog),
+      EMPATHY_AVATAR_CHOOSER_RESPONSE_FILE);
+
+  g_signal_connect (chooser_dialog, "response",
+      G_CALLBACK (avatar_chooser_response_cb),
+      self);
+
+  gtk_widget_show (GTK_WIDGET (chooser_dialog));
+
+  g_free (saved_dir);
 }
 
 static void
-avatar_chooser_clicked_cb (GtkWidget            *button,
-                          EmpathyAvatarChooser *chooser)
-{
-       GtkFileChooser *chooser_dialog;
-       GtkWidget      *image;
-       gchar          *saved_dir = NULL;
-       const gchar    *default_dir = DEFAULT_DIR;
-       const gchar    *pics_dir;
-       GtkFileFilter  *filter;
-
-       chooser_dialog = GTK_FILE_CHOOSER (
-               gtk_file_chooser_dialog_new (_("Select Your Avatar Image"),
-                                            empathy_get_toplevel_window (GTK_WIDGET (chooser)),
-                                            GTK_FILE_CHOOSER_ACTION_OPEN,
-                                            _("No Image"),
-                                            GTK_RESPONSE_NO,
-                                            GTK_STOCK_CANCEL,
-                                            GTK_RESPONSE_CANCEL,
-                                            GTK_STOCK_OPEN,
-                                            GTK_RESPONSE_OK,
-                                            NULL));
-
-       /* Get special dirs */
-       empathy_conf_get_string (empathy_conf_get (),
-                                EMPATHY_PREFS_UI_AVATAR_DIRECTORY,
-                                &saved_dir);
-       if (saved_dir && !g_file_test (saved_dir, G_FILE_TEST_IS_DIR)) {
-               g_free (saved_dir);
-               saved_dir = NULL;
-       }
-       if (!g_file_test (default_dir, G_FILE_TEST_IS_DIR)) {
-               default_dir = NULL;
-       }
-       pics_dir = g_get_user_special_dir (G_USER_DIRECTORY_PICTURES);
-       if (pics_dir && !g_file_test (pics_dir, G_FILE_TEST_IS_DIR)) {
-               pics_dir = NULL;
-       }
-
-       /* Set current dir to the last one or to DEFAULT_DIR or to home */
-       if (saved_dir) {
-               gtk_file_chooser_set_current_folder (chooser_dialog, saved_dir);
-       }
-       else if (pics_dir) {
-               gtk_file_chooser_set_current_folder (chooser_dialog, pics_dir);
-       }
-       else if (default_dir) {
-               gtk_file_chooser_set_current_folder (chooser_dialog, default_dir);
-       } else {
-               gtk_file_chooser_set_current_folder (chooser_dialog, g_get_home_dir ());
-       }
-
-       /* Add shortcuts to special dirs */
-       if (saved_dir) {
-               gtk_file_chooser_add_shortcut_folder (chooser_dialog, saved_dir, NULL);
-       }
-       else if (pics_dir) {
-               gtk_file_chooser_add_shortcut_folder (chooser_dialog, pics_dir, NULL);
-       }
-       if (default_dir) {
-               gtk_file_chooser_add_shortcut_folder (chooser_dialog, default_dir, NULL);
-       }
-
-       /* Setup preview image */
-       image = gtk_image_new ();
-       gtk_file_chooser_set_preview_widget (chooser_dialog, image);
-       gtk_widget_set_size_request (image, AVATAR_SIZE_SAVE, AVATAR_SIZE_SAVE);
-       gtk_widget_show (image);
-       gtk_file_chooser_set_use_preview_label (chooser_dialog, FALSE);
-       g_signal_connect (chooser_dialog, "update-preview",
-                         G_CALLBACK (avatar_chooser_update_preview_cb),
-                         chooser);
-
-       /* Setup filers */
-       filter = gtk_file_filter_new ();
-       gtk_file_filter_set_name (filter, _("Images"));
-       gtk_file_filter_add_pixbuf_formats (filter);
-       gtk_file_chooser_add_filter (chooser_dialog, filter);
-       filter = gtk_file_filter_new ();
-       gtk_file_filter_set_name (filter, _("All Files"));
-       gtk_file_filter_add_pattern(filter, "*");
-       gtk_file_chooser_add_filter (chooser_dialog, filter);
-
-       /* Setup response */
-       gtk_dialog_set_default_response (GTK_DIALOG (chooser_dialog), GTK_RESPONSE_OK);
-       g_signal_connect (chooser_dialog, "response",
-                         G_CALLBACK (avatar_chooser_response_cb),
-                         chooser);
-
-       gtk_widget_show (GTK_WIDGET (chooser_dialog));
-       g_free (saved_dir);
+empathy_avatar_chooser_init (EmpathyAvatarChooser *self)
+{
+  self->priv = G_TYPE_INSTANCE_GET_PRIVATE (self,
+    EMPATHY_TYPE_AVATAR_CHOOSER, EmpathyAvatarChooserPrivate);
+
+  gtk_drag_dest_set (GTK_WIDGET (self),
+      GTK_DEST_DEFAULT_ALL,
+      drop_types,
+      G_N_ELEMENTS (drop_types),
+      GDK_ACTION_COPY);
+
+  self->priv->gsettings_ui = g_settings_new (EMPATHY_PREFS_UI_SCHEMA);
+
+  g_signal_connect (self, "drag-motion",
+      G_CALLBACK (avatar_chooser_drag_motion_cb),
+      self);
+  g_signal_connect (self, "drag-drop",
+      G_CALLBACK (avatar_chooser_drag_drop_cb),
+      self);
+  g_signal_connect (self, "drag-data-received",
+      G_CALLBACK (avatar_chooser_drag_data_received_cb),
+      self);
+  g_signal_connect (self, "clicked",
+      G_CALLBACK (avatar_chooser_clicked_cb),
+      self);
+
+  avatar_chooser_clear_image (self);
 }
 
+/**
+ * empathy_avatar_chooser_new:
+ * @account: a #TpAccount
+ *
+ * Creates a new #EmpathyAvatarChooser.
+ *
+ * Return value: a new #EmpathyAvatarChooser
+ */
 GtkWidget *
-empathy_avatar_chooser_new ()
+empathy_avatar_chooser_new (TpAccount *account)
 {
-       return g_object_new (EMPATHY_TYPE_AVATAR_CHOOSER, NULL);
+  g_return_val_if_fail (TP_IS_ACCOUNT (account), NULL);
+
+  return g_object_new (EMPATHY_TYPE_AVATAR_CHOOSER,
+      "account", account,
+      NULL);
 }
 
-void
-empathy_avatar_chooser_set (EmpathyAvatarChooser *chooser,
-                           EmpathyAvatar        *avatar)
+static void
+set_avatar_cb (GObject *source,
+    GAsyncResult *result,
+    gpointer user_data)
 {
-       g_return_if_fail (EMPATHY_IS_AVATAR_CHOOSER (chooser));
+  GSimpleAsyncResult *my_result = user_data;
+  GError *error = NULL;
 
-       if (avatar != NULL) {
-               avatar_chooser_set_image_from_avatar (chooser, avatar, FALSE);
-       } else {
-               avatar_chooser_clear_image (chooser);
-       }
+  if (!tp_account_set_avatar_finish (TP_ACCOUNT (source), result, &error))
+    g_simple_async_result_take_error (my_result, error);
+
+  g_simple_async_result_complete (my_result);
+  g_object_unref (my_result);
 }
 
 void
-empathy_avatar_chooser_get_image_data (EmpathyAvatarChooser  *chooser,
-                                      const gchar          **data,
-                                      gsize                 *data_size,
-                                      const gchar          **mime_type)
-{
-       EmpathyAvatarChooserPriv *priv;
-
-       g_return_if_fail (EMPATHY_IS_AVATAR_CHOOSER (chooser));
-
-       priv = GET_PRIV (chooser);
-
-       if (priv->avatar != NULL) {
-               if (data != NULL) {
-                       *data = priv->avatar->data;
-               }
-               if (data_size != NULL) {
-                       *data_size = priv->avatar->len;
-               }
-               if (mime_type != NULL) {
-                       *mime_type = priv->avatar->format;
-               }
-       } else {
-               if (data != NULL) {
-                       *data = NULL;
-               }
-               if (data_size != NULL) {
-                       *data_size = 0;
-               }
-               if (mime_type != NULL) {
-                       *mime_type = NULL;
-               }
-       }
+empathy_avatar_chooser_apply_async (EmpathyAvatarChooser *self,
+    GAsyncReadyCallback callback,
+    gpointer user_data)
+{
+  GSimpleAsyncResult *result;
+
+  g_return_if_fail (EMPATHY_IS_AVATAR_CHOOSER (self));
+
+  result = g_simple_async_result_new ((GObject *) self, callback, user_data,
+      empathy_avatar_chooser_apply_async);
+
+  if (!self->priv->changed)
+    {
+      g_simple_async_result_complete_in_idle (result);
+      g_object_unref (result);
+      return;
+    }
+
+  self->priv->changed = FALSE;
+
+  DEBUG ("%s Account.Avatar on %s", self->priv->avatar != NULL ? "Set": "Clear",
+      tp_proxy_get_object_path (self->priv->account));
+
+  tp_account_set_avatar_async (self->priv->account,
+      self->priv->avatar != NULL ? (guchar *) self->priv->avatar->data : NULL,
+      self->priv->avatar != NULL ? self->priv->avatar->len : 0,
+      self->priv->mime_type, set_avatar_cb, result);
 }
 
+gboolean
+empathy_avatar_chooser_apply_finish (EmpathyAvatarChooser *self,
+    GAsyncResult *result,
+    GError **error)
+{
+  empathy_implement_finish_void (self, empathy_avatar_chooser_apply_async);
+}