Minor cleanups in MediaManager: shorten MediaManager::GetUserMedia(), better LOG macros

RESOLVED FIXED in Firefox 66

Status

()

defect
P2
minor
RESOLVED FIXED
6 months ago
5 months ago

People

(Reporter: jib, Assigned: jib)

Tracking

Trunk
mozilla66
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox65 wontfix, firefox66 fixed)

Details

Attachments

(4 attachments)

No description provided.
Summary: Minor cleanups in MediaManager, shorten MediaManager::GetUserMedia(), better LOG macros → Minor cleanups in MediaManager: shorten MediaManager::GetUserMedia(), better LOG macros

Comment 5

5 months ago
Pushed by jbruaroey@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/616721cad893
Fix some nits and shorten MediaManager::GetUserMedia(). r=achronop

Comment 6

5 months ago
Pushed by jbruaroey@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/9a68c5f1ca0d
Make LOG macros clang-format friendlier. r=padenot

Comment 7

5 months ago
Pushed by jbruaroey@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/9463a1a3d790
Add out-parameter to EnumerateDevicesImpl(); clearer semantics and flattens things. r=pehrsons

Comment 8

5 months ago
Pushed by jbruaroey@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/abe934612616
Less verbose code in MediaManager in places. r=pehrsons

Comment 9

5 months ago
Backout by dluca@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/4ef33177d5ba
Backed out 4 changesets for bmsvc build bustage

Comment 10

5 months ago
Pushed by jbruaroey@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/995a65365aa1
Fix some nits and shorten MediaManager::GetUserMedia(). r=achronop

Comment 11

5 months ago
Pushed by jbruaroey@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2f227a365a67
Make LOG macros clang-format friendlier. r=padenot

Comment 12

5 months ago
Pushed by jbruaroey@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/79b1f09eb5c5
Add out-parameter to EnumerateDevicesImpl(); clearer semantics and flattens things. r=pehrsons

Comment 13

5 months ago
Pushed by jbruaroey@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/dff4f40ef703
Less verbose code in MediaManager in places. r=pehrsons
Blocks: 1497175
You need to log in before you can comment on or make changes to this bug.