[epiphany] ephy-web-view: Rename set_loading_title as set_loading_message
- From: Carlos Garcia Campos <carlosgc src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [epiphany] ephy-web-view: Rename set_loading_title as set_loading_message
- Date: Mon, 10 Mar 2014 10:17:35 +0000 (UTC)
commit 1e38e858a809585a62b0a6d116719314d4c68e7a
Author: Carlos Garcia Campos <cgarcia igalia com>
Date: Tue Mar 4 11:02:30 2014 +0100
ephy-web-view: Rename set_loading_title as set_loading_message
This is no longer used to set a loading title, since it's always called
with a URL or NULL. Also make it static since it's only used by
ephy-web-view.c. Remove the public getter since it's unused too.
https://bugzilla.gnome.org/show_bug.cgi?id=725649
embed/ephy-embed-private.h | 3 -
embed/ephy-web-view.c | 89 +++++++++++++++++--------------------------
embed/ephy-web-view.h | 1 -
3 files changed, 35 insertions(+), 58 deletions(-)
---
diff --git a/embed/ephy-embed-private.h b/embed/ephy-embed-private.h
index 890f791..34d648b 100644
--- a/embed/ephy-embed-private.h
+++ b/embed/ephy-embed-private.h
@@ -49,9 +49,6 @@ G_BEGIN_DECLS
void ephy_web_view_set_visit_type (EphyWebView *view,
EphyHistoryPageVisitType visit_type);
EphyHistoryPageVisitType ephy_web_view_get_visit_type (EphyWebView *view);
-void ephy_web_view_set_loading_title (EphyWebView *view,
- const char *title,
- gboolean is_address);
void ephy_web_view_popups_manager_reset (EphyWebView *view);
void ephy_web_view_save (EphyWebView *view,
const char *uri);
diff --git a/embed/ephy-web-view.c b/embed/ephy-web-view.c
index 7889167..7756634 100644
--- a/embed/ephy-web-view.c
+++ b/embed/ephy-web-view.c
@@ -81,8 +81,8 @@ struct _EphyWebViewPrivate {
char *address;
char *typed_address;
char *title;
- char *loading_title;
char *status_message;
+ char *loading_message;
char *link_message;
GdkPixbuf *icon;
@@ -766,7 +766,7 @@ ephy_web_view_finalize (GObject *object)
g_free (priv->title);
g_free (priv->status_message);
g_free (priv->link_message);
- g_free (priv->loading_title);
+ g_free (priv->loading_message);
G_OBJECT_CLASS (ephy_web_view_parent_class)->finalize (object);
}
@@ -1513,6 +1513,36 @@ restore_zoom_level (EphyWebView *view,
(EphyHistoryJobCallback)get_host_for_url_cb, view);
}
+/**
+ * ephy_web_view_set_loading_message:
+ * @view: an #EphyWebView
+ * @address: the loading address
+ *
+ * Update @view's loading message
+ **/
+static void
+ephy_web_view_set_loading_message (EphyWebView *view,
+ const char *address)
+{
+ EphyWebViewPrivate *priv = view->priv;
+
+ g_clear_pointer (&priv->loading_message, g_free);
+ if (address) {
+ char *title;
+
+ title = get_title_from_address (address);
+
+ if (title != NULL && title[0] != '\0') {
+ /* translators: %s here is the address of the web page */
+ priv->loading_message = g_strdup_printf (_ ("Loading “%s”…"), title);
+ } else {
+ priv->loading_message = g_strdup (_("Loading…"));
+ }
+
+ g_free (title);
+ }
+}
+
static void
ephy_web_view_location_changed (EphyWebView *view,
const char *location)
@@ -1542,7 +1572,7 @@ ephy_web_view_location_changed (EphyWebView *view,
} else {
/* We do this to get rid of an eventual password in the URL. */
ephy_web_view_set_address (view, location);
- ephy_web_view_set_loading_title (view, location, TRUE);
+ ephy_web_view_set_loading_message (view, location);
}
ephy_web_view_set_link_message (view, NULL);
@@ -1585,10 +1615,9 @@ load_changed_cb (WebKitWebView *web_view,
if (priv->address == NULL || priv->address[0] == '\0')
ephy_web_view_set_address (view, loading_uri);
- ephy_web_view_set_loading_title (view, loading_uri, TRUE);
-
g_free (priv->status_message);
priv->status_message = g_strdup (priv->loading_title);
+ ephy_web_view_set_loading_message (view, loading_uri);
g_object_notify (object, "status-message");
/* Zoom level. */
@@ -1641,8 +1670,8 @@ load_changed_cb (WebKitWebView *web_view,
case WEBKIT_LOAD_FINISHED:
g_free (priv->status_message);
priv->status_message = NULL;
+ ephy_web_view_set_loading_message (view, NULL);
g_object_notify (object, "status-message");
- ephy_web_view_set_loading_title (view, NULL, FALSE);
if (priv->is_blank || !webkit_web_view_get_title (web_view))
ephy_web_view_set_title (view, NULL);
@@ -2191,40 +2220,6 @@ ephy_web_view_get_title (EphyWebView *view)
}
/**
- * ephy_web_view_set_loading_title:
- * @view: an #EphyWebView
- * @title: new loading title for @view
- * @is_address: %TRUE if @title is an address
- *
- * Update @view's loading title to @title, if @is_address is %TRUE it will
- * retrieve the title of the page at @title.
- **/
-void
-ephy_web_view_set_loading_title (EphyWebView *view,
- const char *title,
- gboolean is_address)
-{
- EphyWebViewPrivate *priv = view->priv;
- char *freeme = NULL;
-
- g_free (priv->loading_title);
- priv->loading_title = NULL;
-
- if (is_address && title) {
- title = freeme = get_title_from_address (title);
- }
-
- if (title != NULL && title[0] != '\0') {
- /* translators: %s here is the address of the web page */
- priv->loading_title = g_strdup_printf (_ ("Loading “%s”…"), title);
- } else {
- priv->loading_title = g_strdup (_ ("Loading…"));
- }
-
- g_free (freeme);
-}
-
-/**
* ephy_web_view_is_loading:
* @view: an #EphyWebView
*
@@ -2257,20 +2252,6 @@ ephy_web_view_load_failed (EphyWebView *view)
}
/**
- * ephy_web_view_get_loading_title:
- * @view: an #EphyWebView
- *
- * Returns the loading title for @view.
- *
- * Return value: the provisional title of @view while loading
- **/
-const char *
-ephy_web_view_get_loading_title (EphyWebView *view)
-{
- return view->priv->loading_title;
-}
-
-/**
* ephy_web_view_get_icon:
* @view: an #EphyWebView
*
diff --git a/embed/ephy-web-view.h b/embed/ephy-web-view.h
index 910ef81..5d13c5d 100644
--- a/embed/ephy-web-view.h
+++ b/embed/ephy-web-view.h
@@ -121,7 +121,6 @@ void ephy_web_view_load_url (EphyWebView
const char *url);
gboolean ephy_web_view_is_loading (EphyWebView *view);
gboolean ephy_web_view_load_failed (EphyWebView *view);
-const char * ephy_web_view_get_loading_title (EphyWebView *view);
GdkPixbuf * ephy_web_view_get_icon (EphyWebView *view);
EphyWebViewDocumentType ephy_web_view_get_document_type (EphyWebView *view);
EphyWebViewNavigationFlags ephy_web_view_get_navigation_flags (EphyWebView *view);
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]