mirror of
				https://github.com/LadybirdBrowser/ladybird.git
				synced 2025-11-04 07:10:57 +00:00 
			
		
		
		
	Instead of making the "Entry" inner struct GC-allocated and marking *that*, we now mark the image instead. This fixes an issue found by ASAN on https://mozilla.com/
		
			
				
	
	
		
			60 lines
		
	
	
	
		
			1.6 KiB
		
	
	
	
		
			C++
		
	
	
	
	
	
			
		
		
	
	
			60 lines
		
	
	
	
		
			1.6 KiB
		
	
	
	
		
			C++
		
	
	
	
	
	
/*
 | 
						|
 * Copyright (c) 2023, Andreas Kling <kling@serenityos.org>
 | 
						|
 *
 | 
						|
 * SPDX-License-Identifier: BSD-2-Clause
 | 
						|
 */
 | 
						|
 | 
						|
#include <LibWeb/HTML/DecodedImageData.h>
 | 
						|
#include <LibWeb/HTML/ListOfAvailableImages.h>
 | 
						|
 | 
						|
namespace Web::HTML {
 | 
						|
 | 
						|
JS_DEFINE_ALLOCATOR(ListOfAvailableImages);
 | 
						|
 | 
						|
ListOfAvailableImages::ListOfAvailableImages() = default;
 | 
						|
ListOfAvailableImages::~ListOfAvailableImages() = default;
 | 
						|
 | 
						|
bool ListOfAvailableImages::Key::operator==(Key const& other) const
 | 
						|
{
 | 
						|
    return url == other.url && mode == other.mode && origin == other.origin;
 | 
						|
}
 | 
						|
 | 
						|
u32 ListOfAvailableImages::Key::hash() const
 | 
						|
{
 | 
						|
    if (!cached_hash.has_value()) {
 | 
						|
        u32 url_hash = Traits<AK::URL>::hash(url);
 | 
						|
        u32 mode_hash = static_cast<u32>(mode);
 | 
						|
        u32 origin_hash = 0;
 | 
						|
        if (origin.has_value())
 | 
						|
            origin_hash = Traits<HTML::Origin>::hash(origin.value());
 | 
						|
        cached_hash = pair_int_hash(url_hash, pair_int_hash(mode_hash, origin_hash));
 | 
						|
    }
 | 
						|
    return cached_hash.value();
 | 
						|
}
 | 
						|
 | 
						|
void ListOfAvailableImages::visit_edges(JS::Cell::Visitor& visitor)
 | 
						|
{
 | 
						|
    Base::visit_edges(visitor);
 | 
						|
    for (auto& it : m_images)
 | 
						|
        visitor.visit(it.value->image_data);
 | 
						|
}
 | 
						|
 | 
						|
void ListOfAvailableImages::add(Key const& key, JS::NonnullGCPtr<DecodedImageData> image_data, bool ignore_higher_layer_caching)
 | 
						|
{
 | 
						|
    m_images.set(key, make<Entry>(image_data, ignore_higher_layer_caching));
 | 
						|
}
 | 
						|
 | 
						|
void ListOfAvailableImages::remove(Key const& key)
 | 
						|
{
 | 
						|
    m_images.remove(key);
 | 
						|
}
 | 
						|
 | 
						|
ListOfAvailableImages::Entry* ListOfAvailableImages::get(Key const& key)
 | 
						|
{
 | 
						|
    auto it = m_images.find(key);
 | 
						|
    if (it == m_images.end())
 | 
						|
        return nullptr;
 | 
						|
    return it->value.ptr();
 | 
						|
}
 | 
						|
 | 
						|
}
 |