1235679 - [Media] UPnP Dev app to search UPnP server and play media content

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia, defect)

Description

[Punam Dahiya [:pdahiya]]

UPnP test app to search for UPnP media servers and browse and play media files. APP leverages work done and SSDP service API implemented as part of Bug 751172. 

Bug 1060250 implements DLNA player for TV exposing a more polished interface for running on smart TV.

Comment 1

[Punam Dahiya [:pdahiya]]

Attachment: PR to implement UPnP dev app

Comment 2

[Punam Dahiya [:pdahiya]]

Comment on attachment 8702773 [details] [review]
PR to implement UPnP dev app

Hi Jim, David

Setting review flag for the dev app  tested on Flame, Aries. As briefly described in original description of the bug DLNA player under tv_apps leverages similar APIs to discover UPnP server in same network. However I failed to run it on TV simulator and decided to do a basic dev app that 
a) Discover uPnP Servers
b) Plays media files when browsed and clicked on same screen
c) Shows response logs returned from uPnP services.

I would like to use this proof of concept to explore further gecko support and best approach to stream and display media files across uPnP devices. Thanks!

Comment 3

[David Flanagan [:djf]]

When I try using this app with Kodi, it finds the server right away, and I can stream videos from my macbook to my aries with it.  Kodi won't show me any of my music however: it just shows empty Artists, Albums, etc. folders.

When I try with Logitech Media Server, this app can't seem to find the server. But if I stop and restart the server while the app is running then it is found, and I can stream photos, music and videos from my macbook to my aries device.

So the plug-and-play bit isn't as automatic as it should be, but it does seem to be mostly working.

The app itself is clearly a test app. A little CSS and HTML could clean it up quite a bit. Putting the browsing UI in a scrolling element would be nice so that I don't have to scroll all the way down to the bottom to play the media.  Setting autoplay on the music player would also be good.

Those are all just nits, and shouldn't stop us from landing this.

In order to try this app out, I had to use WebIDE to push it to the phone.  This does not work for me:

  APP=uPnP-test make install-gaia 

Maybe that is because it is in dev-apps? It reports an error about the manifest. Maybe I just need to update my node version or something.  Punam: if you can install with make install-gaia then this seems fine. If you can't install it, then it seems to me that we should figure out that issue before landing.

Comment 4

[David Flanagan [:djf]]

Comment on attachment 8702773 [details] [review]
PR to implement UPnP dev app

Consider changing the font size so the buttons are a little easier to push.

I recommend adding a toplevel README file that includes links to the upstream version of the plug.play.js library and the ssdp.js library

I did not review any of the library code, since this is just going into dev_apps. Before we landed a non-test app, I'd want the libraries reviewed as well.

Thanks for creating this, Punam!

Comment 5

[Punam Dahiya [:pdahiya]]

(In reply to David Flanagan [:djf] from comment #3)
> When I try using this app with Kodi, it finds the server right away, and I
> can stream videos from my macbook to my aries with it.  Kodi won't show me
> any of my music however: it just shows empty Artists, Albums, etc. folders.
> 
> When I try with Logitech Media Server, this app can't seem to find the
> server. But if I stop and restart the server while the app is running then
> it is found, and I can stream photos, music and videos from my macbook to my
> aries device.
> 
> So the plug-and-play bit isn't as automatic as it should be, but it does
> seem to be mostly working.
> 
> The app itself is clearly a test app. A little CSS and HTML could clean it
> up quite a bit. Putting the browsing UI in a scrolling element would be nice
> so that I don't have to scroll all the way down to the bottom to play the
> media.  Setting autoplay on the music player would also be good.
> 
> Those are all just nits, and shouldn't stop us from landing this.
> 
> In order to try this app out, I had to use WebIDE to push it to the phone. 
> This does not work for me:
> 
>   APP=uPnP-test make install-gaia 
> 
> Maybe that is because it is in dev-apps? It reports an error about the
> manifest. Maybe I just need to update my node version or something.  Punam:
> if you can install with make install-gaia then this seems fine. If you can't
> install it, then it seems to me that we should figure out that issue before
> landing.

Adding upnp-test to build/config/phone/apps-engineering.list helps resolve this issue.

Comment 6

[Punam Dahiya [:pdahiya]]

Landed on master
https://github.com/mozilla-b2g/gaia/commit/7e86151abd048cf0e669f2a3bf17c179a5bd1223

Comment 7

[GitHub Autolander Bot]

Attachment: [gaia] punamdahiya:Bug_1235679 > mozilla-b2g:master

Comment 8

[Punam Dahiya [:pdahiya]]

Comment on attachment 8704825 [details] [review]
[gaia] punamdahiya:Bug_1235679 > mozilla-b2g:master

Updated ReadMe file with UPnP Test App bug Id and landed on master

https://github.com/mozilla-b2g/gaia/commit/2cdef3da4d03d8677b471836d47884bac39e6bc6