m.soundcloud.com doesn't show tracks anymore

Bug #1309673 reported by Adnane Belmadiaf
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
webbrowser-app
Fix Released
High
Olivier Tilloy
webbrowser-app (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

not sure what's happening, but for some reason the ajax request doesn't succeed and nothing in the logs

Related branches

Revision history for this message
Adnane Belmadiaf (daker) wrote :
summary: - m.soundcloud.com doesn't tracks work anymore
+ m.soundcloud.com doesn't show tracks work anymore
summary: - m.soundcloud.com doesn't show tracks work anymore
+ m.soundcloud.com doesn't show tracks anymore
Revision history for this message
Pat McGowan (pat-mcgowan) wrote :

May need a UA override

Changed in webbrowser-app:
assignee: nobody → Olivier Tilloy (osomon)
importance: Undecided → High
status: New → Confirmed
Revision history for this message
Olivier Tilloy (osomon) wrote :

I can reliably reproduce the issue (including on desktop if I fake the form factor to be "mobile", otherwise I get redirected to the desktop version of soundcloud.com).

I tried changing the default UA to "Mozilla/5.0 (Linux; Android 4.0.4; MB865 Build/6.7.2-226-EDICS-10) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/34.0.1847.114 Mobile Safari/537.36" (this is the default UA for chrome on android on my Atrix2 where m.soundcloud.com renders well), but this didn’t help, I’m still getting the mostly empty page, so it doesn’t look like a UA override would solve the issue. Requires further investigation.

Revision history for this message
Olivier Tilloy (osomon) wrote :

If I enable JS logging (see https://code.launchpad.net/~osomon/webbrowser-app/handle-js-console-msgs/+merge/214687), I’m seeing the following error:

    [JS] (http://m-a-tb.sndcdn.com/all_b82ea7d.js:313) Uncaught TypeError: Cannot read property 'setItem' of null

The culprit is the following bit of JS:

    window.localStorage.setItem("sc-test-private-browsing","1")

If I enable localStorage (by setting 'preferences.localStorageEnabled' to 'true' on the webview), the page loads correctly. The question is: should we enable localStorage by default in the browser?

Revision history for this message
Olivier Tilloy (osomon) wrote :

Both chrome on android and chromium on desktop enable localStorage by default, so it looks like a safe option.

Changed in webbrowser-app:
status: Confirmed → In Progress
Revision history for this message
Adnane Belmadiaf (daker) wrote :

We will get +5points with localStorage enable by default on the html5test.com

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package webbrowser-app - 0.23+14.10.20140505.1-0ubuntu1

---------------
webbrowser-app (0.23+14.10.20140505.1-0ubuntu1) utopic; urgency=low

  [ Michael Sheldon ]
  * Resolve image URLs beginning with a double slash correctly for
    context menu items (LP: #1311626)

  [ Adnane Belmadiaf ]
  * Enabled passwordEchoEnabled (LP: #1314251)

  [ Alberto Mardegan ]
  * Split UbuntuWebContext into two different components:
    UbuntuWebContext, which is a WebContext derivative with the UA
    overrides for Ubuntu. UbuntuSharedWebContext, which is a singleton
    for UbuntuWebContext .

  [ Olivier Tilloy ]
  * Build the models in a separate static lib, and link the unit tests
    against it. This speeds up build time by avoiding having to
    recompile the models’ source for each unit test.
  * Handle javascript console messages.
  * Escape literal dots in UA override matching regular expressions.
  * Enable localStorage by default in the browser. (LP: #1309673)
  * Ensure that the URL actually changes so that the address bar is
    updated in case the user has entered a new address that redirects to
    where she previously was. (LP: #1306615)
  * Update bzr ignore rules.
 -- Ubuntu daily release <email address hidden> Mon, 05 May 2014 20:47:19 +0000

Changed in webbrowser-app (Ubuntu):
status: New → Fix Released
Olivier Tilloy (osomon)
Changed in webbrowser-app:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Bug attachments

Remote bug watches

Bug watches keep track of this bug in other bug trackers.