Closed Bug 916609 Opened 7 years ago Closed 7 years ago
RTC permission UI broken on Fx26 ("browser is not defined)
Must have been landed and uplifted recently... Beta (24) is not affected.
Regression from http://hg.mozilla.org/mozilla-central/rev/cf373e408a6b via bug 882145 Specifically http://hg.mozilla.org/mozilla-central/diff/cf373e408a6b/browser/modules/webrtcUI.jsm
Ummmm... bug 882145 is only on central, not Aurora
This is reproducible (for me) on mozilla-central (Nightly) 09/15 only. I just installed Aurora (09/15) and am not seeing any issue.
We need to backout bug 882145 then if that's the confirmed regression.
Confirmed the same thing Aaron saw - reproduces on Fx26 Nightly, doesn't reproduce on Fx25 Aurora.
I can confirm also that this was working in yesterday's nightly in the build here: ftp://ftp.mozilla.org/pub/mobile/tinderbox-builds/mozilla-central-android/1379154724/ I think Aaron is right - this looks like a regression from bug 882145.
Looks like bug 882145 updated the desktop UI but the Android fork of the UI (which apparently has no tests??) needs similar changes. There's no reason to back anything out. We just need to patch http://hg.mozilla.org/mozilla-central/file/9366ee039645/mobile/android/chrome/content/WebrtcUI.js#l76 with a patch like http://hg.mozilla.org/mozilla-central/diff/cf373e408a6b/browser/modules/webrtcUI.jsm And the add a test, for goodness' sake.
Assignee: nobody → ryanvm
Bug 882145 backed out.
This leaves the non-handling of .picture as it was. And again, this needs tests; if it's not tested a Tinderbox, it's not reasonable to expect people to not break it, or to magically know when it's fixed..
Attachment #805064 - Flags: feedback?(aaron.train) → feedback?(gpascutto)
Apparently both my android devices got "stuck" on Aurora 25a2 around or on 8/2; and apparently gUM was broken in Aurora on that rev. (I normally run nightly, and I was also testing Beta in the run-up to release, but not Aurora.)
Attachment #805064 - Flags: feedback?(rjesup) → feedback+
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
No longer need to track since the patch that regressed this was backed out. We might need to move this patch attached here over to bug 882145.
Comment on attachment 805064 [details] [diff] [review] This should do it Thanks guys for acting so quickly on this, and bz for the quick patch! Since this issue has been closed as fixed, I've moved the patch over to bug 882145 as jsmith suggested. My bad for not noticing there were two webrtcUI files :-(
You need to log in before you can comment on or make changes to this bug.