From 6119b95ca4c0c2d4d78b27422dfb6ba9203bab56 Mon Sep 17 00:00:00 2001 From: Sergio Villar Senin Date: Wed, 7 Mar 2012 18:28:02 +0100 Subject: Replace EphyFaviconCache by WebKit's icon database cache https://bugzilla.gnome.org/show_bug.cgi?id=648653 --- embed/Makefile.am | 2 - embed/ephy-embed-prefs.h | 1 + embed/ephy-embed-shell.c | 30 -- embed/ephy-embed-shell.h | 2 - embed/ephy-embed-single.c | 6 + embed/ephy-favicon-cache.c | 869 ---------------------------------- embed/ephy-favicon-cache.h | 73 --- embed/ephy-web-view.c | 42 +- src/Makefile.am | 2 - src/bookmarks/ephy-bookmark-action.c | 44 +- src/bookmarks/ephy-bookmarks-editor.c | 44 +- src/ephy-completion-model.c | 61 ++- src/ephy-history-window.c | 24 +- src/ephy-shell.c | 1 - src/pdm-dialog.c | 5 +- src/prefs-dialog.c | 1 - 16 files changed, 130 insertions(+), 1077 deletions(-) delete mode 100644 embed/ephy-favicon-cache.c delete mode 100644 embed/ephy-favicon-cache.h diff --git a/embed/Makefile.am b/embed/Makefile.am index ff79e02e6..ddcdb14c6 100644 --- a/embed/Makefile.am +++ b/embed/Makefile.am @@ -11,7 +11,6 @@ NOINST_H_FILES = \ ephy-embed-dialog.h \ ephy-embed-private.h \ ephy-encodings.h \ - ephy-favicon-cache.h \ ephy-request-about.h \ ephy-web-app-utils.h @@ -47,7 +46,6 @@ libephyembed_la_SOURCES = \ ephy-embed-shell.c \ ephy-embed-utils.c \ ephy-encodings.c \ - ephy-favicon-cache.c \ ephy-history.c \ ephy-permission-manager.c \ ephy-request-about.c \ diff --git a/embed/ephy-embed-prefs.h b/embed/ephy-embed-prefs.h index 0afbef5d3..ee86da04a 100644 --- a/embed/ephy-embed-prefs.h +++ b/embed/ephy-embed-prefs.h @@ -28,6 +28,7 @@ #endif #define USER_STYLESHEET_FILENAME "user-stylesheet.css" +#define FAVICON_SIZE 16 G_BEGIN_DECLS diff --git a/embed/ephy-embed-shell.c b/embed/ephy-embed-shell.c index cda96e2e0..3b521f18a 100644 --- a/embed/ephy-embed-shell.c +++ b/embed/ephy-embed-shell.c @@ -34,7 +34,6 @@ #include "ephy-embed-single.h" #include "ephy-embed-type-builtins.h" #include "ephy-encodings.h" -#include "ephy-favicon-cache.h" #include "ephy-file-helpers.h" #include "ephy-history.h" #include "ephy-history-service.h" @@ -54,7 +53,6 @@ struct _EphyEmbedShellPrivate EphyHistory *global_history; EphyHistoryService *global_history_service; GList *downloads; - EphyFaviconCache *favicon_cache; EphyEmbedSingle *embed_single; EphyEncodings *encodings; EphyAdBlockManager *adblock_manager; @@ -96,13 +94,6 @@ ephy_embed_shell_dispose (GObject *object) EphyEmbedShell *shell = EPHY_EMBED_SHELL (object); EphyEmbedShellPrivate *priv = shell->priv; - if (priv->favicon_cache != NULL) - { - LOG ("Unref favicon cache"); - g_object_unref (priv->favicon_cache); - priv->favicon_cache = NULL; - } - if (priv->encodings != NULL) { LOG ("Unref encodings"); @@ -166,27 +157,6 @@ ephy_embed_shell_finalize (GObject *object) G_OBJECT_CLASS (ephy_embed_shell_parent_class)->finalize (object); } -/** - * ephy_embed_shell_get_favicon_cache: - * @shell: the #EphyEmbedShell - * - * Returns the favicons cache. - * - * Return value: (transfer none): the favicons cache - **/ -GObject * -ephy_embed_shell_get_favicon_cache (EphyEmbedShell *shell) -{ - g_return_val_if_fail (EPHY_IS_EMBED_SHELL (shell), NULL); - - if (shell->priv->favicon_cache == NULL) - { - shell->priv->favicon_cache = ephy_favicon_cache_new (); - } - - return G_OBJECT (shell->priv->favicon_cache); -} - /** * ephy_embed_shell_get_global_history: * @shell: the #EphyEmbedShell diff --git a/embed/ephy-embed-shell.h b/embed/ephy-embed-shell.h index 4bab3a4ed..84b507fbc 100644 --- a/embed/ephy-embed-shell.h +++ b/embed/ephy-embed-shell.h @@ -79,8 +79,6 @@ GType ephy_embed_shell_get_type (void); EphyEmbedShell *ephy_embed_shell_get_default (void); -GObject *ephy_embed_shell_get_favicon_cache (EphyEmbedShell *shell); - GObject *ephy_embed_shell_get_global_history (EphyEmbedShell *shell); GObject *ephy_embed_shell_get_global_history_service (EphyEmbedShell *shell); diff --git a/embed/ephy-embed-single.c b/embed/ephy-embed-single.c index bf815ed19..a9a2017c3 100644 --- a/embed/ephy-embed-single.c +++ b/embed/ephy-embed-single.c @@ -350,6 +350,7 @@ ephy_embed_single_initialize (EphyEmbedSingle *single) char *filename; char *cookie_policy; char *cache_dir; + char *favicon_db_path; EphyEmbedSinglePrivate *priv = single->priv; SoupSessionFeature *requester; @@ -412,6 +413,11 @@ ephy_embed_single_initialize (EphyEmbedSingle *single) soup_session_add_feature_by_type (session, SOUP_TYPE_PASSWORD_MANAGER_GNOME); #endif + /* Initialize the favicon cache. */ + favicon_db_path = g_build_filename (g_get_user_data_dir (), g_get_prgname (), NULL); + webkit_favicon_database_set_path (webkit_get_favicon_database (), favicon_db_path); + g_free (favicon_db_path); + return TRUE; } diff --git a/embed/ephy-favicon-cache.c b/embed/ephy-favicon-cache.c deleted file mode 100644 index e4fc82ec4..000000000 --- a/embed/ephy-favicon-cache.c +++ /dev/null @@ -1,869 +0,0 @@ -/* -*- Mode: C; tab-width: 8; indent-tabs-mode: t; c-basic-offset: 8 -*- */ -/* - * Copyright © 2002 Jorn Baayen - * Copyright © 2003-2004 Marco Pesenti Gritti - * Copyright © 2004, 2005 Christian Persch - * - * This program is free software; you can redistribute it and/or modify - * it under the terms of the GNU General Public License as published by - * the Free Software Foundation; either version 2 of the License, or - * (at your option) any later version. - * - * This program is distributed in the hope that it will be useful, - * but WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the - * GNU General Public License for more details. - * - * You should have received a copy of the GNU General Public License - * along with this program; if not, write to the Free Software - * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. - * - */ - -#include "config.h" - -#include "ephy-favicon-cache.h" - -#include -#include -#include - -#include "ephy-embed-shell.h" -#include "ephy-file-helpers.h" -#include "ephy-node-common.h" -#include "ephy-node.h" -#include "ephy-debug.h" - -#include -#include -#include -#include - -#define EPHY_FAVICON_CACHE_XML_ROOT (const xmlChar *)"ephy_favicons_cache" -#define EPHY_FAVICON_CACHE_XML_VERSION (const xmlChar *)"1.1" - -#define EPHY_FAVICON_CACHE_OBSOLETE_DAYS 30 -#define SECS_PER_DAY (60*60*24) - -/* how often to save the cache, in seconds */ -#define CACHE_SAVE_INTERVAL (10 * 60) /* seconds */ - -/* how often to delete old pixbufs from the cache, in seconds */ -#define CACHE_CLEANUP_INTERVAL (5 * 60) /* seconds */ - -/* how long to keep pixbufs in cache, in seconds. This should be longer than CACHE_CLEANUP_INTERVAL */ -#define PIXBUF_CACHE_KEEP_TIME 10 * 60 /* s */ - -/* this is very generous, most files are 4k */ -#define EPHY_FAVICON_MAX_SIZE 64 * 1024 /* byte */ - -static void ephy_favicon_cache_class_init (EphyFaviconCacheClass *klass); -static void ephy_favicon_cache_init (EphyFaviconCache *cache); -static void ephy_favicon_cache_finalize (GObject *object); -static gboolean kill_download (const char*, WebKitDownload*, EphyFaviconCache*); - -#define EPHY_FAVICON_CACHE_GET_PRIVATE(object)(G_TYPE_INSTANCE_GET_PRIVATE ((object), EPHY_TYPE_FAVICON_CACHE, EphyFaviconCachePrivate)) - -struct _EphyFaviconCachePrivate -{ - char *directory; - char *xml_file; - EphyNodeDb *db; - EphyNode *icons; - GHashTable *icons_hash; - GHashTable *downloads_hash; - guint autosave_timeout; - guint cleanup_timeout; - guint dirty : 1; - guint64 requests; - guint64 cached; -}; - -typedef struct -{ - EphyNode *node; - time_t timestamp; - GdkPixbuf *pixbuf; - guint load_failed : 1; -} PixbufCacheEntry; - -typedef gboolean (* FilterFunc) (EphyNode*, time_t); - -enum -{ - CHANGED, - LAST_SIGNAL -}; - -enum -{ - EPHY_NODE_FAVICON_PROP_URL = 2, - EPHY_NODE_FAVICON_PROP_FILENAME = 3, - EPHY_NODE_FAVICON_PROP_LAST_USED = 4, - EPHY_NODE_FAVICON_PROP_STATE = 5, - EPHY_NODE_FAVICON_PROP_CHECKED = 7, -}; - -enum -{ - NEEDS_TYPE_CHECK = 1 << 0, - NEEDS_RENAME = 1 << 1, - NEEDS_ICO_CHECK = 1 << 2, - NEEDS_MASK = 0x3f -}; - -static guint signals[LAST_SIGNAL]; - -G_DEFINE_TYPE (EphyFaviconCache, ephy_favicon_cache, G_TYPE_OBJECT) - -static void -ephy_favicon_cache_class_init (EphyFaviconCacheClass *klass) -{ - GObjectClass *object_class = G_OBJECT_CLASS (klass); - - object_class->finalize = ephy_favicon_cache_finalize; - - signals[CHANGED] = - g_signal_new ("changed", - G_OBJECT_CLASS_TYPE (object_class), - G_SIGNAL_RUN_LAST, - G_STRUCT_OFFSET (EphyFaviconCacheClass, changed), - NULL, NULL, - g_cclosure_marshal_VOID__STRING, - G_TYPE_NONE, - 1, - G_TYPE_STRING); - - g_type_class_add_private (object_class, sizeof (EphyFaviconCachePrivate)); -} - -EphyFaviconCache * -ephy_favicon_cache_new (void) -{ - return EPHY_FAVICON_CACHE (g_object_new (EPHY_TYPE_FAVICON_CACHE, NULL)); -} - -static void -pixbuf_cache_entry_free (PixbufCacheEntry *entry) -{ - if (entry->pixbuf != NULL) - { - g_object_unref (entry->pixbuf); - } - - g_slice_free (PixbufCacheEntry, entry); -} - -static gboolean -icon_is_obsolete (EphyNode *node, time_t now) -{ - int last_visit; - - last_visit = ephy_node_get_property_int - (node, EPHY_NODE_FAVICON_PROP_LAST_USED); - return now - last_visit >= EPHY_FAVICON_CACHE_OBSOLETE_DAYS*SECS_PER_DAY; -} - -static void -icons_added_cb (EphyNode *node, - EphyNode *child, - EphyFaviconCache *eb) -{ - PixbufCacheEntry *entry = g_slice_new0 (PixbufCacheEntry); - - entry->node = child; - - g_hash_table_insert (eb->priv->icons_hash, - (char *) ephy_node_get_property_string (child, EPHY_NODE_FAVICON_PROP_URL), - entry); -} - -static void -icons_removed_cb (EphyNode *node, - EphyNode *child, - guint old_index, - EphyFaviconCache *eb) -{ - g_hash_table_remove (eb->priv->icons_hash, - ephy_node_get_property_string (child, EPHY_NODE_FAVICON_PROP_URL)); -} - -static void -remove_obsolete_icons (EphyFaviconCache *cache, - FilterFunc filter) -{ - EphyFaviconCachePrivate *priv = cache->priv; - GPtrArray *children; - int i; - time_t now; - - now = time (NULL); - - children = ephy_node_get_children (priv->icons); - for (i = (int) children->len - 1; i >= 0; i--) - { - EphyNode *kid; - - kid = g_ptr_array_index (children, i); - - if (!filter || filter (kid, now)) - { - const char *filename; - char *path; - - filename = ephy_node_get_property_string - (kid, EPHY_NODE_FAVICON_PROP_FILENAME); - path = g_build_filename (priv->directory, - filename, NULL); - if (g_unlink (path) < 0) - { - LOG ("Unable to delete %s", path); - } - - g_free (path); - ephy_node_unref (kid); - priv->dirty = TRUE; - } - } -} - -static void -prepare_close_cb (EphyEmbedShell *shell, - EphyFaviconCache *cache) -{ - EphyFaviconCachePrivate *priv = cache->priv; - - g_hash_table_foreach_remove (priv->downloads_hash, - (GHRFunc) kill_download, cache); -} - -static void -ephy_favicon_cache_save (EphyFaviconCache *cache) -{ - EphyFaviconCachePrivate *priv = cache->priv; - int ret; - - if (priv->dirty) - { - LOG ("Saving favicon cache\n"); - - ret = ephy_node_db_write_to_xml_safe - (cache->priv->db, (const xmlChar *) priv->xml_file, - EPHY_FAVICON_CACHE_XML_ROOT, - EPHY_FAVICON_CACHE_XML_VERSION, - NULL, - priv->icons, NULL, NULL, - NULL); - - /* still dirty if save failed */ - priv->dirty = ret < 0; - } -} - -static gboolean -periodic_save_cb (EphyFaviconCache *cache) -{ - ephy_favicon_cache_save (cache); - return TRUE; -} - -static void -cleanup_entry (char *key, - PixbufCacheEntry *entry, - time_t* now) -{ - if (entry->pixbuf != NULL && - *now - entry->timestamp > PIXBUF_CACHE_KEEP_TIME) - { - LOG ("Evicting pixbuf for \"%s\" from pixbuf cache", key); - - g_object_unref (entry->pixbuf); - entry->pixbuf = NULL; - entry->load_failed = FALSE; - entry->timestamp = 0; - } -} - -static gboolean -periodic_cleanup_cb (EphyFaviconCache *cache) -{ - EphyFaviconCachePrivate *priv = cache->priv; - time_t now; - - LOG ("Cleanup"); - - now = time (NULL); - g_hash_table_foreach (priv->icons_hash, (GHFunc) cleanup_entry, &now); - - return TRUE; -} - -static void -ephy_favicon_cache_init (EphyFaviconCache *cache) -{ - EphyFaviconCachePrivate *priv; - EphyNodeDb *db; - - priv = cache->priv = EPHY_FAVICON_CACHE_GET_PRIVATE (cache); - - db = ephy_node_db_new (EPHY_NODE_DB_SITEICONS); - cache->priv->db = db; - - cache->priv->xml_file = g_build_filename (ephy_dot_dir (), - "ephy-favicon-cache.xml", - NULL); - - cache->priv->directory = g_build_filename (ephy_dot_dir (), - "favicon_cache", - NULL); - - if (g_file_test (cache->priv->directory, G_FILE_TEST_IS_DIR) == FALSE) - { - if (mkdir (cache->priv->directory, 488) != 0) - { - g_error ("Couldn't mkdir %s.", cache->priv->directory); - } - } - - /* The key is owned by the node */ - priv->icons_hash = g_hash_table_new_full (g_str_hash, g_str_equal, NULL, - (GDestroyNotify) pixbuf_cache_entry_free); - priv->downloads_hash = g_hash_table_new_full (g_str_hash, - g_str_equal, - g_free, NULL); - - /* Icons */ - cache->priv->icons = ephy_node_new_with_id (db, ICONS_NODE_ID); - ephy_node_signal_connect_object (cache->priv->icons, - EPHY_NODE_CHILD_ADDED, - (EphyNodeCallback) icons_added_cb, - G_OBJECT (cache)); - ephy_node_signal_connect_object (cache->priv->icons, - EPHY_NODE_CHILD_REMOVED, - (EphyNodeCallback) icons_removed_cb, - G_OBJECT (cache)); - - ephy_node_db_load_from_file (cache->priv->db, cache->priv->xml_file, - EPHY_FAVICON_CACHE_XML_ROOT, - EPHY_FAVICON_CACHE_XML_VERSION); - - /* listen to prepare-close on the shell */ - g_signal_connect_object (embed_shell, "prepare-close", - G_CALLBACK (prepare_close_cb), cache, 0); - - priv->autosave_timeout = g_timeout_add_seconds (CACHE_SAVE_INTERVAL, - (GSourceFunc) periodic_save_cb, - cache); - priv->cleanup_timeout = g_timeout_add_seconds (CACHE_CLEANUP_INTERVAL, - (GSourceFunc) periodic_cleanup_cb, - cache); -} - -static gboolean -kill_download (const char *key, - WebKitDownload *download, - EphyFaviconCache *cache) -{ - EphyNode *icon; - - /* disconnect "completed" and "cancelled" callbacks */ - g_signal_handlers_disconnect_matched - (download,G_SIGNAL_MATCH_DATA , 0, 0, NULL, NULL, cache); - - /* now cancel the download */ - webkit_download_cancel (download); - - icon = g_hash_table_lookup (cache->priv->icons_hash, key); - g_return_val_if_fail (EPHY_IS_NODE (icon), TRUE); - - ephy_node_unref (icon); - - return TRUE; -} - -static void -delete_file (GFile *dir, - GFileInfo *file_info) -{ - GFileType type; - - type = g_file_info_get_file_type (file_info); - - if (type == G_FILE_TYPE_REGULAR) - { - char *path; - - path = g_build_filename (g_file_get_path (dir), - g_file_info_get_name (file_info), - NULL); - if (g_unlink (path) < 0) - { - LOG ("Unable to delete %s", path); - } - - g_free (path); - } -} - -static void -ephy_favicon_cache_finalize (GObject *object) -{ - EphyFaviconCache *cache = EPHY_FAVICON_CACHE (object); - EphyFaviconCachePrivate *priv = cache->priv; - - LOG ("EphyFaviconCache finalising"); - - g_hash_table_foreach_remove (priv->downloads_hash, - (GHRFunc) kill_download, cache); - - if (priv->autosave_timeout != 0) - { - g_source_remove (priv->autosave_timeout); - } - - if (priv->cleanup_timeout != 0) - { - g_source_remove (priv->cleanup_timeout); - } - - remove_obsolete_icons (cache, icon_is_obsolete); - - ephy_favicon_cache_save (cache); - - g_free (priv->xml_file); - g_free (priv->directory); - g_hash_table_destroy (priv->icons_hash); - g_hash_table_destroy (priv->downloads_hash); - - g_object_unref (priv->db); - - LOG ("Requests: cached %" G_GUINT64_FORMAT " / total %" G_GUINT64_FORMAT " = %.3f", - priv->cached, priv->requests, ((double) priv->cached) / ((double) priv->requests)); - - G_OBJECT_CLASS (ephy_favicon_cache_parent_class)->finalize (object); -} - -static gboolean -is_valid_mime_type (const char *mime_type) -{ - - gboolean valid = FALSE; - - valid = g_strcmp0 (mime_type, "image/x-ico") == 0 || - g_strcmp0 (mime_type, "image/vnd.microsoft.icon") == 0 || - g_strcmp0 (mime_type, "image/png") == 0 || - g_strcmp0 (mime_type, "image/gif") == 0 || - g_strcmp0 (mime_type, "application/octet-stream") == 0; - - return valid; -} - -static void -favicon_download_status_changed_cb (WebKitDownload *download, - GParamSpec *spec, - EphyFaviconCache *cache) -{ - WebKitDownloadStatus status = webkit_download_get_status (download); - const char *url = webkit_download_get_uri (download); - const char *destination = webkit_download_get_destination_uri (download); - const char *mime_type; - GFile *file; - GFileInfo *file_info; - gboolean valid; - - switch (status) { - case WEBKIT_DOWNLOAD_STATUS_FINISHED: - LOG ("Favicon cache download completed for %s", url); - - g_hash_table_remove (cache->priv->downloads_hash, url); - - file = g_file_new_for_uri (destination); - file_info = g_file_query_info (file, - G_FILE_ATTRIBUTE_STANDARD_CONTENT_TYPE, - 0, NULL, NULL); - - mime_type = g_file_info_get_content_type (file_info); - valid = is_valid_mime_type (mime_type); - - if (!valid) - { - LOG ("Deleting invalid %s type", mime_type); - g_file_delete (file, NULL, NULL); - } - else - { - g_signal_emit (G_OBJECT (cache), signals[CHANGED], 0, url); - } - - g_object_unref (file); - g_object_unref (file_info); - - g_object_unref (download); - - cache->priv->dirty = TRUE; - break; - case WEBKIT_DOWNLOAD_STATUS_ERROR: - case WEBKIT_DOWNLOAD_STATUS_CANCELLED: - LOG ("Favicon cache download cancelled %s", url); - - g_hash_table_remove (cache->priv->downloads_hash, url); - - /* FIXME: re-schedule to try again after n days? */ - file = g_file_new_for_uri (destination); - if (g_file_query_exists (file, NULL)) - { - LOG ("Deleting incomplete favicon download"); - g_file_delete (file, NULL, NULL); - } - - g_object_unref (file); - g_object_unref (download); - - cache->priv->dirty = TRUE; - break; - default: - break; - } -} - -static void -ephy_favicon_cache_download (EphyFaviconCache *cache, - const char *favicon_url, - const char *filename) -{ - WebKitNetworkRequest *request; - WebKitDownload *download; - char *dest; - char *dest_uri; - - LOG ("Download favicon: %s", favicon_url); - - g_return_if_fail (EPHY_IS_FAVICON_CACHE (cache)); - g_return_if_fail (favicon_url != NULL); - g_return_if_fail (filename != NULL); - - request = webkit_network_request_new (favicon_url); - download = webkit_download_new (request); - g_object_unref (request); - - dest = g_build_filename (cache->priv->directory, filename, NULL); - dest_uri = g_filename_to_uri (dest, NULL, NULL); - - webkit_download_set_destination_uri (download, dest_uri); - - g_free (dest); - g_free (dest_uri); - - g_signal_connect (G_OBJECT (download), "notify::status", - G_CALLBACK (favicon_download_status_changed_cb), cache); - - g_hash_table_insert (cache->priv->downloads_hash, - g_strdup (favicon_url), download); - - webkit_download_start (download); -} - -/** - * ephy_favicon_cache_get: - * @cache: an #EphyFaviconCache - * @url: the URL of the icon to retrieve - * - * Note: This will always return %NULL for non-http URLs. - * - * Return value: (transfer full): the site icon at @url as a #GdkPixbuf, or - * %NULL if if could not be retrieved. Unref when you don't need it anymore. - */ -GdkPixbuf * -ephy_favicon_cache_get (EphyFaviconCache *cache, - const char *url) -{ - EphyFaviconCachePrivate *priv = cache->priv; - time_t now; - PixbufCacheEntry *entry; - EphyNode *icon; - char *pix_file; - GdkPixbuf *pixbuf = NULL; - guint checklevel = NEEDS_MASK; - int width, height; - int favicon_checked; - - if (url == NULL) return NULL; - - if (!g_str_has_prefix (url, "http://") && - !g_str_has_prefix (url, "https://")) return NULL; - - priv->requests += 1; - - now = time (NULL); - - entry = g_hash_table_lookup (priv->icons_hash, url); - - if (entry != NULL && entry->pixbuf != NULL) - { - LOG ("Pixbuf in cache for \"%s\"", url); - - priv->cached += 1; - entry->timestamp = now; - return g_object_ref (entry->pixbuf); - } - else if (entry != NULL && entry->load_failed) - { - /* No need to try again */ - priv->cached += 1; - return NULL; - } - - if (entry == NULL) - { - char *filename; - - filename = g_compute_checksum_for_string (G_CHECKSUM_MD5, url, -1); - - icon = ephy_node_new (cache->priv->db); - ephy_node_set_property_string (icon, - EPHY_NODE_FAVICON_PROP_URL, - url); - ephy_node_set_property_string (icon, - EPHY_NODE_FAVICON_PROP_FILENAME, - filename); - ephy_node_set_property_int (icon, - EPHY_NODE_FAVICON_PROP_CHECKED, - (int) NEEDS_MASK & ~NEEDS_RENAME); - - /* This will also add an entry to the icons hash */ - ephy_node_add_child (priv->icons, icon); - - ephy_favicon_cache_download (cache, url, filename); - - g_free (filename); - - entry = g_hash_table_lookup (priv->icons_hash, url); - } - - g_return_val_if_fail (entry != NULL, NULL); - g_return_val_if_fail (entry->pixbuf == NULL, NULL); - - icon = entry->node; - - /* update timestamp */ - ephy_node_set_property_int (icon, EPHY_NODE_FAVICON_PROP_LAST_USED, - now); - - priv->dirty = TRUE; - - if (g_hash_table_lookup (cache->priv->downloads_hash, url) != NULL) - { - /* still downloading, return NULL */ - return NULL; - } - - pix_file = g_build_filename - (cache->priv->directory, - ephy_node_get_property_string (icon, EPHY_NODE_FAVICON_PROP_FILENAME), - NULL); - - /* FIXME queue re-download? */ - if (g_file_test (pix_file, G_FILE_TEST_EXISTS) == FALSE) - { - g_free (pix_file); - return NULL; - } - - /* Check for supported icon types */ - favicon_checked = ephy_node_get_property_int - (icon, EPHY_NODE_FAVICON_PROP_CHECKED); - if (favicon_checked != -1) - { - checklevel = (guint) favicon_checked; - } - - /* First, update the filename */ - if (checklevel & NEEDS_RENAME) - { - char *new_pix_file, *urlhash; - GValue value = { 0, }; - - urlhash = g_compute_checksum_for_string (G_CHECKSUM_MD5, url, -1); - new_pix_file = g_build_filename (cache->priv->directory, urlhash, NULL); - - g_value_init (&value, G_TYPE_STRING); - g_value_take_string (&value, urlhash); - ephy_node_set_property (icon, EPHY_NODE_FAVICON_PROP_FILENAME, &value); - g_value_unset (&value); - - /* rename the file */ - g_rename (pix_file, new_pix_file); - - g_free (pix_file); - pix_file = new_pix_file; - - checklevel &= ~NEEDS_RENAME; - ephy_node_set_property_int (icon, - EPHY_NODE_FAVICON_PROP_CHECKED, - (int) checklevel); - } - - /* Now check the type. We renamed the file above, so glib does NOT - * fall back to extension checking if the slow mime check fails for - * whatever reason - */ - if (checklevel & NEEDS_TYPE_CHECK) - { - GFile *file; - GFileInfo *file_info; - const char *mime_type; - gboolean valid = FALSE, is_ao = FALSE; - - file = g_file_new_for_path (pix_file); - - /* Sniff mime type and check if it's safe to open */ - file_info = g_file_query_info (file, - G_FILE_ATTRIBUTE_STANDARD_CONTENT_TYPE, - 0, NULL, NULL); - mime_type = g_file_info_get_content_type (file_info); - if (file_info == NULL) - { - return NULL; - } - - valid = is_valid_mime_type (mime_type); - is_ao = g_strcmp0 (mime_type, "application/octet-stream") == 0; - - g_object_unref (file_info); - - /* As a special measure, we try to load an - * application/octet-stream file as an ICO file, since we - * cannot detect a ICO file without .ico extension (the mime - * system has no magic for it). - */ - if (is_ao) - { - GdkPixbufLoader *loader; - char *buf = NULL; - gsize count = 0; - - loader = gdk_pixbuf_loader_new_with_type ("ico", NULL); - if (loader != NULL) - { - if (g_file_get_contents (pix_file, &buf, &count, NULL)) - { - gdk_pixbuf_loader_write (loader, (const guchar *) buf, count, NULL); - g_free (buf); - } - - gdk_pixbuf_loader_close (loader, NULL); - - pixbuf = gdk_pixbuf_loader_get_pixbuf (loader); - if (pixbuf != NULL) - { - g_object_ref (pixbuf); - valid = TRUE; - } - - g_object_unref (loader); - } - } - - /* persist the check value */ - if (valid) - { - checklevel &= ~NEEDS_TYPE_CHECK; - } - else - { - /* remove invalid file from cache */ - g_file_delete (file, NULL, NULL); - } - g_object_unref (file); - - ephy_node_set_property_int (icon, - EPHY_NODE_FAVICON_PROP_CHECKED, - (int) checklevel); - } - - /* if it still needs the check, mime type couldn't be checked. Deny! */ - if (checklevel & NEEDS_TYPE_CHECK) - { - g_free (pix_file); - return NULL; - } - - /* we could already have a pixbuf from the application/octet-stream check */ - if (pixbuf == NULL) - { - pixbuf = gdk_pixbuf_new_from_file (pix_file, NULL); - } - - g_free (pix_file); - - if (pixbuf == NULL) - { - entry->load_failed = TRUE; - return NULL; - } - - width = gdk_pixbuf_get_width (pixbuf); - height = gdk_pixbuf_get_height (pixbuf); - - /* Reject icons that are too small */ - if (width < 12 || height < 12) - { - entry->load_failed = TRUE; - return NULL; - } - - /* Scale icons that are too big */ - if (width > 16 || height > 16) - { - GdkPixbuf *scaled = gdk_pixbuf_scale_simple (pixbuf, 16, 16, - GDK_INTERP_NEAREST); - g_object_unref (G_OBJECT (pixbuf)); - pixbuf = scaled; - } - - /* Put the icon in the cache */ - LOG ("Putting pixbuf for \"%s\" in cache", url); - - entry->pixbuf = g_object_ref (pixbuf); - entry->timestamp = now; - entry->load_failed = FALSE; - - return pixbuf; -} - -/** - * ephy_favicons_cache_clear: - * @cache: the #EphyFaviconCache to clear - * - * Clears the favicon cache and removes any stored icon files from disk. - */ -void -ephy_favicon_cache_clear (EphyFaviconCache *cache) -{ - GFileEnumerator *file_enum; - GFile *dir; - GFileInfo *file_info = NULL; - EphyFaviconCachePrivate *priv = cache->priv; - - g_return_if_fail (EPHY_IS_FAVICON_CACHE (cache)); - - remove_obsolete_icons (cache, NULL); - ephy_favicon_cache_save (cache); - - /* Now remove any remaining files from the cache directory */ - dir = g_file_new_for_path (priv->directory); - file_enum = g_file_enumerate_children (dir, - G_FILE_ATTRIBUTE_STANDARD_TYPE "," - G_FILE_ATTRIBUTE_STANDARD_NAME "," - G_FILE_ATTRIBUTE_STANDARD_CONTENT_TYPE, - 0, NULL, NULL); - file_info = g_file_enumerator_next_file (file_enum, NULL, NULL); - while (file_info != NULL) - { - delete_file (dir, file_info); - file_info = g_file_enumerator_next_file (file_enum, NULL, NULL); - g_object_unref (file_info); - } - g_object_unref (dir); - g_file_enumerator_close (file_enum, NULL, NULL); -} diff --git a/embed/ephy-favicon-cache.h b/embed/ephy-favicon-cache.h deleted file mode 100644 index 216c33d30..000000000 --- a/embed/ephy-favicon-cache.h +++ /dev/null @@ -1,73 +0,0 @@ -/* - * Copyright © 2002 Jorn Baayen - * Copyright © 2003-2004 Marco Pesenti Gritti - * Copyright © 2004, 2005 Christian Persch - * - * This program is free software; you can redistribute it and/or modify - * it under the terms of the GNU General Public License as published by - * the Free Software Foundation; either version 2, or (at your option) - * any later version. - * - * This program is distributed in the hope that it will be useful, - * but WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the - * GNU General Public License for more details. - * - * You should have received a copy of the GNU General Public License - * along with this program; if not, write to the Free Software - * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. - * - */ - -#if !defined (__EPHY_EPIPHANY_H_INSIDE__) && !defined (EPIPHANY_COMPILATION) -#error "Only can be included directly." -#endif - -#ifndef EPHY_FAVICON_CACHE_H -#define EPHY_FAVICON_CACHE_H - -#include -#include - -G_BEGIN_DECLS - -#define EPHY_TYPE_FAVICON_CACHE (ephy_favicon_cache_get_type ()) -#define EPHY_FAVICON_CACHE(o) (G_TYPE_CHECK_INSTANCE_CAST ((o), EPHY_TYPE_FAVICON_CACHE, EphyFaviconCache)) -#define EPHY_FAVICON_CACHE_CLASS(k) (G_TYPE_CHECK_CLASS_CAST ((k), EPHY_TYPE_FAVICON_CACHE, EphyFaviconCacheClass)) -#define EPHY_IS_FAVICON_CACHE(o) (G_TYPE_CHECK_INSTANCE_TYPE ((o), EPHY_TYPE_FAVICON_CACHE)) -#define EPHY_IS_FAVICON_CACHE_CLASS(k) (G_TYPE_CHECK_CLASS_TYPE ((k), EPHY_TYPE_FAVICON_CACHE)) -#define EPHY_FAVICON_CACHE_GET_CLASS(o) (G_TYPE_INSTANCE_GET_CLASS ((o), EPHY_TYPE_FAVICON_CACHE, EphyFaviconCacheClass)) - -typedef struct _EphyFaviconCacheClass EphyFaviconCacheClass; -typedef struct _EphyFaviconCache EphyFaviconCache; -typedef struct _EphyFaviconCachePrivate EphyFaviconCachePrivate; - -struct _EphyFaviconCache -{ - GObject parent; - - /*< private >*/ - EphyFaviconCachePrivate *priv; -}; - -struct _EphyFaviconCacheClass -{ - GObjectClass parent_class; - - /* Signals */ - void (*changed) (EphyFaviconCache *cache, - const char *url); -}; - -GType ephy_favicon_cache_get_type (void); - -EphyFaviconCache *ephy_favicon_cache_new (void); - -GdkPixbuf *ephy_favicon_cache_get (EphyFaviconCache *cache, - const char *url); - -void ephy_favicon_cache_clear (EphyFaviconCache *cache); - -G_END_DECLS - -#endif /* EPHY_FAVICON_CACHE_H */ diff --git a/embed/ephy-web-view.c b/embed/ephy-web-view.c index 2f6579e7e..66c057d0e 100644 --- a/embed/ephy-web-view.c +++ b/embed/ephy-web-view.c @@ -32,7 +32,6 @@ #include "ephy-embed-type-builtins.h" #include "ephy-embed-utils.h" #include "ephy-embed.h" -#include "ephy-favicon-cache.h" #include "ephy-file-helpers.h" #include "ephy-history.h" #include "ephy-history-service.h" @@ -1593,41 +1592,18 @@ static void _ephy_web_view_load_icon (EphyWebView *view) { EphyWebViewPrivate *priv = view->priv; - EphyEmbedShell *shell; - EphyFaviconCache *cache; - const char *icon_address; - - icon_address = webkit_web_view_get_icon_uri (WEBKIT_WEB_VIEW (view)); + const char* uri; - if (icon_address == NULL || priv->icon != NULL) return; - - shell = ephy_embed_shell_get_default (); - cache = EPHY_FAVICON_CACHE (ephy_embed_shell_get_favicon_cache (shell)); + if (priv->icon != NULL) + return; - /* ephy_favicon_cache_get returns a reference already */ - priv->icon = ephy_favicon_cache_get (cache, icon_address); + uri = webkit_web_view_get_uri (WEBKIT_WEB_VIEW (view)); + priv->icon = webkit_favicon_database_try_get_favicon_pixbuf (webkit_get_favicon_database (), uri, + FAVICON_SIZE, FAVICON_SIZE); g_object_notify (G_OBJECT (view), "icon"); } -static void -icon_cache_changed_cb (EphyFaviconCache *cache, - const char *address, - EphyWebView *view) -{ - const char *icon_address; - - g_return_if_fail (address != NULL); - - icon_address = webkit_web_view_get_icon_uri (WEBKIT_WEB_VIEW (view)); - - /* is this for us? */ - if (icon_address != NULL && - strcmp (icon_address, address) == 0) { - _ephy_web_view_load_icon (view); - } -} - static void _ephy_web_view_set_icon_address (EphyWebView *view, const char *icon_address) @@ -2337,7 +2313,6 @@ static void ephy_web_view_init (EphyWebView *web_view) { EphyWebViewPrivate *priv; - EphyFaviconCache *cache; priv = web_view->priv = EPHY_WEB_VIEW_GET_PRIVATE (web_view); @@ -2407,11 +2382,6 @@ ephy_web_view_init (EphyWebView *web_view) G_CALLBACK (ge_popup_blocked_cb), NULL); - cache = EPHY_FAVICON_CACHE - (ephy_embed_shell_get_favicon_cache (embed_shell)); - g_signal_connect_object (G_OBJECT (cache), "changed", - G_CALLBACK (icon_cache_changed_cb), - web_view, (GConnectFlags)0); } /** diff --git a/src/Makefile.am b/src/Makefile.am index 56e699b77..7ea753b5b 100644 --- a/src/Makefile.am +++ b/src/Makefile.am @@ -224,7 +224,6 @@ EPHY_GIR_H_FILES = \ $(top_srcdir)/embed/ephy-embed-event.h \ $(top_srcdir)/embed/ephy-embed-shell.h \ $(top_srcdir)/embed/ephy-embed-single.h \ - $(top_srcdir)/embed/ephy-favicon-cache.h \ $(top_srcdir)/embed/ephy-history.h \ $(top_srcdir)/embed/ephy-permission-manager.h \ $(top_srcdir)/embed/ephy-web-view.h \ @@ -254,7 +253,6 @@ EPHY_GIR_C_FILES = \ $(top_srcdir)/embed/ephy-embed-event.c \ $(top_srcdir)/embed/ephy-embed-shell.c \ $(top_srcdir)/embed/ephy-embed-single.c \ - $(top_srcdir)/embed/ephy-favicon-cache.c \ $(top_srcdir)/embed/ephy-history.c \ $(top_srcdir)/embed/ephy-permission-manager.c \ $(top_srcdir)/embed/ephy-web-view.c \ diff --git a/src/bookmarks/ephy-bookmark-action.c b/src/bookmarks/ephy-bookmark-action.c index 814fadd7e..7aa30010b 100644 --- a/src/bookmarks/ephy-bookmark-action.c +++ b/src/bookmarks/ephy-bookmark-action.c @@ -26,7 +26,7 @@ #include "ephy-bookmarks.h" #include "ephy-debug.h" #include "ephy-dnd.h" -#include "ephy-favicon-cache.h" +#include "ephy-embed-prefs.h" #include "ephy-gui.h" #include "ephy-shell.h" #include "ephy-string.h" @@ -67,20 +67,22 @@ typedef struct G_DEFINE_TYPE (EphyBookmarkAction, ephy_bookmark_action, EPHY_TYPE_LINK_ACTION) static void -favicon_cache_changed_cb (EphyFaviconCache *cache, - const char *icon_address, - EphyBookmarkAction *action) +favicon_loaded_cb (WebKitFaviconDatabase *database, + const char *page_address, + EphyBookmarkAction *action) { const char *icon; + char *icon_address; g_return_if_fail (action->priv->node != NULL); icon = ephy_node_get_property_string (action->priv->node, EPHY_NODE_BMK_PROP_ICON); - - if (icon != NULL && strcmp (icon, icon_address) == 0) + icon_address = webkit_favicon_database_get_favicon_uri (database, page_address); + + if (g_strcmp0 (icon, icon_address) == 0) { - g_signal_handler_disconnect (cache, action->priv->cache_handler); + g_signal_handler_disconnect (database, action->priv->cache_handler); action->priv->cache_handler = 0; g_object_notify (G_OBJECT (action), "icon"); @@ -93,28 +95,27 @@ ephy_bookmark_action_sync_icon (GtkAction *action, GtkWidget *proxy) { EphyBookmarkAction *bma = EPHY_BOOKMARK_ACTION (action); - const char *icon_location; - EphyFaviconCache *cache; + const char *page_location; + WebKitFaviconDatabase *database; GdkPixbuf *pixbuf = NULL; g_return_if_fail (bma->priv->node != NULL); - icon_location = ephy_node_get_property_string (bma->priv->node, - EPHY_NODE_BMK_PROP_ICON); + page_location = ephy_node_get_property_string (bma->priv->node, + EPHY_NODE_BMK_PROP_LOCATION); - cache = EPHY_FAVICON_CACHE (ephy_embed_shell_get_favicon_cache - (ephy_embed_shell_get_default ())); - - if (icon_location && *icon_location) + database = webkit_get_favicon_database (); + if (page_location && *page_location) { - pixbuf = ephy_favicon_cache_get (cache, icon_location); + pixbuf = webkit_favicon_database_try_get_favicon_pixbuf (database, page_location, + FAVICON_SIZE, FAVICON_SIZE); if (pixbuf == NULL && bma->priv->cache_handler == 0) { bma->priv->cache_handler = g_signal_connect_object - (cache, "changed", - G_CALLBACK (favicon_cache_changed_cb), + (database, "icon-loaded", + G_CALLBACK (favicon_loaded_cb), action, 0); } } @@ -370,14 +371,11 @@ ephy_bookmark_action_dispose (GObject *object) { EphyBookmarkAction *action = (EphyBookmarkAction *) object; EphyBookmarkActionPrivate *priv = action->priv; - GObject *cache; if (priv->cache_handler != 0) { - cache = ephy_embed_shell_get_favicon_cache - (ephy_embed_shell_get_default ()); - - g_signal_handler_disconnect (cache, priv->cache_handler); + WebKitFaviconDatabase *database = webkit_get_favicon_database (); + g_signal_handler_disconnect (database, priv->cache_handler); priv->cache_handler = 0; } diff --git a/src/bookmarks/ephy-bookmarks-editor.c b/src/bookmarks/ephy-bookmarks-editor.c index c64d1db48..b30b542b6 100644 --- a/src/bookmarks/ephy-bookmarks-editor.c +++ b/src/bookmarks/ephy-bookmarks-editor.c @@ -26,7 +26,7 @@ #include "ephy-bookmarks-ui.h" #include "ephy-debug.h" #include "ephy-dnd.h" -#include "ephy-favicon-cache.h" +#include "ephy-embed-prefs.h" #include "ephy-file-chooser.h" #include "ephy-file-helpers.h" #include "ephy-gui.h" @@ -1460,27 +1460,43 @@ node_dropped_cb (EphyNodeView *view, } } +static void +icon_loaded_cb (WebKitFaviconDatabase *database, GAsyncResult *result, GValue *value) +{ + GdkPixbuf *favicon = webkit_favicon_database_get_favicon_pixbuf_finish (database, result, NULL); + if (!favicon) + return; + + g_value_take_object (value, favicon); +} + static void provide_favicon (EphyNode *node, GValue *value, gpointer user_data) { - EphyFaviconCache *cache; - const char *icon_location; - GdkPixbuf *pixbuf = NULL; + const char *page_location; + GdkPixbuf *favicon = NULL; - cache = EPHY_FAVICON_CACHE - (ephy_embed_shell_get_favicon_cache (EPHY_EMBED_SHELL (ephy_shell))); - icon_location = ephy_node_get_property_string - (node, EPHY_NODE_BMK_PROP_ICON); + page_location = ephy_node_get_property_string + (node, EPHY_NODE_BMK_PROP_LOCATION); - LOG ("Get favicon for %s", icon_location ? icon_location : "None"); + LOG ("Get favicon for %s", page_location ? page_location : "None"); - if (icon_location) - { - pixbuf = ephy_favicon_cache_get (cache, icon_location); - } + if (page_location) + { + WebKitFaviconDatabase *database = webkit_get_favicon_database (); + + /* Try with the sync version first as this method will be frequently called. */ + favicon = webkit_favicon_database_try_get_favicon_pixbuf (database, page_location, + FAVICON_SIZE, FAVICON_SIZE); + + if (!favicon && webkit_favicon_database_get_favicon_uri (database, page_location)) + webkit_favicon_database_get_favicon_pixbuf (database, page_location, + FAVICON_SIZE, FAVICON_SIZE, NULL, + (GAsyncReadyCallback) icon_loaded_cb, value); + } g_value_init (value, GDK_TYPE_PIXBUF); - g_value_take_object (value, pixbuf); + g_value_take_object (value, favicon); } static void diff --git a/src/ephy-completion-model.c b/src/ephy-completion-model.c index e3f25bda5..9b6507e1a 100644 --- a/src/ephy-completion-model.c +++ b/src/ephy-completion-model.c @@ -21,8 +21,8 @@ #include "config.h" #include "ephy-completion-model.h" +#include "ephy-embed-prefs.h" #include "ephy-embed-shell.h" -#include "ephy-favicon-cache.h" #include "ephy-history.h" #include "ephy-history-service.h" #include "ephy-shell.h" @@ -36,7 +36,6 @@ G_DEFINE_TYPE (EphyCompletionModel, ephy_completion_model, GTK_TYPE_LIST_STORE) struct _EphyCompletionModelPrivate { EphyHistoryService *history_service; EphyHistory *legacy_history_service; - EphyFaviconCache *favicon_cache; EphyNode *bookmarks; GSList *search_terms; @@ -99,7 +98,6 @@ ephy_completion_model_init (EphyCompletionModel *model) priv->history_service = EPHY_HISTORY_SERVICE (ephy_embed_shell_get_global_history_service (embed_shell)); priv->legacy_history_service = EPHY_HISTORY (ephy_embed_shell_get_global_history (embed_shell)); - priv->favicon_cache = EPHY_FAVICON_CACHE (ephy_embed_shell_get_favicon_cache (embed_shell)); bookmarks_service = ephy_shell_get_bookmarks (ephy_shell); priv->bookmarks = ephy_bookmarks_get_bookmarks (bookmarks_service); @@ -165,22 +163,69 @@ typedef struct { gboolean is_bookmark; } PotentialRow; +typedef struct { + GtkListStore *model; + GtkTreeRowReference *row_reference; +} IconLoadData; + + +static void +icon_loaded_cb (GObject *source, GAsyncResult *result, gpointer user_data) +{ + GtkTreeIter iter; + IconLoadData *data = (IconLoadData *) user_data; + GdkPixbuf *favicon = webkit_favicon_database_get_favicon_pixbuf_finish (webkit_get_favicon_database (), result, NULL); + + if (favicon) { + /* The completion model might have changed its contents */ + if (gtk_tree_row_reference_valid (data->row_reference)) { + gtk_tree_model_get_iter (GTK_TREE_MODEL (data->model), &iter, + gtk_tree_row_reference_get_path (data->row_reference)); + gtk_list_store_set (data->model, &iter, EPHY_COMPLETION_FAVICON_COL, favicon, -1); + } + } + + g_object_unref (data->model); + gtk_tree_row_reference_free (data->row_reference); + g_slice_free (IconLoadData, data); +} + static void set_row_in_model (EphyCompletionModel *model, int position, PotentialRow *row) { - const char *favicon_location = ephy_history_get_icon (model->priv->legacy_history_service, - row->location); + GtkTreeIter iter; + GdkPixbuf *favicon; + GtkTreePath *path; + IconLoadData *data; + WebKitFaviconDatabase* database = webkit_get_favicon_database (); - gtk_list_store_insert_with_values (GTK_LIST_STORE (model), NULL, position, + gtk_list_store_insert_with_values (GTK_LIST_STORE (model), &iter, position, EPHY_COMPLETION_TEXT_COL, row->title ? row->title : "", EPHY_COMPLETION_URL_COL, row->location, EPHY_COMPLETION_ACTION_COL, row->location, EPHY_COMPLETION_KEYWORDS_COL, row->keywords ? row->keywords : "", EPHY_COMPLETION_EXTRA_COL, row->is_bookmark, - EPHY_COMPLETION_FAVICON_COL, ephy_favicon_cache_get (model->priv->favicon_cache, - favicon_location), EPHY_COMPLETION_RELEVANCE_COL, row->relevance, -1); + + /* We try first with the try_get_favicon_pixbuf() because if the icon + is in the DB it's faster than the async version. */ + favicon = webkit_favicon_database_try_get_favicon_pixbuf (database, row->location, + FAVICON_SIZE, FAVICON_SIZE); + if (favicon) { + gtk_list_store_set (GTK_LIST_STORE (model), &iter, EPHY_COMPLETION_FAVICON_COL, favicon, -1); + return; + } + + data = g_slice_new (IconLoadData); + data->model = GTK_LIST_STORE (g_object_ref(model)); + path = gtk_tree_model_get_path (GTK_TREE_MODEL (model), &iter); + data->row_reference = gtk_tree_row_reference_new (GTK_TREE_MODEL (model), path); + gtk_tree_path_free (path); + + webkit_favicon_database_get_favicon_pixbuf (webkit_get_favicon_database (), row->location, + FAVICON_SIZE, FAVICON_SIZE, NULL, + icon_loaded_cb, data); } static void diff --git a/src/ephy-history-window.c b/src/ephy-history-window.c index 5ddb0cb2e..39783b573 100644 --- a/src/ephy-history-window.c +++ b/src/ephy-history-window.c @@ -26,7 +26,6 @@ #include "ephy-bookmarks-ui.h" #include "ephy-debug.h" #include "ephy-dnd.h" -#include "ephy-favicon-cache.h" #include "ephy-file-helpers.h" #include "ephy-gui.h" #include "ephy-hosts-store.h" @@ -870,21 +869,22 @@ delete_event_cb (EphyHistoryWindow *editor) static void provide_favicon (EphyNode *node, GValue *value, gpointer user_data) { - EphyFaviconCache *cache; - const char *icon_location; + const char *page_location; GdkPixbuf *pixbuf = NULL; - cache = EPHY_FAVICON_CACHE - (ephy_embed_shell_get_favicon_cache (EPHY_EMBED_SHELL (ephy_shell))); - icon_location = ephy_node_get_property_string - (node, EPHY_NODE_PAGE_PROP_ICON); + page_location = ephy_node_get_property_string + (node, EPHY_NODE_PAGE_PROP_LOCATION); - LOG ("Get favicon for %s", icon_location ? icon_location : "None"); + LOG ("Get favicon for %s", page_location ? page_location : "None"); - if (icon_location) - { - pixbuf = ephy_favicon_cache_get (cache, icon_location); - } + if (page_location) + { + /* No need to use the async version as this function will be + called many times by the treeview. */ + WebKitFaviconDatabase *database = webkit_get_favicon_database (); + pixbuf = webkit_favicon_database_get_favicon_pixbuf (database, page_location, + FAVICON_SIZE, FAVICON_SIZE); + } g_value_init (value, GDK_TYPE_PIXBUF); g_value_take_object (value, pixbuf); diff --git a/src/ephy-shell.c b/src/ephy-shell.c index 6964cefb4..510017a7f 100644 --- a/src/ephy-shell.c +++ b/src/ephy-shell.c @@ -30,7 +30,6 @@ #include "ephy-embed-single.h" #include "ephy-embed-utils.h" #include "ephy-extensions-manager.h" -#include "ephy-favicon-cache.h" #include "ephy-file-helpers.h" #include "ephy-gui.h" #include "ephy-history-window.h" diff --git a/src/pdm-dialog.c b/src/pdm-dialog.c index 4f0139e6d..24d075869 100644 --- a/src/pdm-dialog.c +++ b/src/pdm-dialog.c @@ -33,7 +33,6 @@ #include "ephy-debug.h" #include "ephy-time-helpers.h" #include "ephy-embed-single.h" -#include "ephy-favicon-cache.h" #include "ephy-history-service.h" #include "ephy-password-info.h" @@ -284,7 +283,6 @@ clear_all_dialog_response_cb (GtkDialog *dialog, { EphyEmbedShell *shell; EphyEmbedSingle *single; - EphyFaviconCache *cache; shell = ephy_embed_shell_get_default (); @@ -292,8 +290,7 @@ clear_all_dialog_response_cb (GtkDialog *dialog, ephy_embed_single_clear_cache (single); - cache = EPHY_FAVICON_CACHE (ephy_embed_shell_get_favicon_cache (shell)); - ephy_favicon_cache_clear (cache); + webkit_favicon_database_clear (webkit_get_favicon_database ()); } } gtk_widget_destroy (GTK_WIDGET (dialog)); diff --git a/src/prefs-dialog.c b/src/prefs-dialog.c index 7c3ec5c15..d3c0c09ac 100644 --- a/src/prefs-dialog.c +++ b/src/prefs-dialog.c @@ -31,7 +31,6 @@ #include "ephy-embed-single.h" #include "ephy-embed-utils.h" #include "ephy-encodings.h" -#include "ephy-favicon-cache.h" #include "ephy-file-chooser.h" #include "ephy-file-helpers.h" #include "ephy-gui.h" -- cgit v1.2.3