Merge lp:~rodrigo-moya/libubuntuone/initial-page into lp:libubuntuone

[Rodrigo Moya]

Show a temporary view when loading the initial store page

Show a temporary view when loading the initial store page

[John Lenton (chipaca)]

Nice!
When are we _()'ing all these strings?

review: Approve

[Martin Albisetti (beuno)]

review: Approve

[Rodrigo Moya (rodrigo-moya)]

The attempt to merge lp:~rodrigo-moya/libubuntuone/initial-page into lp:libubuntuone failed.Below is the output from the failed tests.

/usr/bin/gnome-autogen.sh
checking for autoconf >= 2.53...
(B testing autoconf2.50... not found.
  testing autoconf... found 2.65
checking for automake >= 1.6...
(B testing automake-1.11... found 1.11.1
checking for libtool >= 1.5...
(B testing libtoolize... found 2.2.6b
checking for glib-gettext >= 2.2.0...
(B testing glib-gettextize... found 2.23.5
checking for intltool >= 0.30...
(B testing intltoolize... found 0.41.0
checking for pkg-config >= 0.14.0...
(B testing pkg-config... found 0.22
checking for gtk-doc >= 1.0...
(B testing gtkdocize... found 1.11
Checking for required M4 macros...
(BChecking for forbidden M4 macros...
(BProcessing ./configure.ac
(BRunning libtoolize...
(Blibtoolize: putting auxiliary files in `.'.
libtoolize: copying file `./ltmain.sh'
libtoolize: You should add the contents of the following files to `aclocal.m4':
libtoolize: `/usr/share/aclocal/libtool.m4'
libtoolize: `/usr/share/aclocal/ltoptions.m4'
libtoolize: `/usr/share/aclocal/ltversion.m4'
libtoolize: `/usr/share/aclocal/ltsugar.m4'
libtoolize: `/usr/share/aclocal/lt~obsolete.m4'
libtoolize: Consider adding `AC_CONFIG_MACRO_DIR([m4])' to configure.ac and
libtoolize: rerunning libtoolize, to keep the correct libtool macros in-tree.
libtoolize: Consider adding `-I m4' to ACLOCAL_AMFLAGS in Makefile.am.
Running glib-gettextize... Ignore non-fatal messages.
(BCopying file mkinstalldirs
Copying file po/Makefile.in.in

Please add the files
  codeset.m4 gettext.m4 glibc21.m4 iconv.m4 isc-posix.m4 lcmessage.m4
  progtest.m4
from the /aclocal directory to your autoconf macro directory
or directly to your aclocal.m4 file.
You will also need config.guess and config.sub, which you can get from
ftp://ftp.gnu.org/pub/gnu/config/.

Running intltoolize...
(BRunning gtkdocize...
(BRunning aclocal-1.11...
(BRunning autoconf...
(BRunning autoheader...
(BRunning automake-1.11...
(BRunning ./configure --prefix=/usr ...
(Bchecking for a BSD-compatible install... /usr/bin/install -c
checking whether build environment is sane... yes
checking for a thread-safe mkdir -p... /bin/mkdir -p
checking for gawk... gawk
checking whether make sets $(MAKE)... yes
checking for style of include used by make... GNU
checking for gcc... gcc
checking whether the C compiler works... yes
checking for C compiler default output file name... a.out
checking for suffix of executables...
checking whether we are cross compiling... no
checking for suffix of object files... o
checking whether we are using the GNU C compiler... yes
checking whether gcc accepts -g... yes
checking for gcc option to accept ISO C89... none needed
checking dependency style of gcc... gcc3
checking how to run the C preprocessor... gcc -E
checking for grep that handles long lines and -e... /bin/grep
checking for egrep... /bin/grep -E
checking for ANSI C header files... yes
checking for sys/types.h... yes
checking for sys/stat.h... yes...

[Rodrigo Moya (rodrigo-moya)]

The attempt to merge lp:~rodrigo-moya/libubuntuone/initial-page into lp:libubuntuone failed.Below is the output from the failed tests.

/usr/bin/gnome-autogen.sh
checking for autoconf >= 2.53...
(B testing autoconf2.50... not found.
  testing autoconf... found 2.65
checking for automake >= 1.6...
(B testing automake-1.11... found 1.11.1
checking for libtool >= 1.5...
(B testing libtoolize... found 2.2.6b
checking for glib-gettext >= 2.2.0...
(B testing glib-gettextize... found 2.23.5
checking for intltool >= 0.30...
(B testing intltoolize... found 0.41.0
checking for pkg-config >= 0.14.0...
(B testing pkg-config... found 0.22
checking for gtk-doc >= 1.0...
(B testing gtkdocize... found 1.11
Checking for required M4 macros...
(BChecking for forbidden M4 macros...
(BProcessing ./configure.ac
(BRunning libtoolize...
(Blibtoolize: putting auxiliary files in `.'.
libtoolize: copying file `./ltmain.sh'
libtoolize: Consider adding `AC_CONFIG_MACRO_DIR([m4])' to configure.ac and
libtoolize: rerunning libtoolize, to keep the correct libtool macros in-tree.
libtoolize: Consider adding `-I m4' to ACLOCAL_AMFLAGS in Makefile.am.
Running glib-gettextize... Ignore non-fatal messages.
(BCopying file mkinstalldirs
Copying file po/Makefile.in.in

Please add the files
  codeset.m4 gettext.m4 glibc21.m4 iconv.m4 isc-posix.m4 lcmessage.m4
  progtest.m4
from the /aclocal directory to your autoconf macro directory
or directly to your aclocal.m4 file.
You will also need config.guess and config.sub, which you can get from
ftp://ftp.gnu.org/pub/gnu/config/.

Running intltoolize...
(BRunning gtkdocize...
(BRunning aclocal-1.11...
(BRunning autoconf...
(BRunning autoheader...
(BRunning automake-1.11...
(BRunning ./configure --prefix=/usr ...
(Bchecking for a BSD-compatible install... /usr/bin/install -c
checking whether build environment is sane... yes
checking for a thread-safe mkdir -p... /bin/mkdir -p
checking for gawk... gawk
checking whether make sets $(MAKE)... yes
checking for style of include used by make... GNU
checking for gcc... gcc
checking whether the C compiler works... yes
checking for C compiler default output file name... a.out
checking for suffix of executables...
checking whether we are cross compiling... no
checking for suffix of object files... o
checking whether we are using the GNU C compiler... yes
checking whether gcc accepts -g... yes
checking for gcc option to accept ISO C89... none needed
checking dependency style of gcc... gcc3
checking how to run the C preprocessor... gcc -E
checking for grep that handles long lines and -e... /bin/grep
checking for egrep... /bin/grep -E
checking for ANSI C header files... yes
checking for sys/types.h... yes
checking for sys/stat.h... yes
checking for stdlib.h... yes
checking for string.h... yes
checking for memory.h... yes
checking for strings.h... yes
checking for inttypes.h... yes
checking for stdint.h... yes
checking for unistd.h... yes
checking minix/config.h usability... no
checking minix/config.h presence... no
checking for minix/config.h... n...

=== modified file 'libubuntuone/u1-music-store.c'
--- libubuntuone/u1-music-store.c	2010-03-11 13:06:15 +0000
+++ libubuntuone/u1-music-store.c	2010-03-12 16:18:18 +0000
@@ -37,6 +37,7 @@
 #define U1_NOT_LOGGED_IN_STORE_URL "/music/store-no-token"
 #define U1_NOT_REGISTERED_URL      "https://one.ubuntu.com/music/notregistered?returnUrl="
 
+#define U1_INITIAL_PAGE "<html><body>Loading Ubuntu One music store</body></html>"
 #define U1_DEFAULT_ERROR_PAGE "<html><body>Could not load Music Store</body></html>"
 #define U1_IN_DEVELOPMENT_PAGE "<html><body>The music store is under development, please be patient</body></html>"
 #define U1_CONNECTING_PAGE "<html><body>Connecting to Ubuntu One. Please wait...</body></html>"
@@ -45,6 +46,8 @@
 	DBusGConnection *bus;
 	gchar *base_url;
 
+	GtkWidget *scroll;
+	GtkWidget *initial_web_viewer;
 	GtkWidget *web_viewer;
 	WebKitWebFrame *frame_to_use;
 
@@ -573,6 +576,18 @@
 	gchar *tmp_url;
 	U1MusicStore *music_store = U1_MUSIC_STORE (user_data);
 
+	/* Remove the initial view if still around */
+	if (music_store->priv->initial_web_viewer != NULL) {
+		gtk_widget_destroy (music_store->priv->initial_web_viewer);
+		music_store->priv->initial_web_viewer = NULL;
+
+		/* And show the real store viewer */
+		g_signal_connect (G_OBJECT (music_store->priv->web_viewer), "navigation-policy-decision-requested",
+				  G_CALLBACK (navigation_requested_cb), music_store);
+		gtk_widget_show (music_store->priv->web_viewer);
+		gtk_container_add (GTK_CONTAINER (music_store->priv->scroll), music_store->priv->web_viewer);
+	}
+
 	/* Inject some JavaScript to enable previews and hide the Flash player */
 	execute_script (web_view, "u1-preview.js", NULL);
 
@@ -601,7 +616,7 @@
 
 	/* Signal users the URL finished loading */
 	g_signal_emit (music_store, u1_music_store_signals[URL_LOADED_SIGNAL], 0,
-		       (const gchar *) webkit_web_view_get_uri (music_store->priv->web_viewer));
+		       (const gchar *) webkit_web_view_get_uri (WEBKIT_WEB_VIEW (music_store->priv->web_viewer)));
 }
 
 static void
@@ -694,11 +709,27 @@
 	return real_url;
 }
 
+static gboolean
+load_real_store_cb (gpointer user_data)
+{
+	gchar *real_url;
+	U1MusicStore *music_store = U1_MUSIC_STORE (user_data);
+
+	g_debug ("Loading the real store page");
+	real_url = get_url_to_use (music_store);
+	if (real_url != NULL) {
+		webkit_web_view_open (WEBKIT_WEB_VIEW (music_store->priv->web_viewer), real_url);
+
+		/* Free memory */
+		g_free (real_url);
+	}
+
+	return FALSE;
+}
+
 static void
 u1_music_store_init (U1MusicStore *music_store)
 {
-	GtkWidget *scroll;
-	gchar *real_url;
 	gchar *new_user_agent;
 	GError *error = NULL;
 
@@ -732,11 +763,11 @@
 	syncdaemon_init ();
 
 	/* Create web viewer object */
-	scroll = gtk_scrolled_window_new (NULL, NULL);
-	gtk_scrolled_window_set_policy (GTK_SCROLLED_WINDOW (scroll),
+	music_store->priv->scroll = gtk_scrolled_window_new (NULL, NULL);
+	gtk_scrolled_window_set_policy (GTK_SCROLLED_WINDOW (music_store->priv->scroll),
 					GTK_POLICY_AUTOMATIC,
 					GTK_POLICY_AUTOMATIC);
-	gtk_widget_show (scroll);
+	gtk_widget_show (music_store->priv->scroll);
 
 	music_store->priv->web_viewer = webkit_web_view_new ();
 
@@ -751,27 +782,28 @@
 
 	g_free(new_user_agent);
 
-	g_signal_connect (G_OBJECT (music_store->priv->web_viewer), "navigation-policy-decision-requested",
-			  G_CALLBACK (navigation_requested_cb), music_store);
 	g_signal_connect (G_OBJECT (music_store->priv->web_viewer), "load-finished",
 			  G_CALLBACK (load_finished_cb), music_store);
 	g_signal_connect (G_OBJECT (music_store->priv->web_viewer), "status-bar-text-changed",
 			  G_CALLBACK (status_bar_text_changed_cb), music_store);
 	g_signal_connect (G_OBJECT (music_store->priv->web_viewer), "load-error",
 			  G_CALLBACK (load_error_cb), music_store);
-	gtk_widget_show (music_store->priv->web_viewer);
-	gtk_container_add (GTK_CONTAINER (scroll), music_store->priv->web_viewer);
-
-
-	real_url = get_url_to_use (music_store);
-	if (real_url != NULL) {
-		webkit_web_view_open (WEBKIT_WEB_VIEW (music_store->priv->web_viewer), real_url);
-
-		/* Free memory */
-		g_free (real_url);
-	}
-
-	gtk_box_pack_start (GTK_BOX (music_store), scroll, TRUE, TRUE, 3);
+
+	/* Show a temp view when loading the initial store page */
+	music_store->priv->initial_web_viewer = webkit_web_view_new ();
+	g_object_set (G_OBJECT (webkit_web_view_get_settings (WEBKIT_WEB_VIEW (music_store->priv->initial_web_viewer))),
+		      "enable-default-context-menu", g_getenv ("U1SHOWCONTEXTMENU") != NULL,
+		      NULL);
+	webkit_web_view_load_string (WEBKIT_WEB_VIEW (music_store->priv->initial_web_viewer),
+				     U1_INITIAL_PAGE,
+				     "text/html", "utf-8", "file:///");
+	gtk_widget_show (music_store->priv->initial_web_viewer);
+	gtk_container_add (GTK_CONTAINER (music_store->priv->scroll), music_store->priv->initial_web_viewer);
+
+	/* And load the real store in the background */
+	g_idle_add ((GSourceFunc) load_real_store_cb, music_store);
+
+	gtk_box_pack_start (GTK_BOX (music_store), music_store->priv->scroll, TRUE, TRUE, 3);
 }
 
 /**