gfgit/claws-mail
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Implement size limit for Litehtml image cache

Browse source
This commit is contained in:
Andrej Kacian 2019-02-08 19:42:54 +01:00
parent f9d496bff6
commit 79bf393df1
2 changed files with 43 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

45
src/plugins/litehtml_viewer/container_linux.cpp
View file

@ -823,15 +823,48 @@ void container_linux::fill_ellipse( cairo_t* cr, int x, int y, int width, int he
void container_linux::clear_images()
{
/* for(images_map::iterator i = m_images.begin(); i != m_images.end(); i++)
{
if(i->second)
{
delete i->second;
for(auto i = m_images.begin(); i != m_images.end(); ++i) {
image *img = &(*i);
if (img->second) {
g_object_unref(img->second);
}
}
m_images.clear();
*/
}
void container_linux::clear_images(gint desired_size)
{
gint size = 0;
/* First, tally up size of all the stored GdkPixbufs and
* deallocate those which make the total size be above
* the desired_size limit. We will remove their list
* elements later. */
for (auto i = m_images.rbegin(); i != m_images.rend(); ++i) {
image *img = &(*i);
gint cursize;
if (img->second == NULL)
continue;
cursize = gdk_pixbuf_get_byte_length(img->second);
if (size + cursize > desired_size) {
g_object_unref(img->second);
img->second = NULL;
} else {
size += cursize;
}
}
/* Remove elements whose GdkPixbuf pointers point to NULL. */
m_images.remove_if([&](image _img) -> bool {
if (_img.second == NULL)
return true;
return false;
});
}
const litehtml::tchar_t* container_linux::get_default_font_name() const

4
src/plugins/litehtml_viewer/container_linux.h
View file

@ -86,6 +86,10 @@ public:
void clear_images();
/* Trim down images cache to less than desired_size [bytes],
* starting from oldest stored. */
void clear_images(gint desired_size);
protected:
virtual void draw_ellipse(cairo_t* cr, int x, int y, int width, int height, const litehtml::web_color& color, int line_width);
virtual void fill_ellipse(cairo_t* cr, int x, int y, int width, int height, const litehtml::web_color& color);