[gnome-photos] base-item: Serialize thumbnail creation for all items

[Debarshi Ray <debarshir src gnome org>]

commit e61cf278df24b45b66d7b7d5bb4e5bcda3d41d2b
Author: Debarshi Ray <debarshir gnome org>
Date:   Tue Aug 20 09:50:08 2013 +0200

    base-item: Serialize thumbnail creation for all items
    
    We don't want to spawn 50 different threads to create the thumbnails
    of all the items in the first batch. Instead, we want to create them
    one at a time. Hence, use a class level lock instead of a per-object
    one.

 src/photos-base-item.c |   11 +++++------
 1 files changed, 5 insertions(+), 6 deletions(-)
---
diff --git a/src/photos-base-item.c b/src/photos-base-item.c
index b20ab00..463ccb8 100644
--- a/src/photos-base-item.c
+++ b/src/photos-base-item.c
@@ -49,7 +49,6 @@ struct _PhotosBaseItemPrivate
   GeglNode *graph;
   GeglNode *node;
   GeglRectangle bbox;
-  GMutex mutex_create_thumbnail;
   GMutex mutex_download;
   GMutex mutex;
   GQuark equipment;
@@ -102,6 +101,9 @@ static guint signals[LAST_SIGNAL] = { 0 };
 G_DEFINE_TYPE_WITH_PRIVATE (PhotosBaseItem, photos_base_item, G_TYPE_OBJECT);
 
 
+G_LOCK_DEFINE (create_thumbnail);
+
+
 static void photos_base_item_populate_from_cursor (PhotosBaseItem *self, TrackerSparqlCursor *cursor);
 
 
@@ -217,11 +219,10 @@ photos_base_item_create_thumbnail_in_thread_func (GSimpleAsyncResult *simple,
                                                   GCancellable *cancellable)
 {
   PhotosBaseItem *self = PHOTOS_BASE_ITEM (object);
-  PhotosBaseItemPrivate *priv = self->priv;
   GError *error;
   gboolean op_res;
 
-  g_mutex_lock (&priv->mutex_create_thumbnail);
+  G_LOCK (create_thumbnail);
 
   error = NULL;
   op_res = PHOTOS_BASE_ITEM_GET_CLASS (self)->create_thumbnail (self, cancellable, &error);
@@ -230,7 +231,7 @@ photos_base_item_create_thumbnail_in_thread_func (GSimpleAsyncResult *simple,
 
   g_simple_async_result_set_op_res_gboolean (simple, op_res);
 
-  g_mutex_unlock (&priv->mutex_create_thumbnail);
+  G_UNLOCK (create_thumbnail);
 }
 
 
@@ -836,7 +837,6 @@ photos_base_item_finalize (GObject *object)
   g_free (priv->type_description);
   g_free (priv->uri);
 
-  g_mutex_clear (&priv->mutex_create_thumbnail);
   g_mutex_clear (&priv->mutex_download);
   g_mutex_clear (&priv->mutex);
 
@@ -893,7 +893,6 @@ photos_base_item_init (PhotosBaseItem *self)
   self->priv = photos_base_item_get_instance_private (self);
   priv = self->priv;
 
-  g_mutex_init (&priv->mutex_create_thumbnail);
   g_mutex_init (&priv->mutex_download);
   g_mutex_init (&priv->mutex);