Closed
Bug 814807
Opened 12 years ago
Closed 12 years ago
Pref on the gum automation in mozilla-central
Categories
(Core :: WebRTC, defect, P1)
Core
WebRTC
Tracking
()
RESOLVED
FIXED
mozilla20
People
(Reporter: jsmith, Assigned: jesup)
References
Details
(Whiteboard: [getUserMedia][blocking-gum+][qa-])
Attachments
(1 file, 1 obsolete file)
1.37 KB,
patch
|
jesup
:
review+
|
Details | Diff | Splinter Review |
We currently cannot pref on the basic gum automation in bug 781534 because of leaks (bug 814721) and a functional error with OS X opt builds (bug 814718). When those two bugs are fixed, let's pref on the automation on central.
Reporter | ||
Updated•12 years ago
|
Reporter | ||
Updated•12 years ago
|
Whiteboard: [getUserMedia][blocking-gum+]
Comment 1•12 years ago
|
||
Any current tests which leak we should not push to mozilla-central but to the alder branch. As of bug 816489 we do not do final leak checks there.
Comment 2•12 years ago
|
||
We will pref on mochitests for WebRTC once all the leaks have been fixed. That means we will not cherry-pick. But once nothing leaks anymore we will take the tests which have been landed on alder in the meantime.
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → hskupin
Priority: -- → P1
Assignee | ||
Updated•12 years ago
|
Assignee: hskupin → rjesup
Summary: Pref on the gum automation from bug 781534 on mozilla central → Pref on the gum automation in mozilla-central
Assignee | ||
Comment 3•12 years ago
|
||
Moving patch from Bug 802538 to here, carrying forward r=whimboo
Attachment #696457 -
Flags: review+
Reporter | ||
Comment 5•12 years ago
|
||
Comment on attachment 696457 [details] [diff] [review]
Enable getUserMedia() mochitests
This is missing one of the three tests that need to be prefed on. Patch coming to fix this.
Attachment #696457 -
Attachment is obsolete: true
Attachment #696457 -
Flags: review-
Reporter | ||
Comment 6•12 years ago
|
||
Reporter | ||
Updated•12 years ago
|
Attachment #696902 -
Flags: review?(rjesup)
Assignee | ||
Updated•12 years ago
|
Attachment #696902 -
Flags: review?(rjesup) → review+
Reporter | ||
Comment 7•12 years ago
|
||
I *think* we can land this now given the two issues have been resolved that were causing the failures originally - last try run in bug 814718 indicates I *think* we should be okay to land this.
Keywords: checkin-needed
Assignee | ||
Comment 8•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/c78f5379558a
Once more into the fray!
Target Milestone: --- → mozilla20
Reporter | ||
Updated•12 years ago
|
Keywords: checkin-needed
Comment 9•12 years ago
|
||
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Reporter | ||
Updated•12 years ago
|
Whiteboard: [getUserMedia][blocking-gum+] → [getUserMedia][blocking-gum+][qa-]
Reporter | ||
Updated•12 years ago
|
Flags: in-testsuite+
You need to log in
before you can comment on or make changes to this bug.
Description
•