[epiphany/mcatanzaro/webapp-profile] web-app-utils: Stop looking under old profile dir for web apps
- From: Michael Catanzaro <mcatanzaro src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [epiphany/mcatanzaro/webapp-profile] web-app-utils: Stop looking under old profile dir for web apps
- Date: Sat, 16 Mar 2019 16:02:17 +0000 (UTC)
commit a13d34183073f2968c2f58c99c3bb376852bc586
Author: Michael Catanzaro <mcatanzaro igalia com>
Date: Sat Mar 16 10:56:30 2019 -0500
web-app-utils: Stop looking under old profile dir for web apps
This code doesn't make any sense because web apps are no longer stored
under the default profile dir. It's also probably harmless, because the
code should not be called except when running the default browser
instance with the default profile dir, in which case it already behaved
properly, but still, it's weird and confusing and should be removed.
lib/ephy-web-app-utils.c | 27 +++++++++++----------------
1 file changed, 11 insertions(+), 16 deletions(-)
---
diff --git a/lib/ephy-web-app-utils.c b/lib/ephy-web-app-utils.c
index fbb7554cd..f9cbece40 100644
--- a/lib/ephy-web-app-utils.c
+++ b/lib/ephy-web-app-utils.c
@@ -560,22 +560,18 @@ ephy_web_application_get_application_list_internal (gboolean only_legacy)
GFileEnumerator *children = NULL;
GFileInfo *info;
GList *applications = NULL;
- GFile *dot_dir;
- char *default_dot_dir;
- g_autofree char *profile_base = NULL;
-
- default_dot_dir = !ephy_profile_dir_is_default () ? ephy_default_profile_dir () : NULL;
- if (only_legacy) {
- profile_base = g_build_filename (g_get_user_config_dir (), "epiphany", NULL);
- dot_dir = g_file_new_for_path (profile_base);
- } else {
- profile_base = g_strdup (g_get_user_data_dir ());
- dot_dir = g_file_new_for_path (default_dot_dir ? default_dot_dir : g_get_user_data_dir ());
- }
- children = g_file_enumerate_children (dot_dir,
+ g_autofree char *parent_directory_path = NULL;
+ g_autoptr(GFile) parent_directory = NULL;
+
+ if (only_legacy)
+ parent_directory_path = g_build_filename (g_get_user_config_dir (), "epiphany", NULL);
+ else
+ parent_directory_path = g_strdup (g_get_user_data_dir ());
+
+ parent_directory = g_file_new_for_path (parent_directory_path);
+ children = g_file_enumerate_children (parent_directory,
"standard::name",
0, NULL, NULL);
- g_object_unref (dot_dir);
info = g_file_enumerator_next_file (children, NULL, NULL);
while (info) {
@@ -587,7 +583,7 @@ ephy_web_application_get_application_list_internal (gboolean only_legacy)
EphyWebApplication *app;
char *profile_dir;
- profile_dir = g_build_filename (profile_base, name, NULL);
+ profile_dir = g_build_filename (parent_directory_path, name, NULL);
app = ephy_web_application_for_profile_directory (profile_dir);
if (app) {
if (!only_legacy) {
@@ -609,7 +605,6 @@ ephy_web_application_get_application_list_internal (gboolean only_legacy)
}
g_object_unref (children);
- g_free (default_dot_dir);
return g_list_reverse (applications);
}
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]