Closed Bug 1329562 Opened 4 years ago Closed 4 years ago

Remove WebRTC checks for Windows Vista

Categories

(Core :: WebRTC: Audio/Video, defect, P2)

51 Branch
defect

Tracking

()

RESOLVED FIXED
mozilla53
Tracking Status
firefox50 --- wontfix
firefox51 --- wontfix
firefox52 --- wontfix
firefox53 --- fixed

People

(Reporter: cpeterson, Assigned: cpeterson)

References

(Depends on 1 open bug)

Details

Attachments

(2 files)

Firefox 52 is the last version to support Windows XP or Vista, so we can remove Windows version checks for XP, Vista, or 7.
Comment on attachment 8824866 [details]
Bug 1329562 - Part 1: Remove WebRTC checks for Vista.

https://reviewboard.mozilla.org/r/103176/#review103902
Attachment #8824866 - Flags: review?(rjesup) → review+
Comment on attachment 8824867 [details]
Bug 1329562 - Part 2: Remove gUM screensharing.allow_on_old_platforms check for XP.

https://reviewboard.mozilla.org/r/103178/#review103904
Attachment #8824867 - Flags: review?(rjesup) → review+
Rank: 25
Priority: -- → P2
Pushed by cpeterson@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/bbe4d2f590a5
Part 1: Remove WebRTC checks for Vista. r=jesup
https://hg.mozilla.org/integration/autoland/rev/2d7af132a40c
Part 2: Remove gUM screensharing.allow_on_old_platforms check for XP. r=jesup
https://hg.mozilla.org/mozilla-central/rev/bbe4d2f590a5
https://hg.mozilla.org/mozilla-central/rev/2d7af132a40c
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
Chris, would this fix benefit from manual testing?
Flags: qe-verify?
Flags: needinfo?(cpeterson)
(In reply to Andrei Vaida, QA [:avaida] – please ni? me from comment #9)
> Chris, would this fix benefit from manual testing?

I don't think manual testing is necessary. This was a small change.
Flags: needinfo?(cpeterson)
(In reply to Chris Peterson [:cpeterson] from comment #10)
> (In reply to Andrei Vaida, QA [:avaida] – please ni? me from comment #9)
> > Chris, would this fix benefit from manual testing?
> 
> I don't think manual testing is necessary. This was a small change.

Thank you for following up!
Flags: qe-verify? → qe-verify-
You need to log in before you can comment on or make changes to this bug.