Create PurpleUi that merges PurpleCoreUiOps and PurpleUiInfo

This created a lot more refactoring that I imagined, but it's mostly just moving stuff around.

I skipped the unit tests for now because we already have a `test_ui.[ch]` in the tests directory which we can/should move to a separate library or something, so it's available outside of the `libpurple/tests/` directory.

Testing Done:
Ran the unit tests, launched pidgin3, and launched finch3 with out issue.

Bugs closed: PIDGIN-17697

Reviewed at https://reviews.imfreedom.org/r/1925/

1 files changed, 40 insertions, 19 deletions

diff --git a/libpurple/tests/test_ui.c b/libpurple/tests/test_ui.c
index f172462678..30f30daf73 100644
--- a/libpurple/tests/test_ui.c
+++ b/libpurple/tests/test_ui.c
@@ -58,21 +58,51 @@ static PurpleConversationUiOps test_conv_uiops = {
 	.write_conv = test_write_conv
 };
 
+/******************************************************************************
+ * PurpleUi Implementation
+ *****************************************************************************/
+struct _TestPurpleUi {
+	PurpleUi parent;
+};
+
+G_DECLARE_FINAL_TYPE(TestPurpleUi, test_purple_ui, TEST_PURPLE, UI, PurpleUi)
+
+G_DEFINE_TYPE(TestPurpleUi, test_purple_ui, PURPLE_TYPE_UI)
+
 static void
-test_ui_init(void)
-{
+test_purple_ui_start(G_GNUC_UNUSED PurpleUi *ui) {
 	purple_conversations_set_ui_ops(&test_conv_uiops);
 }
 
 static gpointer
-test_ui_get_settings_backend(void) {
+test_purple_ui_get_settings_backend(G_GNUC_UNUSED PurpleUi *ui) {
 	return g_memory_settings_backend_new();
 }
 
-static PurpleCoreUiOps test_core_uiops = {
-	.ui_init = test_ui_init,
-	.get_settings_backend = test_ui_get_settings_backend,
-};
+static void
+test_purple_ui_init(G_GNUC_UNUSED TestPurpleUi *ui) {
+}
+
+static void
+test_purple_ui_class_init(TestPurpleUiClass *klass) {
+	PurpleUiClass *ui_class = PURPLE_UI_CLASS(klass);
+
+	ui_class->start = test_purple_ui_start;
+	ui_class->get_settings_backend = test_purple_ui_get_settings_backend;
+}
+
+static PurpleUi *
+test_purple_ui_new(void) {
+	return g_object_new(
+		test_purple_ui_get_type(),
+		"id", "test",
+		"name", "Test-UI",
+		"version", VERSION,
+		"website", PURPLE_WEBSITE,
+		"support-website", PURPLE_WEBSITE,
+		"client-type", "test",
+		NULL);
+}
 
 static gboolean
 test_ui_init_history(GError **error) {
@@ -101,7 +131,7 @@ test_ui_init_history(GError **error) {
 
 void
 test_ui_purple_init(void) {
-	PurpleUiInfo *ui_info = NULL;
+	PurpleUi *ui = NULL;
 	GError *error = NULL;
 
 #ifndef _WIN32
@@ -116,21 +146,12 @@ test_ui_purple_init(void) {
 	/* set the magic PURPLE_PLUGINS_SKIP environment variable */
 	g_setenv("PURPLE_PLUGINS_SKIP", "1", TRUE);
 
-	/* Set the core-uiops, which is used to
-	 * 	- initialize the ui specific preferences.
-	 * 	- initialize the debug ui.
-	 * 	- initialize the ui components for all the modules.
-	 * 	- uninitialize the ui components for all the modules when the core terminates.
-	 */
-	purple_core_set_ui_ops(&test_core_uiops);
-
-	ui_info = purple_ui_info_new("test", "Test-UI", VERSION, PURPLE_WEBSITE,
-	                             PURPLE_WEBSITE, "test");
+	ui = test_purple_ui_new();
 
 	/* Now that all the essential stuff has been set, let's try to init the core. It's
 	 * necessary to provide a non-NULL name for the current ui to the core. This name
 	 * is used by stuff that depends on this ui, for example the ui-specific plugins. */
-	if (!purple_core_init(ui_info)) {
+	if (!purple_core_init(ui)) {
 		/* Initializing the core failed. Terminate. */
 		fprintf(stderr,
 				"libpurple initialization failed. Dumping core.\n"