Closed Bug 1410500 Opened 7 years ago Closed 7 years ago

Write unittests for configuring VideoAdapter in VideoConduit

Categories

(Core :: WebRTC: Audio/Video, enhancement, P3)

Product:
Core
Component:
WebRTC: Audio/Video
Version:
58 Branch
Type:
enhancement

Tracking

()

Status:
RESOLVED WORKSFORME
Tracking Flags:
Tracking Status
firefox58 --- affected

People

(Reporter: dminor, Unassigned)

References

(Blocks 1 open bug)

Details

There are a number of places in VideoConduit where we change the VideoAdapter configuration. Bug 1404039 covers OnSinkWantsUpdated, but we need to make sure we have coverage of the other places it is configured. There are some notes from Andreas on this in https://bugzilla.mozilla.org/show_bug.cgi?id=1404039#c8.
I did some manual code coverage checking and with the current set of VideoConduit tests we have coverage of every place that the VideoAdapter is modified.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.