summaryrefslogtreecommitdiff
path: root/embed
diff options
context:
space:
mode:
authorPhaedrus Leeds <mwleeds@protonmail.com>2022-02-23 15:43:50 -0800
committerPhaedrus Leeds <mwleeds@protonmail.com>2022-03-21 13:07:44 -0700
commit752aca5ae9a1ab7f85d486f3853d2450345cf5cf (patch)
treeed9bd628b6ca6ec101004410ea75d2f1a13e62d7 /embed
parent097e0e3fb6c71aa080112d011c58f74292508a3c (diff)
downloadepiphany-752aca5ae9a1ab7f85d486f3853d2450345cf5cf.tar.gz
Fix web app icon url vs path confusion
Confusingly, the icon_url member of EphyWebApplication is actually a path not a URL or URI. The only place where it gets set to a URI is in prefs-general-page.c when the user changes the icon, and there it is happening erroneously since all the other code that deals with it assumes it is a path. So, rename the struct member and ensure all the use of it treats it as a path.
Diffstat (limited to 'embed')
-rw-r--r--embed/ephy-about-handler.c16
1 files changed, 8 insertions, 8 deletions
diff --git a/embed/ephy-about-handler.c b/embed/ephy-about-handler.c
index cd8b1c381..fc0459c97 100644
--- a/embed/ephy-about-handler.c
+++ b/embed/ephy-about-handler.c
@@ -264,8 +264,8 @@ handle_applications_finished_cb (EphyAboutHandler *handler,
for (p = applications; p; p = p->next) {
EphyWebApplication *app = (EphyWebApplication *)p->data;
- const char *icon_url;
- g_autofree char *encoded_icon_url = NULL;
+ const char *icon_path = NULL;
+ g_autofree char *encoded_icon_path = NULL;
g_autofree char *encoded_name = NULL;
g_autofree char *encoded_url = NULL;
g_autofree char *js_encoded_id = NULL;
@@ -281,12 +281,12 @@ handle_applications_finished_cb (EphyAboutHandler *handler,
/* In the sandbox we don't have access to the host side icon file */
if (ephy_is_running_inside_sandbox ())
- icon_url = app->tmp_icon_url;
+ icon_path = app->tmp_icon_path;
else
- icon_url = app->icon_url;
+ icon_path = app->icon_path;
- if (!icon_url) {
- g_warning ("Failed to get icon url for app %s", app->id);
+ if (!icon_path) {
+ g_warning ("Failed to get icon path for app %s", app->id);
continue;
}
@@ -297,7 +297,7 @@ handle_applications_finished_cb (EphyAboutHandler *handler,
* anything at all, so those need to be encoded for sure. Install date
* should be fine because it's constructed by Epiphany.
*/
- encoded_icon_url = ephy_encode_for_html_attribute (icon_url);
+ encoded_icon_path = ephy_encode_for_html_attribute (icon_path);
encoded_name = ephy_encode_for_html_entity (app->name);
encoded_url = ephy_encode_for_html_entity (app->url);
g_string_append_printf (data_str,
@@ -307,7 +307,7 @@ handle_applications_finished_cb (EphyAboutHandler *handler,
"<td class=\"input\"><input type=\"button\" value=\"%s\" onclick=\"deleteWebApp('%s');\" "
"class=\"destructive-action\"></td>"
"<td class=\"date\">%s <br /> %s</td></tr></tbody>",
- app->id, encoded_icon_url, encoded_name, encoded_url, _("Delete"), app->id,
+ app->id, encoded_icon_path, encoded_name, encoded_url, _("Delete"), app->id,
/* Note for translators: this refers to the installation date. */
_("Installed on:"), install_date);
}