LibGUI+Everywhere: Use fallible Window::set_main_widget() everywhere :^)

Rip that bandaid off!

This does the following, in one big, awkward jump:
- Replace all uses of `set_main_widget<Foo>()` with the `try` version.
- Remove `set_main_widget<Foo>()`.
- Rename the `try` version to just be `set_main_widget` because it's now
  the only one.

The majority of places that call `set_main_widget<Foo>()` are inside
constructors, so this unfortunately gives us a big batch of new
`release_value_but_fixme_should_propagate_errors()` calls.

1 files changed, 1 insertions, 9 deletions

diff --git a/Userland/Libraries/LibGUI/Window.h b/Userland/Libraries/LibGUI/Window.h
index 03a4bece7c..43652891b8 100644
--- a/Userland/Libraries/LibGUI/Window.h
+++ b/Userland/Libraries/LibGUI/Window.h
@@ -143,21 +143,13 @@ public:
     void set_main_widget(Widget*);
 
     template<class T, class... Args>
-    inline ErrorOr<NonnullRefPtr<T>> try_set_main_widget(Args&&... args)
+    inline ErrorOr<NonnullRefPtr<T>> set_main_widget(Args&&... args)
     {
         auto widget = TRY(T::try_create(forward<Args>(args)...));
         set_main_widget(widget.ptr());
         return widget;
     }
 
-    template<class T, class... Args>
-    inline T& set_main_widget(Args&&... args)
-    {
-        auto widget = T::construct(forward<Args>(args)...);
-        set_main_widget(widget.ptr());
-        return *widget;
-    }
-
     Widget* default_return_key_widget() { return m_default_return_key_widget; }
     Widget const* default_return_key_widget() const { return m_default_return_key_widget; }
     void set_default_return_key_widget(Widget*);