diff options
author | Vincent Sanders <vince@kyllikki.org> | 2016-08-09 12:26:37 +0100 |
---|---|---|
committer | Vincent Sanders <vince@kyllikki.org> | 2016-08-09 12:26:37 +0100 |
commit | 115985f069247d5eb6c5955de7de110114db9ca4 (patch) | |
tree | 3deaf429495e69c36023220849f37a6c5b27c1b4 /content/urldb.c | |
parent | d35ad1b8e707f9dc8ccad5b11efc4eec0302f3ad (diff) | |
download | netsurf-115985f069247d5eb6c5955de7de110114db9ca4.tar.gz netsurf-115985f069247d5eb6c5955de7de110114db9ca4.tar.bz2 |
extend url database unit testing to cover more API
Diffstat (limited to 'content/urldb.c')
-rw-r--r-- | content/urldb.c | 34 |
1 files changed, 24 insertions, 10 deletions
diff --git a/content/urldb.c b/content/urldb.c index 35254921a..12a3d4182 100644 --- a/content/urldb.c +++ b/content/urldb.c @@ -2801,8 +2801,16 @@ bool urldb_add_url(nsurl *url) h = urldb_add_host(host_str); /* Get path entry */ - p = (h != NULL) ? urldb_add_path(scheme, port_int, h, path_query, - fragment, url) : NULL; + if (h != NULL) { + p = urldb_add_path(scheme, + port_int, + h, + path_query, + fragment, + url); + } else { + p = NULL; + } lwc_string_unref(scheme); if (fragment != NULL) @@ -3074,23 +3082,29 @@ bool urldb_get_cert_permissions(nsurl *url) /* exported interface documented in content/urldb.h */ -void urldb_set_thumbnail(nsurl *url, struct bitmap *bitmap) +bool urldb_set_thumbnail(nsurl *url, struct bitmap *bitmap) { struct path_data *p; assert(url); - p = urldb_find_url(url); - if (p != NULL) { + /* add url, in case it's missing */ + urldb_add_url(url); - LOG("Setting bitmap on %s", nsurl_access(url)); + p = urldb_find_url(url); + if (p == NULL) { + return false; + } - if (p->thumb && p->thumb != bitmap) { - guit->bitmap->destroy(p->thumb); - } + LOG("Setting bitmap on %s", nsurl_access(url)); - p->thumb = bitmap; + if ((p->thumb) && (p->thumb != bitmap)) { + guit->bitmap->destroy(p->thumb); } + + p->thumb = bitmap; + + return true; } |