903540 - ICY streams not unloaded correctly

Status: RESOLVED INACTIVE

(Core :: Audio/Video: Playback, defect)

Description

[(not reading bugmail) Nick Desaulniers [:\n]]

Testing on a Unagi with a 1.2 build from August 1st.

STR:
1. Open browser app
2. Navigate to `50.117.121.10/;`, do not forget the `/;`
3. ShoutCAST mp3 stream over ICY protocol should play (this stream could theoretically go down in the future, it's not mine)
4. Open new tab and close the ShoutCAST tab (or restart the browser app)
5. In new tab, browse back to `50.117.121.10/;`

Expected:
ShoutCAST stream should play again

Actual:
Stream will not play again until device is restarted


I don't know anything about the media layers, but I recall hearing that mp3 files that were being decoded in hardware (on device) needed to be manually unloaded. Ex.

audio.src = null;
audio.load();

I'm not sure that that is the issue, but if it is, then this bug should probably be moved to the gaia browser component.

Comment 1

[(not reading bugmail) Nick Desaulniers [:\n]]

Also, I have not tested it, but this could be the case for mp3s in general in the browser app (when they are loaded as the only resource in the URI, not via the source attribute of an audio element).

Comment 2

[cajbir (:cajbir)]

Are you able to try STR on a non-ICY stream? I have an mp3 at http://cd.pn/a but don't have a gonk device handy to try.

Comment 3

[(not reading bugmail) Nick Desaulniers [:\n]]

@doublec, I just tested your mp3 on 1.2 and 1.0.1 and it worked, no problems.  So that rules out gaia::browser issues.

Comment 4

[Ivan Tsay (:ITsay)]

Nick, could you give us more information about why this is nominated to 1.2

Comment 5

[(not reading bugmail) Nick Desaulniers [:\n]]

After further testing this isn't a huge issue.  When using an Audio element, there are no problems.  It only seems to affect Gaia browser when navigating directly to a stream.  ICY support is important for third party internet radio app developers.

Comment 6

[Jason Smith [:jsmith]]

(In reply to Nick Desaulniers [:\n] from comment #5)
> After further testing this isn't a huge issue.  When using an Audio element,
> there are no problems.  It only seems to affect Gaia browser when navigating
> directly to a stream.  ICY support is important for third party internet
> radio app developers.

I'll followup over email on this. I need more details if this is blocking a target 1.2 partner or not.

Comment 7

[Jason Smith [:jsmith]]

Talked with Nick offline - this isn't a blocker for 1.2, only cause it impacts use cases when browsing directly to a stream.

Comment 8

[Nils Ohlmeier [:drno]]

Mass closing do to inactivity.
Feel free to re-open if still needed.