aboutsummaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
Diffstat (limited to 'src')
-rw-r--r--src/ephy-extensions-manager.c135
-rw-r--r--src/ephy-extensions-manager.h5
-rw-r--r--src/ephy-python-loader.c12
3 files changed, 49 insertions, 103 deletions
diff --git a/src/ephy-extensions-manager.c b/src/ephy-extensions-manager.c
index e7589be6d..30fdb11f8 100644
--- a/src/ephy-extensions-manager.c
+++ b/src/ephy-extensions-manager.c
@@ -59,7 +59,8 @@
#endif
#define CONF_LOADED_EXTENSIONS "/apps/epiphany/general/active_extensions"
-#define DOT_INI ".ephy-extension"
+#define EE_GROUP "Epiphany Extension"
+#define DOT_INI ".ephy-extension"
#define ENABLE_LEGACY_FORMAT
@@ -84,11 +85,9 @@ struct _EphyExtensionsManagerPrivate
typedef struct
{
EphyExtensionInfo info;
- guint version;
gboolean load_failed;
char *loader_type;
- GData *loader_attributes;
EphyLoader *loader; /* NULL if never loaded */
GObject *extension; /* NULL if unloaded */
@@ -266,13 +265,8 @@ free_extension_info (ExtensionInfo *info)
EphyExtensionInfo *einfo = (EphyExtensionInfo *) info;
g_free (einfo->identifier);
- g_free (einfo->name);
- g_free (einfo->description);
- g_list_foreach (einfo->authors, (GFunc) g_free, NULL);
- g_list_free (einfo->authors);
- g_free (einfo->url);
+ g_key_file_free (einfo->keyfile);
g_free (info->loader_type);
- g_datalist_clear (&info->loader_attributes);
if (info->extension != NULL)
{
@@ -310,10 +304,9 @@ ephy_extensions_manager_load_ini_string (EphyExtensionsManager *manager,
{
ExtensionInfo *info;
EphyExtensionInfo *einfo;
- gchar ** list;
- int i;
GKeyFile *key_file;
GError *err = NULL;
+ char *start_group;
LOG ("Loading INI description file for '%s'", identifier);
@@ -336,89 +329,41 @@ ephy_extensions_manager_load_ini_string (EphyExtensionsManager *manager,
return;
}
- if (g_key_file_has_group (key_file, "Epiphany Extension") == FALSE ||
- g_key_file_has_group (key_file, "Loader") == FALSE)
+ start_group = g_key_file_get_start_group (key_file);
+ if (start_group == NULL ||
+ strcmp (start_group, EE_GROUP) != 0 ||
+ !g_key_file_has_group (key_file, "Loader"))
{
g_warning ("Invalid extension description file for '%s'; "
"missing 'Epiphany Extension' or 'Loader' group",
identifier);
g_key_file_free (key_file);
+ g_free (start_group);
return;
}
+ g_free (start_group);
- info = g_new0 (ExtensionInfo, 1);
- einfo = (EphyExtensionInfo *) info;
- einfo->identifier = g_strdup (identifier);
- g_datalist_init (&info->loader_attributes);
-
- einfo->name = g_key_file_get_locale_string (key_file,
- "Epiphany Extension",
- "Name",
- NULL, NULL);
-
- einfo->description = g_key_file_get_locale_string (key_file,
- "Epiphany Extension",
- "Description",
- NULL, NULL);
-
- einfo->url = g_key_file_get_locale_string (key_file,
- "Epiphany Extension",
- "URL",
- NULL, NULL);
-
- list = g_key_file_get_string_list (key_file,
- "Epiphany Extension",
- "Authors", NULL, NULL);
- if (list)
+ if (!g_key_file_has_key (key_file, EE_GROUP, "Name", NULL) ||
+ !g_key_file_has_key (key_file, EE_GROUP, "Description", NULL))
{
- for (i = 0 ; list[i]; i++ )
- {
- einfo->authors = g_list_prepend (einfo->authors,
- g_strstrip (g_strdup (list[i])));
- }
- g_strfreev (list);
- }
-
- einfo->authors = g_list_reverse (einfo->authors);
-
- info->version = g_key_file_get_integer (key_file,
- "Epiphany Extension",
- "Version", NULL);
-
- /* Load the loader flags */
- list = g_key_file_get_keys (key_file, "Loader", NULL, NULL);
-
- if (list)
- {
- for (i = 0 ; list[i]; i++ )
- {
- char * value;
- GQuark attr_quark = 0;
-
- value = g_key_file_get_string (key_file, "Loader",
- list[i], NULL);
+ g_warning ("Invalid extension description file for '%s'; "
+ "missing 'Name' or 'Description' keys.",
+ identifier);
- if (strcmp (list[i], "Type") == 0)
- {
- info->loader_type = value;
- continue;
- }
-
- attr_quark = g_quark_from_string (list[i]);
+ g_key_file_free (key_file);
+ return;
+ }
- g_datalist_id_set_data_full (&info->loader_attributes,
- attr_quark, value,
- (GDestroyNotify) g_free);
- }
- g_strfreev (list);
- }
+ info = g_new0 (ExtensionInfo, 1);
+ einfo = (EphyExtensionInfo *) info;
+ einfo->identifier = g_strdup (identifier);
+ einfo->keyfile = key_file;
- g_key_file_free (key_file);
+ info->loader_type = g_key_file_get_string (key_file, "Loader", "Type", NULL);
/* sanity check */
- if (einfo->name == NULL || einfo->description == NULL ||
- info->loader_type == NULL || info->loader_type[0] == '\0')
+ if (info->loader_type == NULL || info->loader_type[0] == '\0')
{
free_extension_info (info);
return;
@@ -566,8 +511,8 @@ get_loader_for_type (EphyExtensionsManager *manager,
{
LoaderInfo *info;
GList *l;
- GData *attr = NULL;
- char *path, *name, *stype;
+ char *path, *name, *stype, *data;
+ GKeyFile *keyfile;
EphyLoader *shlib_loader;
GObject *loader;
@@ -608,21 +553,27 @@ get_loader_for_type (EphyExtensionsManager *manager,
#endif
}
- stype = sanitise_type (type);
- name = g_strconcat ("lib", stype, "loader.", G_MODULE_SUFFIX, NULL);
- path = g_build_filename (LOADER_DIR, name, NULL);
- g_datalist_init (&attr);
- g_datalist_set_data (&attr, "library", path);
-
shlib_loader = get_loader_for_type (manager, "shlib");
g_return_val_if_fail (shlib_loader != NULL, NULL);
- loader = ephy_loader_get_object (shlib_loader, &attr);
- g_datalist_clear (&attr);
+ stype = sanitise_type (type);
+ name = g_strconcat ("lib", stype, "loader.", G_MODULE_SUFFIX, NULL);
+ path = g_build_filename (LOADER_DIR, name, NULL);
+ data = g_strconcat ("[Loader]\nType=shlib\nLibrary=", path, "\n", NULL);
g_free (stype);
g_free (name);
g_free (path);
+ keyfile = g_key_file_new ();
+ if (!g_key_file_load_from_data (keyfile, data, strlen (data), 0, NULL))
+ {
+ g_free (data);
+ return NULL;
+ }
+
+ loader = ephy_loader_get_object (shlib_loader, keyfile);
+ g_key_file_free (keyfile);
+
if (EPHY_IS_LOADER (loader))
{
info = g_new (LoaderInfo, 1);
@@ -671,8 +622,7 @@ load_extension (EphyExtensionsManager *manager,
if (info->load_failed) return;
/* get a loader */
- loader = get_loader_for_type (manager,
- info->loader_type);
+ loader = get_loader_for_type (manager, info->loader_type);
if (loader == NULL)
{
g_message ("No loader found for extension '%s' of type '%s'\n",
@@ -682,8 +632,7 @@ load_extension (EphyExtensionsManager *manager,
info->loader = loader;
- info->extension = ephy_loader_get_object (loader,
- &info->loader_attributes);
+ info->extension = ephy_loader_get_object (loader, info->info.keyfile);
/* attach if the extension implements EphyExtensionIface */
if (EPHY_IS_EXTENSION (info->extension))
diff --git a/src/ephy-extensions-manager.h b/src/ephy-extensions-manager.h
index 22e70a8d7..2e3d1a07b 100644
--- a/src/ephy-extensions-manager.h
+++ b/src/ephy-extensions-manager.h
@@ -44,10 +44,7 @@ typedef struct _EphyExtensionsManagerPrivate EphyExtensionsManagerPrivate;
typedef struct
{
char *identifier;
- char *name;
- char *description;
- GList *authors;
- char *url;
+ GKeyFile *keyfile;
gboolean active;
} EphyExtensionInfo;
diff --git a/src/ephy-python-loader.c b/src/ephy-python-loader.c
index ff23bd7ce..1e0e697c8 100644
--- a/src/ephy-python-loader.c
+++ b/src/ephy-python-loader.c
@@ -39,16 +39,14 @@ static GObjectClass *parent_class = NULL;
static GObject *
impl_get_object (EphyLoader *eloader,
- GData **attributes)
+ GKeyFile *keyfile)
{
char *filename;
GObject *object;
- filename = g_datalist_get_data (attributes, "Module");
- if (filename == NULL)
- {
- filename = g_datalist_get_data (attributes, "module");
- }
+ g_return_val_if_fail (keyfile != NULL, NULL);
+
+ filename = g_key_file_get_string (keyfile, "Loader", "Module", NULL);
if (filename == NULL)
{
g_warning ("NULL module name!\n");
@@ -59,6 +57,8 @@ impl_get_object (EphyLoader *eloader,
"filename", filename,
NULL);
+ g_free (filename);
+
/* we own one ref */
return g_object_ref (object);
}