Merge lp:~sil/libubuntuone/better-user-agent into lp:libubuntuone

Proposed by Stuart Langridge
Status: Merged
Approved by: Rodrigo Moya
Approved revision: 48
Merged at revision: not available
Proposed branch: lp:~sil/libubuntuone/better-user-agent
Merge into: lp:libubuntuone
Diff against target: 31 lines (+10/-1)
1 file modified
libubuntuone/u1-music-store.c (+10/-1)
To merge this branch: bzr merge lp:~sil/libubuntuone/better-user-agent
Reviewer Review Type Date Requested Status
Rodrigo Moya (community) Approve
Review via email: mp+20529@code.launchpad.net

Commit message

Add ourselves to the user agent, because the 7d servers browser-detect based on user agent and carefully don't serve required JavaScript to us if they don't recognise our user agent

To post a comment you must log in.
Revision history for this message
Stuart Langridge (sil) wrote :

Add ourselves to the user agent, because the 7d servers browser-detect based on user agent and carefully don't serve required JavaScript to us if they don't recognise our user agent. ASP.NET fail.

Revision history for this message
Rodrigo Moya (rodrigo-moya) wrote :

Looks good

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'libubuntuone/u1-music-store.c'
--- libubuntuone/u1-music-store.c 2010-03-02 10:07:48 +0000
+++ libubuntuone/u1-music-store.c 2010-03-03 11:19:13 +0000
@@ -574,6 +574,7 @@
574{574{
575 GtkWidget *scroll;575 GtkWidget *scroll;
576 gchar *real_url;576 gchar *real_url;
577 gchar *new_user_agent;
577578
578 music_store->priv = g_new0 (U1MusicStorePrivate, 1);579 music_store->priv = g_new0 (U1MusicStorePrivate, 1);
579580
@@ -588,10 +589,18 @@
588 gtk_widget_show (scroll);589 gtk_widget_show (scroll);
589590
590 music_store->priv->web_viewer = webkit_web_view_new ();591 music_store->priv->web_viewer = webkit_web_view_new ();
592
593 new_user_agent = g_strdup_printf("%s U1MusicStore/" VERSION ,
594 webkit_web_settings_get_user_agent (webkit_web_view_get_settings (WEBKIT_WEB_VIEW (music_store->priv->web_viewer)))
595 );
596 g_debug(new_user_agent);
597
591 g_object_set (G_OBJECT (webkit_web_view_get_settings (WEBKIT_WEB_VIEW (music_store->priv->web_viewer))),598 g_object_set (G_OBJECT (webkit_web_view_get_settings (WEBKIT_WEB_VIEW (music_store->priv->web_viewer))),
592 "user-agent", "U1MusicStore widget (" VERSION ")",599 "user-agent", new_user_agent,
593 NULL);600 NULL);
594601
602 g_free(new_user_agent);
603
595 g_signal_connect (G_OBJECT (music_store->priv->web_viewer), "navigation-policy-decision-requested",604 g_signal_connect (G_OBJECT (music_store->priv->web_viewer), "navigation-policy-decision-requested",
596 G_CALLBACK (navigation_requested_cb), music_store);605 G_CALLBACK (navigation_requested_cb), music_store);
597 g_signal_connect (G_OBJECT (music_store->priv->web_viewer), "load-finished",606 g_signal_connect (G_OBJECT (music_store->priv->web_viewer), "load-finished",

Subscribers

People subscribed via source and target branches