beenull/ladybird
1
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2024-11-22 07:30:19 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 6

LibWeb: Respect width and height attributes of <iframe>

Browse source 
We have to respect the width and height attributes of <iframe> elements
the same way as <img> elements.
This commit is contained in:
Luke Wilde 2022-10-10 17:22:30 +01:00 committed by Andreas Kling
parent 8589f1115e
commit 6a7c560849
Notes: sideshowbarker 2024-07-17 07:20:49 +09:00 
Author: https://github.com/Lubrsi
Commit: https://github.com/SerenityOS/serenity/commit/6a7c560849
Pull-request: https://github.com/SerenityOS/serenity/pull/15532
Reviewed-by: https://github.com/linusg
2 changed files with 17 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
Userland/Libraries/LibWeb/HTML/HTMLIFrameElement.cpp
View file

@ -9,6 +9,7 @@
#include <LibWeb/HTML/BrowsingContext.h>
#include <LibWeb/HTML/HTMLIFrameElement.h>
#include <LibWeb/HTML/Origin.h>
#include <LibWeb/HTML/Parser/HTMLParser.h>
#include <LibWeb/Layout/FrameBox.h>
namespace Web::HTML {
@ -126,6 +127,20 @@ void HTMLIFrameElement::load_src(String const& value)
m_nested_browsing_context->loader().load(url, FrameLoader::Type::IFrame);
}
// https://html.spec.whatwg.org/multipage/rendering.html#attributes-for-embedded-content-and-images
void HTMLIFrameElement::apply_presentational_hints(CSS::StyleProperties& style) const
{
for_each_attribute([&](auto& name, auto& value) {
if (name == HTML::AttributeNames::width) {
if (auto parsed_value = parse_dimension_value(value))
style.set_property(CSS::PropertyID::Width, parsed_value.release_nonnull());
} else if (name == HTML::AttributeNames::height) {
if (auto parsed_value = parse_dimension_value(value))
style.set_property(CSS::PropertyID::Height, parsed_value.release_nonnull());
}
});
}
// https://html.spec.whatwg.org/multipage/iframe-embed-object.html#iframe-load-event-steps
void run_iframe_load_event_steps(HTML::HTMLIFrameElement& element)
{

2
Userland/Libraries/LibWeb/HTML/HTMLIFrameElement.h
View file

@ -23,6 +23,8 @@ public:
void set_current_navigation_was_lazy_loaded(bool value) { m_current_navigation_was_lazy_loaded = value; }
virtual void apply_presentational_hints(CSS::StyleProperties&) const override;
private:
HTMLIFrameElement(DOM::Document&, DOM::QualifiedName);