From 762617a028e8355ea55ec2c13f12ab9d40ea9270 Mon Sep 17 00:00:00 2001
From: Andreas Kling <kling@serenityos.org>
Date: Fri, 5 Jun 2020 13:24:35 +0200
Subject: [PATCH] LibWeb: Don't create a layout node for <noscript> when
 scripting enabled

This makes stuff inside <noscript> correctly not show up since we run
with scripting enabled.

In the future, we can add a way to disable scripting, but for now,
Document::is_scripting_enabled() just returns true.
---
 Libraries/LibWeb/DOM/Document.h  | 2 ++
 Libraries/LibWeb/DOM/Element.cpp | 4 ++++
 2 files changed, 6 insertions(+)

diff --git a/Libraries/LibWeb/DOM/Document.h b/Libraries/LibWeb/DOM/Document.h
index 6679e46e23..5ae6e1fb17 100644
--- a/Libraries/LibWeb/DOM/Document.h
+++ b/Libraries/LibWeb/DOM/Document.h
@@ -57,6 +57,8 @@ public:
 
     Origin origin() const;
 
+    bool is_scripting_enabled() const { return true; }
+
     URL complete_url(const String&) const;
 
     void fixup();
diff --git a/Libraries/LibWeb/DOM/Element.cpp b/Libraries/LibWeb/DOM/Element.cpp
index 24ddcfa793..41cf9c2587 100644
--- a/Libraries/LibWeb/DOM/Element.cpp
+++ b/Libraries/LibWeb/DOM/Element.cpp
@@ -114,6 +114,10 @@ RefPtr<LayoutNode> Element::create_layout_node(const StyleProperties* parent_sty
 
     if (display == "none")
         return nullptr;
+
+    if (tag_name() == "noscript" && document().is_scripting_enabled())
+        return nullptr;
+
     if (display == "block")
         return adopt(*new LayoutBlock(this, move(style)));
     if (display == "inline")