Closed
Bug 1329562
Opened 7 years ago
Closed 7 years ago
Remove WebRTC checks for Windows Vista
Categories
(Core :: WebRTC: Audio/Video, defect, P2)
Tracking
()
RESOLVED
FIXED
mozilla53
People
(Reporter: cpeterson, Assigned: cpeterson)
References
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 hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
|
||
Comment 5•7 years ago
|
||
| mozreview-review | ||
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 6•7 years ago
|
||
| mozreview-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+
|
|
||
Updated•7 years ago
|
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
|
||
Comment 8•7 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/bbe4d2f590a5 https://hg.mozilla.org/mozilla-central/rev/2d7af132a40c
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
|
||
Comment 9•7 years ago
|
||
Chris, would this fix benefit from manual testing?
Flags: qe-verify?
Flags: needinfo?(cpeterson)
|
Assignee | |
Comment 10•7 years ago
|
||
(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)
|
|
||
Comment 11•7 years ago
|
||
(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.
Description
•