Open Bug 965140 Opened 10 years ago Updated 10 years ago

MWC demo: WebRTC demo test page

Categories

(Marketing :: General, task)

x86_64
Windows 7
task
Not set
normal

Tracking

(Not tracked)

People

(Reporter: mreavy, Assigned: lstowasser)

References

Details

We will probably want to re-skin a WebRTC web page (e.g. apprtc or talky.io) to be mobile friendly.

Use cases to demo:

1) Audio call ("Vonage" use case) - Connect 2 devices on an audio call. Prefer 2 firefox OS devices (or at least 1 fxOS device and 1 desktop).
2) Video all ("Skype" use case) - Connect 2 fxOS devices in a video call. (or at least 1 fxOS device and 1 desktop). Needs robust wireless connection.

Script is coming at the end of this week.
3rd use case (optional): DataChannels -- Do we want to demo sending blobs/text/pictures between phones?  Not sure that it's an interesting enough use case for a demo.
Additional guidance from Chris Lee (head of FxOS product) on the demo:

"I think the phone<>Desktop case is going to the most interesting and also the most compelling as we tell users they'll have the ability to easily connect with those on any Firefox (or modern) browser.  It speaks to the larger product message that people are expecting to hear from us."

Chris and Peter agree that we'll only have time for one video call from the reference device to desktop.
Going to leave updates here:

Basing off AppRTC as recommended by EKR. So far re-skinned to fit a Peak. Phone to Desktop works great for audio but so far no video and I believe the same will go far DataChannels.

Will need instructions for a specific build of Firefox OS that supports video and data-channels if intending to use it for the demo.
Assignee: nobody → lstowasser
Also will need hosting. AppRTC was originally intended for Google App Engine but a VPS would probably be fine too.
DataChannels should work if Audio works.  Both are enabled in v1.3.  If they don't, I can get someone to help you debug it.

If you're using the current trunk (which will become v1.4), video should work if you disable the camera permission prompt. (The camera permission prompt requires bug 853356, which is about to land but hasn't yet).  

We may want to disable the permission prompts anyway for the demo;  I see them potentially being annoying and adding no value to the demo.

NOTE: The Peak doesn't have the resources to deliver reasonable video calling performance.  We should get you a hotter device (like a Nexus 4) if you don't have one already.

Thanks for jumping on this so quickly, Louis.
This is currently blocked on the fix for bug 853356.  

Louis -- would you feel comfortable applying the current patch on that bug and seeing if it works for you?  If you do, please comment in bug 853356 whether it works for you or not.  Thanks.
Depends on: 853356
Flags: needinfo?(lstowasser)
Also adding a dependency on the nuwa bug.  Nuwa is required for v1.3.
Depends on: 966802
We could do a custom build without Nuwa and with permission prompts disabled for the demo if we want to remove dependencies on bug 853356 and bug 966802.
Comments in response to some above points on this bug:

* We are planning to disable nuwa for MWC builds - I've ran this by product already
* Data channels is working fine for me after bug 952220 landed. Louis - Are you running the latest 1.4 build?
* The QA contracting team did try testing video calls with the permission prompt disabled, but it didn't work. We still weren't getting the camera to come up.
* We do have an open risk of not having a complete understanding if the problem in bug 962338 is app-based or an actual bug in gecko. If it's our bug, then we risk a 40% audio communication failure rate in a FxOS --> FxDesktop call if we follow a similar setup Google has.

Action items needed from this for MWC:

* I need someone to investigate why disabling the permission prompt for camera isn't allowing the camera to come up
* I need firm concrete evidence proving or disproving that bug 962338 is a bug in gecko
Going to try building with the patch. Will report back in this thread.

Just to be sure, the patches are in this comment: https://bugzilla.mozilla.org/show_bug.cgi?id=853356#c251 ?

@jsmith I was running the latest 1.4. Haven't tested Data Channels yet, just the video. Will test today.
Flags: needinfo?(lstowasser) → needinfo?(mreavy)
(In reply to Louis Stowasser [:louisstow] from comment #10)
> Going to try building with the patch. Will report back in this thread.
> 
> Just to be sure, the patches are in this comment:
> https://bugzilla.mozilla.org/show_bug.cgi?id=853356#c251 ?
> 
> @jsmith I was running the latest 1.4. Haven't tested Data Channels yet, just
> the video. Will test today.

Hmm okay. When I tested this, it was before nuwa landed on 1.3. I'll also test on the latest 1.4 just to be sure with my test sites.
Data Channels is working on 1.4. Will add a feature to send text messages.
(In reply to Louis Stowasser [:louisstow] from comment #10)
> Going to try building with the patch. Will report back in this thread.
> 
> Just to be sure, the patches are in this comment:
> https://bugzilla.mozilla.org/show_bug.cgi?id=853356#c251 ?
> 
> @jsmith I was running the latest 1.4. Haven't tested Data Channels yet, just
> the video. Will test today.

FYI - This patch has landed now & can be used off of the latest b2g inbound build (tomorrow in PST it will be in the daily m-c build), so you should be unblocked here.
Flags: needinfo?(mreavy)
Depends on: 970183
You need to log in before you can comment on or make changes to this bug.