path: root/framebuffer
diff options
authorVincent Sanders <>2015-03-15 00:00:45 +0000
committerVincent Sanders <>2015-03-15 00:00:45 +0000
commit8a99b045bc48e0c5eb89129ea8ee034c72b955de (patch)
treef2dabbca020b91b41ab3f476fca1ed2378f8ced2 /framebuffer
parent52b50db3a6f1d52b3c9e18b36121f487c09de906 (diff)
Remove url from content thumbnailers API
The content thumbnailers for each frontend were being provided the contents url. This was only ever used to call the urldb thumbnail setting API. This changes it so the single callsite that passed a valid url adds the bitmap to that url itself in desktop_history.c instead of forcing every frontend to require the urldb API. Additionally the old API could pass the url as NULL which was causing asserts where this was not an expected parameter value. Because of this this fixes bug #2286 which was also present in the monkey frontend as both called nsurl_access() on the url without the NULL check and caused an assertion.
Diffstat (limited to 'framebuffer')
1 files changed, 1 insertions, 6 deletions
diff --git a/framebuffer/thumbnail.c b/framebuffer/thumbnail.c
index 616a59901..24067fd2f 100644
--- a/framebuffer/thumbnail.c
+++ b/framebuffer/thumbnail.c
@@ -34,8 +34,7 @@
thumbnail_create(struct hlcache_handle *content,
- struct bitmap *bitmap,
- nsurl *url)
+ struct bitmap *bitmap)
nsfb_t *tbm = (nsfb_t *)bitmap; /* target bitmap */
nsfb_t *bm; /* temporary bitmap */
@@ -94,9 +93,5 @@ thumbnail_create(struct hlcache_handle *content,
- /* register the thumbnail with the URL */
- if (url != NULL)
- urldb_set_thumbnail(url, bitmap);
return true;