mirror of
https://github.com/LadybirdBrowser/ladybird.git
synced 2025-01-23 09:46:04 -05:00
198bb322ef
When parsing a CSS value in the context of a CSSStyleDeclaration camelCase property setter, we don't necessarily have a Document to provide the CSS parser for context. So the parser can't go assuming that there's always a Document in the ParsingContext. And ImageStyleValue can't go assuming that there's always a Document either. This will require some more work to get things right, I'm just patching up the null dereference for now.
68 lines
2.4 KiB
C++
68 lines
2.4 KiB
C++
/*
|
|
* Copyright (c) 2018-2020, Andreas Kling <kling@serenityos.org>
|
|
*
|
|
* SPDX-License-Identifier: BSD-2-Clause
|
|
*/
|
|
|
|
#include <LibWeb/CSS/StyleProperties.h>
|
|
#include <LibWeb/CSS/StyleValue.h>
|
|
#include <LibWeb/DOM/Document.h>
|
|
#include <LibWeb/DOM/Window.h>
|
|
#include <LibWeb/HTML/HTMLBodyElement.h>
|
|
|
|
namespace Web::HTML {
|
|
|
|
HTMLBodyElement::HTMLBodyElement(DOM::Document& document, QualifiedName qualified_name)
|
|
: HTMLElement(document, move(qualified_name))
|
|
{
|
|
}
|
|
|
|
HTMLBodyElement::~HTMLBodyElement()
|
|
{
|
|
}
|
|
|
|
void HTMLBodyElement::apply_presentational_hints(CSS::StyleProperties& style) const
|
|
{
|
|
for_each_attribute([&](auto& name, auto& value) {
|
|
if (name.equals_ignoring_case("bgcolor")) {
|
|
auto color = Color::from_string(value);
|
|
if (color.has_value())
|
|
style.set_property(CSS::PropertyID::BackgroundColor, CSS::ColorStyleValue::create(color.value()));
|
|
} else if (name.equals_ignoring_case("text")) {
|
|
auto color = Color::from_string(value);
|
|
if (color.has_value())
|
|
style.set_property(CSS::PropertyID::Color, CSS::ColorStyleValue::create(color.value()));
|
|
} else if (name.equals_ignoring_case("background")) {
|
|
VERIFY(m_background_style_value);
|
|
style.set_property(CSS::PropertyID::BackgroundImage, *m_background_style_value);
|
|
}
|
|
});
|
|
}
|
|
|
|
void HTMLBodyElement::parse_attribute(const FlyString& name, const String& value)
|
|
{
|
|
HTMLElement::parse_attribute(name, value);
|
|
if (name.equals_ignoring_case("link")) {
|
|
auto color = Color::from_string(value);
|
|
if (color.has_value())
|
|
document().set_link_color(color.value());
|
|
} else if (name.equals_ignoring_case("alink")) {
|
|
auto color = Color::from_string(value);
|
|
if (color.has_value())
|
|
document().set_active_link_color(color.value());
|
|
} else if (name.equals_ignoring_case("vlink")) {
|
|
auto color = Color::from_string(value);
|
|
if (color.has_value())
|
|
document().set_visited_link_color(color.value());
|
|
} else if (name.equals_ignoring_case("background")) {
|
|
m_background_style_value = CSS::ImageStyleValue::create(document().parse_url(value), &document());
|
|
}
|
|
}
|
|
|
|
DOM::EventTarget& HTMLBodyElement::global_event_handlers_to_event_target()
|
|
{
|
|
// NOTE: This is a little weird, but IIUC document.body.onload actually refers to window.onload
|
|
return document().window();
|
|
}
|
|
|
|
}
|