From 87f655f2af0289b2eadc6cd546db04223a63fe06 Mon Sep 17 00:00:00 2001
From: Idan Horowitz <idan.horowitz@gmail.com>
Date: Tue, 28 Sep 2021 23:10:49 +0300
Subject: [PATCH] LibWeb: Change IDL::parse_interface's return type to
 NonnullOwnPtr

We always returned an interface, so there's no need for the null state.
---
 .../Lagom/Tools/CodeGenerators/LibWeb/WrapperGenerator.cpp | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/Meta/Lagom/Tools/CodeGenerators/LibWeb/WrapperGenerator.cpp b/Meta/Lagom/Tools/CodeGenerators/LibWeb/WrapperGenerator.cpp
index 7db98f6377..0533d33a0c 100644
--- a/Meta/Lagom/Tools/CodeGenerators/LibWeb/WrapperGenerator.cpp
+++ b/Meta/Lagom/Tools/CodeGenerators/LibWeb/WrapperGenerator.cpp
@@ -167,7 +167,7 @@ struct Interface {
     bool is_legacy_platform_object() const { return !extended_attributes.contains("Global") && (supports_indexed_properties() || supports_named_properties()); }
 };
 
-static OwnPtr<Interface> parse_interface(StringView filename, StringView const& input)
+static NonnullOwnPtr<Interface> parse_interface(StringView filename, StringView const& input)
 {
     auto interface = make<Interface>();
 
@@ -619,11 +619,6 @@ int main(int argc, char** argv)
     auto data = file_or_error.value()->read_all();
     auto interface = IDL::parse_interface(path, data);
 
-    if (!interface) {
-        warnln("Cannot parse {}", path);
-        return 1;
-    }
-
     if (namespace_.is_one_of("CSS", "DOM", "HTML", "UIEvents", "Geometry", "HighResolutionTime", "NavigationTiming", "RequestIdleCallback", "SVG", "XHR", "URL")) {
         StringBuilder builder;
         builder.append(namespace_);