Closed
Bug 1688278
Opened 4 years ago
Closed 4 years ago
Rename 'PDMFactoryImpl' in order to better reflect what it's actually doing.
Categories
(Core :: Audio/Video: Playback, task, P2)
Core
Audio/Video: Playback
Tracking
()
RESOLVED
FIXED
87 Branch
| Tracking | Status | |
|---|---|---|
| firefox87 | --- | fixed |
People
(Reporter: alwu, Assigned: alwu)
References
(Blocks 1 open bug)
Details
Attachments
(2 files)
PDMFactoryImpl is actually not a implementation of PDMFactory, it doesn't provide any ability that what a factory would do, it just simply acts as an initializer for PDMs.
|
Assignee | |
Comment 1•4 years ago
|
||
PDMFactoryImpl is actually not a implementation of PDMFactory, it doesn't provide any ability that what a factory would do, it just simply acts as an initializer for PDMs.
|
|
Assignee | |
Comment 2•4 years ago
|
||
Depends on D102792
Pushed by alwu@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/55d5c8c2969f
part1 : modify and rename 'PDMFactoryImpl' for better reflecting what it's actually doing. r=mattwoodrow
https://hg.mozilla.org/integration/autoland/rev/2a554957105f
part2 : add debug log. r=bryce
|
||
Comment 4•4 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/55d5c8c2969f
https://hg.mozilla.org/mozilla-central/rev/2a554957105f
Status: NEW → RESOLVED
Closed: 4 years ago
status-firefox87:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 87 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•