1243274 - [TV][WebRTC] Open apprtc will show security error

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::TV::Browser, defect, P1)

Description

[Mike Lien[:mlien]]

[Description]
Open apprtc will show security error

[Reproduce Steps]
1. Launch Browser
2. Navigating to "apprtc.appspot.com"
3. Join the room

[Build Information]
Environment: 2.5 Simulator
Gaia: https://github.com/mozilla-b2g/gaia/commit/8a0d9deb2cd47df25750ed2c6820909e4feac3bb
Gecko: https://hg.mozilla.org/releases/mozilla-b2g44_v2_5/rev/2b6f8826861a846c3d1d98ef595c8dfbf8b59328
Mulet: https://hg.mozilla.org/releases/mozilla-b2g44_v2_5/rev/2b6f8826861a846c3d1d98ef595c8dfbf8b59328 

[Expected Result]
Can open the chat room successfully

[Actual Result]
It will show security error

Comment 1

[Mike Lien[:mlien]]

Attachment: Screenshot from 2016-01-27 13:32:02.png

Comment 2

[Josh Cheng [:josh]]

Hi ShianYow,
Could you help to find someone who can check this issue?

Comment 3

[Shian-Yow Wu [:swu]]

Junior, could you check this issue?  Thanks.

Comment 4

[Junior [inactive]]

I have no idea about the mulet error.
Need some time to figure out, but I guess it's with lower priority for mulet.

For real TV, it causes error since lack of camera and mic. |GetUserMedia| could not success for this website. If we'd like to test streaming for webrtc, it needs another way to investigate.

Therefore, appspot.com is designed to show this message.

Comment 5

[Junior [inactive]]

Hello Mike, Josh,
Since it's a P1 blocker,
I'd like to have your suggestion about the next step.

Comment 6

[Mike Lien[:mlien]]

Then I'll suggest to disable webrtc or at least disable the streaming channel until we can fix it

Comment 7

[GitHub Autolander Bot]

Attachment: [gaia] schien:bug1243274-gum-permission-for-tv-browser > mozilla-b2g:master

Comment 8

[Shih-Chiang Chien [:schien] (UTC+8) (use ni? plz)]

Comment on attachment 8713416 [details] [review]
[gaia] schien:bug1243274-gum-permission-for-tv-browser > mozilla-b2g:master

missing permission in TV browser app for accessing microphone and camera.

Comment 9

[Shih-Chiang Chien [:schien] (UTC+8) (use ni? plz)]

Comment on attachment 8713416 [details] [review]
[gaia] schien:bug1243274-gum-permission-for-tv-browser > mozilla-b2g:master

@evelyn can you take a look at this patch, since @yifan is on PTO.

Comment 10

[Evelyn Hung]

merged into master:

https://github.com/mozilla-b2g/gaia/commit/e5dedb49b2008202a7fb33a14d58159f731da119

Comment 11

[Evelyn Hung]

Josh: please ask partner to pick this patch, and introduce to their browser app. Thanks.

Comment 12

[Evelyn Hung]

SC: you need to a? your patch for uplift. Thanks.

Comment 13

[Shih-Chiang Chien [:schien] (UTC+8) (use ni? plz)]

Comment on attachment 8713416 [details] [review]
[gaia] schien:bug1243274-gum-permission-for-tv-browser > mozilla-b2g:master

NOTE: Please see https://wiki.mozilla.org/Release_Management/B2G_Landing to better understand the B2G approval process and landings.

[Approval Request Comment]
Bug caused by (feature/regressing bug #): no
User impact if declined: cannot use GetUserMedia WebAPI in TV browser
Testing completed: manual test
Risk to taking this patch (and alternatives if risky): no risk
String or UUID changes made by this patch: no

Comment 14

[Josh Cheng [:josh]]

Comment on attachment 8713416 [details] [review]
[gaia] schien:bug1243274-gum-permission-for-tv-browser > mozilla-b2g:master

Approve for TV 2.5

Comment 15

[Daisuke]

Attachment: apprtc_err.png

with patch to browser as below,
+    "audio-capture":{},
+    "video-capture":{}

erorr message is changed, but not works well.

Is this expected behaviour?

Comment 16

[Josh Cheng [:josh]]

Hi SC,
Could you check comment 15 from partner? Thanks!

Comment 17

[Shih-Chiang Chien [:schien] (UTC+8) (use ni? plz)]

The new error message is the expected behavior because Stingray TV doesn't support camera.

Comment 18

[Carsten Book [:Tomcat]]

https://github.com/mozilla-b2g/gaia/commit/0e402b6b8f4b3806f46db5c928a0b287f61349af

Comment 19

[Daisuke]

partner side introduced this patch for V2.5 (16/2/1)