From 94f1eff2916d104a4158af2b528ba22bd570d4a3 Mon Sep 17 00:00:00 2001
From: Linus Groh <mail@linusgroh.de>
Date: Mon, 6 Mar 2023 11:12:48 +0000
Subject: [PATCH] LibWeb/HTML: Port Window.isSecureContext to IDL

---
 Userland/Libraries/LibWeb/HTML/Window.cpp | 9 ---------
 Userland/Libraries/LibWeb/HTML/Window.h   | 1 -
 Userland/Libraries/LibWeb/HTML/Window.idl | 1 +
 3 files changed, 1 insertion(+), 10 deletions(-)

diff --git a/Userland/Libraries/LibWeb/HTML/Window.cpp b/Userland/Libraries/LibWeb/HTML/Window.cpp
index f74db2d7572..30d40feaee5 100644
--- a/Userland/Libraries/LibWeb/HTML/Window.cpp
+++ b/Userland/Libraries/LibWeb/HTML/Window.cpp
@@ -1102,7 +1102,6 @@ WebIDL::ExceptionOr<void> Window::initialize_web_interfaces(Badge<WindowEnvironm
 
     define_native_accessor(realm, "localStorage", local_storage_getter, {}, attr);
     define_native_accessor(realm, "sessionStorage", session_storage_getter, {}, attr);
-    define_native_accessor(realm, "isSecureContext", is_secure_context_getter, {}, attr);
 
     // Legacy
     define_native_accessor(realm, "event", event_getter, event_setter, JS::Attribute::Enumerable);
@@ -1829,14 +1828,6 @@ JS_DEFINE_NATIVE_FUNCTION(Window::structured_clone)
     }));
 }
 
-// https://html.spec.whatwg.org/multipage/webappapis.html#dom-issecurecontext
-JS_DEFINE_NATIVE_FUNCTION(Window::is_secure_context_getter)
-{
-    auto* impl = TRY(impl_from(vm));
-    // The isSecureContext getter steps are to return true if this's relevant settings object is a secure context, or false otherwise.
-    return JS::Value(HTML::is_secure_context(impl->associated_document().relevant_settings_object()));
-}
-
 JS_DEFINE_NATIVE_FUNCTION(Window::local_storage_getter)
 {
     auto* impl = TRY(impl_from(vm));
diff --git a/Userland/Libraries/LibWeb/HTML/Window.h b/Userland/Libraries/LibWeb/HTML/Window.h
index a98a4a9add1..440311b9308 100644
--- a/Userland/Libraries/LibWeb/HTML/Window.h
+++ b/Userland/Libraries/LibWeb/HTML/Window.h
@@ -257,7 +257,6 @@ private:
 
     JS_DECLARE_NATIVE_FUNCTION(local_storage_getter);
     JS_DECLARE_NATIVE_FUNCTION(session_storage_getter);
-    JS_DECLARE_NATIVE_FUNCTION(is_secure_context_getter);
 
     JS_DECLARE_NATIVE_FUNCTION(set_interval);
     JS_DECLARE_NATIVE_FUNCTION(set_timeout);
diff --git a/Userland/Libraries/LibWeb/HTML/Window.idl b/Userland/Libraries/LibWeb/HTML/Window.idl
index 8fd7f4ce631..7702f5e3638 100644
--- a/Userland/Libraries/LibWeb/HTML/Window.idl
+++ b/Userland/Libraries/LibWeb/HTML/Window.idl
@@ -38,6 +38,7 @@ interface Window : EventTarget {
 
     // FIXME: Replace these with 'Window includes WindowOrWorkerGlobalScope;' once we have feature parity
     [Replaceable] readonly attribute USVString origin;
+    readonly attribute boolean isSecureContext;
 };
 Window includes GlobalEventHandlers;
 Window includes WindowEventHandlers;