From fe64e80f6f486467de76c397e107e0c53498ae0f Mon Sep 17 00:00:00 2001 From: Xan Lopez Date: Thu, 24 Sep 2009 17:30:41 +0300 Subject: ephy-window.c: workaround bug in WebKit progress notification LOAD_FINISHED is notified before progress 100% completed, which makes it very hard to rely on those signals to figure out if a page is still loading or not when syncing progress in a tab. Try to workaround this as best as possible. --- src/ephy-window.c | 15 +++++++++++---- 1 file changed, 11 insertions(+), 4 deletions(-) (limited to 'src/ephy-window.c') diff --git a/src/ephy-window.c b/src/ephy-window.c index 80a37fe60..59a06c72d 100644 --- a/src/ephy-window.c +++ b/src/ephy-window.c @@ -1608,6 +1608,7 @@ sync_tab_load_progress (EphyWebView *view, GParamSpec *pspec, EphyWindow *window gdouble progress; const char *uri; gboolean loading; + gboolean switching_tab = pspec == NULL; if (window->priv->closing) return; @@ -1623,13 +1624,19 @@ sync_tab_load_progress (EphyWebView *view, GParamSpec *pspec, EphyWindow *window for about:blank until the load is finished, so assume NULL here means we are loading about:blank. This might not be rigt :) */ - loading = ephy_web_view_is_loading (view); + /* All the weird checks for progress == 1.0 and !switching_tab + * are because we receive first the LOAD_FINISHED status than + * the 100% progress report, so for progress == 1.0 there's no + * sane way of knowing whether we are still loading or + * not. See https://bugs.webkit.org/show_bug.cgi?id=28851 */ uri = webkit_web_view_get_uri (WEBKIT_WEB_VIEW (view)); - if (loading && (!uri || strcmp (uri, "about:blank") == 0)) + if (!switching_tab && (!uri || strcmp (uri, "about:blank") == 0)) return; progress = webkit_web_view_get_progress (WEBKIT_WEB_VIEW (view)); - if (progress == 1.0 && loading) + loading = ephy_web_view_is_loading (view); + + if (progress == 1.0 && !switching_tab) { window->priv->clear_progress_timeout_id = g_timeout_add (500, @@ -1640,7 +1647,7 @@ sync_tab_load_progress (EphyWebView *view, GParamSpec *pspec, EphyWindow *window /* Do not set progress in the entry if the load is already finished */ gtk_entry_set_progress_fraction (GTK_ENTRY (window->priv->entry), - loading ? progress : 0.0); + loading || (progress == 1.0 && !switching_tab) ? progress : 0.0); } static void -- cgit v1.2.3