[evolution] Simplify em_utils_forward_message() arguments.

[Matthew Barnes <mbarnes src gnome org>]

commit 540acdbfa95d29a66ad94bad68222b5ae9c98720
Author: Matthew Barnes <mbarnes redhat com>
Date:   Sun Jun 2 09:23:58 2013 -0400

    Simplify em_utils_forward_message() arguments.
    
    Replace the EShell and CamelSession arguments with a single EMailBackend
    argument, from which both the EShell and CamelSession can be obtained.

 mail/em-composer-utils.c                 |   44 ++++++++++++++---------------
 mail/em-composer-utils.h                 |    5 +--
 modules/mail/e-mail-attachment-handler.c |   36 +++++++++---------------
 3 files changed, 37 insertions(+), 48 deletions(-)
---
diff --git a/mail/em-composer-utils.c b/mail/em-composer-utils.c
index 342ffbe..8f7bde4 100644
--- a/mail/em-composer-utils.c
+++ b/mail/em-composer-utils.c
@@ -1736,14 +1736,17 @@ setup_forward_attached_callbacks (EMsgComposer *composer,
 }
 
 static EMsgComposer *
-forward_attached (EShell *shell,
+forward_attached (EMailBackend *backend,
                   CamelFolder *folder,
                   GPtrArray *uids,
                   CamelMimePart *part,
                   gchar *subject)
 {
+       EShell *shell;
        EMsgComposer *composer;
 
+       shell = e_shell_backend_get_shell (E_SHELL_BACKEND (backend));
+
        composer = create_new_composer (shell, subject, folder);
 
        e_msg_composer_attach (composer, part);
@@ -1764,7 +1767,6 @@ forward_attached_cb (GObject *source_object,
                      gpointer user_data)
 {
        CamelFolder *folder;
-       EShell *shell;
        EMailBackend *backend;
        EActivity *activity;
        EAlertSink *alert_sink;
@@ -1803,10 +1805,9 @@ forward_attached_cb (GObject *source_object,
        }
 
        backend = e_mail_reader_get_backend (async_context->reader);
-       shell = e_shell_backend_get_shell (E_SHELL_BACKEND (backend));
 
        forward_attached (
-               shell, folder, async_context->ptr_array, part, subject);
+               backend, folder, async_context->ptr_array, part, subject);
 
        e_activity_set_state (activity, E_ACTIVITY_COMPLETED);
 
@@ -1818,14 +1819,15 @@ exit:
 }
 
 static EMsgComposer *
-forward_non_attached (EShell *shell,
-                      CamelSession *session,
+forward_non_attached (EMailBackend *backend,
                       CamelFolder *folder,
                       const gchar *uid,
                       CamelMimeMessage *message,
                       EMailForwardStyle style)
 {
        EMsgComposer *composer = NULL;
+       EMailSession *session;
+       EShell *shell;
        gchar *text, *forward;
        guint32 validity_found = 0;
        guint32 flags;
@@ -1835,9 +1837,13 @@ forward_non_attached (EShell *shell,
        if (style == E_MAIL_FORWARD_STYLE_QUOTED)
                flags |= E_MAIL_FORMATTER_QUOTE_FLAG_CITE;
 
+       session = e_mail_backend_get_session (backend);
+       shell = e_shell_backend_get_shell (E_SHELL_BACKEND (backend));
+
        forward = quoting_text (QUOTING_FORWARD);
        text = em_utils_message_to_html (
-               session, message, forward, flags, NULL, NULL, &validity_found);
+               CAMEL_SESSION (session), message,
+               forward, flags, NULL, NULL, &validity_found);
 
        if (text != NULL) {
                CamelDataWrapper *content;
@@ -1884,8 +1890,7 @@ forward_non_attached (EShell *shell,
 
 /**
  * em_utils_forward_message:
- * @shell: an #EShell
- * @session: a #CamelSession
+ * @backend: an #EMailBackend
  * @message: a #CamelMimeMessage to forward
  * @style: the forward style to use
  * @folder: a #CamelFolder, or %NULL
@@ -1895,8 +1900,7 @@ forward_non_attached (EShell *shell,
  * for more details about forwarding styles.
  **/
 EMsgComposer *
-em_utils_forward_message (EShell *shell,
-                          CamelSession *session,
+em_utils_forward_message (EMailBackend *backend,
                           CamelMimeMessage *message,
                           EMailForwardStyle style,
                           CamelFolder *folder,
@@ -1906,8 +1910,7 @@ em_utils_forward_message (EShell *shell,
        gchar *subject;
        EMsgComposer *composer = NULL;
 
-       g_return_val_if_fail (E_IS_SHELL (shell), NULL);
-       g_return_val_if_fail (CAMEL_IS_SESSION (session), NULL);
+       g_return_val_if_fail (E_IS_MAIL_BACKEND (backend), NULL);
        g_return_val_if_fail (CAMEL_IS_MIME_MESSAGE (message), NULL);
 
        switch (style) {
@@ -1917,7 +1920,7 @@ em_utils_forward_message (EShell *shell,
                        subject = mail_tool_generate_forward_subject (message);
 
                        composer = forward_attached (
-                               shell, NULL, NULL, part, subject);
+                               backend, NULL, NULL, part, subject);
 
                        g_object_unref (part);
                        g_free (subject);
@@ -1926,7 +1929,7 @@ em_utils_forward_message (EShell *shell,
                case E_MAIL_FORWARD_STYLE_INLINE:
                case E_MAIL_FORWARD_STYLE_QUOTED:
                        composer = forward_non_attached (
-                               shell, session, folder, uid, message, style);
+                               backend, folder, uid, message, style);
                        break;
        }
 
@@ -1939,9 +1942,7 @@ forward_got_messages_cb (GObject *source_object,
                          gpointer user_data)
 {
        CamelFolder *folder;
-       EShell *shell;
        EMailBackend *backend;
-       EMailSession *session;
        EActivity *activity;
        EAlertSink *alert_sink;
        GHashTable *hash_table;
@@ -1956,6 +1957,8 @@ forward_got_messages_cb (GObject *source_object,
        activity = async_context->activity;
        alert_sink = e_activity_get_alert_sink (activity);
 
+       backend = e_mail_reader_get_backend (async_context->reader);
+
        hash_table = e_mail_folder_get_multiple_messages_finish (
                folder, result, &local_error);
 
@@ -1979,10 +1982,6 @@ forward_got_messages_cb (GObject *source_object,
                goto exit;
        }
 
-       backend = e_mail_reader_get_backend (async_context->reader);
-       session = e_mail_backend_get_session (backend);
-       shell = e_shell_backend_get_shell (E_SHELL_BACKEND (backend));
-
        /* Create a new composer window for each message. */
 
        g_hash_table_iter_init (&iter, hash_table);
@@ -1995,8 +1994,7 @@ forward_got_messages_cb (GObject *source_object,
                message = CAMEL_MIME_MESSAGE (value);
 
                em_utils_forward_message (
-                       shell, CAMEL_SESSION (session),
-                       message, async_context->style,
+                       backend, message, async_context->style,
                        folder, message_uid);
        }
 
diff --git a/mail/em-composer-utils.h b/mail/em-composer-utils.h
index 14d65f7..3ab4c17 100644
--- a/mail/em-composer-utils.h
+++ b/mail/em-composer-utils.h
@@ -48,9 +48,8 @@ void          em_utils_edit_messages          (EMailReader *reader,
                                                 GPtrArray *uids,
                                                 gboolean replace,
                                                 gboolean keep_signature);
-EMsgComposer * em_utils_forward_message        (EShell *shell,
-                                                CamelSession *session,
-                                                CamelMimeMessage *msg,
+EMsgComposer * em_utils_forward_message        (EMailBackend *backend,
+                                                CamelMimeMessage *message,
                                                 EMailForwardStyle style,
                                                 CamelFolder *folder,
                                                 const gchar *uid);
diff --git a/modules/mail/e-mail-attachment-handler.c b/modules/mail/e-mail-attachment-handler.c
index 5e328c4..fbd0762 100644
--- a/modules/mail/e-mail-attachment-handler.c
+++ b/modules/mail/e-mail-attachment-handler.c
@@ -35,8 +35,7 @@
        ((obj), E_TYPE_MAIL_ATTACHMENT_HANDLER, EMailAttachmentHandlerPrivate))
 
 struct _EMailAttachmentHandlerPrivate {
-       EShell *shell;
-       EMailSession *session;
+       EMailBackend *backend;
 };
 
 static gpointer parent_class;
@@ -153,9 +152,7 @@ mail_attachment_handler_forward (GtkAction *action,
        style = g_settings_get_enum (settings, "forward-style-name");
        g_object_unref (settings);
 
-       em_utils_forward_message (
-               priv->shell, CAMEL_SESSION (priv->session),
-               message, style, NULL, NULL);
+       em_utils_forward_message (priv->backend, message, style, NULL, NULL);
 
        g_object_unref (message);
 }
@@ -168,6 +165,8 @@ mail_attachment_handler_reply (EAttachmentHandler *handler,
        GSettings *settings;
        EMailReplyStyle style;
        CamelMimeMessage *message;
+       EShellBackend *shell_backend;
+       EShell *shell;
 
        priv = E_MAIL_ATTACHMENT_HANDLER_GET_PRIVATE (handler);
 
@@ -178,9 +177,11 @@ mail_attachment_handler_reply (EAttachmentHandler *handler,
        style = g_settings_get_enum (settings, "reply-style-name");
        g_object_unref (settings);
 
+       shell_backend = E_SHELL_BACKEND (priv->backend);
+       shell = e_shell_backend_get_shell (shell_backend);
+
        em_utils_reply_to_message (
-               priv->shell, message,
-               NULL, NULL, reply_type, style, NULL, NULL);
+               shell, message, NULL, NULL, reply_type, style, NULL, NULL);
 
        g_object_unref (message);
 }
@@ -306,6 +307,7 @@ mail_attachment_handler_x_uid_list (EAttachmentView *view,
        CamelFolder *folder = NULL;
        EAttachment *attachment;
        EAttachmentStore *store;
+       EMailSession *session;
        GPtrArray *uids;
        const gchar *data;
        const gchar *cp, *end;
@@ -354,10 +356,12 @@ mail_attachment_handler_x_uid_list (EAttachmentView *view,
        if (uids->len == 0)
                goto exit;
 
+       session = e_mail_backend_get_session (priv->backend);
+
        /* The first string is the folder URI. */
        /* FIXME Not passing a GCancellable here. */
        folder = e_mail_session_uri_to_folder_sync (
-               priv->session, data, 0, NULL, &local_error);
+               session, data, 0, NULL, &local_error);
        if (folder == NULL)
                goto exit;
 
@@ -507,15 +511,7 @@ mail_attachment_handler_dispose (GObject *object)
 
        priv = E_MAIL_ATTACHMENT_HANDLER_GET_PRIVATE (object);
 
-       if (priv->shell != NULL) {
-               g_object_unref (priv->shell);
-               priv->shell = NULL;
-       }
-
-       if (priv->session != NULL) {
-               g_object_unref (priv->session);
-               priv->session = NULL;
-       }
+       g_clear_object (&priv->backend);
 
        /* Chain up to parent's dispose() method. */
        G_OBJECT_CLASS (parent_class)->dispose (object);
@@ -529,7 +525,6 @@ mail_attachment_handler_constructed (GObject *object)
        EShellBackend *shell_backend;
        EAttachmentHandler *handler;
        EAttachmentView *view;
-       EMailSession *session;
        GtkActionGroup *action_group;
        GtkUIManager *ui_manager;
        GError *error = NULL;
@@ -542,10 +537,7 @@ mail_attachment_handler_constructed (GObject *object)
 
        shell = e_shell_get_default ();
        shell_backend = e_shell_get_backend_by_name (shell, "mail");
-       session = e_mail_backend_get_session (E_MAIL_BACKEND (shell_backend));
-
-       priv->shell = g_object_ref (shell);
-       priv->session = g_object_ref (session);
+       priv->backend = g_object_ref (shell_backend);
 
        view = e_attachment_handler_get_view (handler);