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

Status: RESOLVED FIXED

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

Comment 1

[Jan-Ivar Bruaroey [:jib] (needinfo? me)]

Attachment: Bug 1512280 - Fix some nits and shorten MediaManager::GetUserMedia().

Comment 2

[Jan-Ivar Bruaroey [:jib] (needinfo? me)]

Attachment: Bug 1512280 - Make LOG macros clang-format friendlier.

Comment 3

[Jan-Ivar Bruaroey [:jib] (needinfo? me)]

Attachment: Bug 1512280 - Add out-parameter to EnumerateDevicesImpl(); clearer semantics and flattens things. r?pehrsons

Comment 4

[Jan-Ivar Bruaroey [:jib] (needinfo? me)]

Attachment: Bug 1512280 - Less verbose code in MediaManager in places. r?pehrsons

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/616721cad893
Fix some nits and shorten MediaManager::GetUserMedia(). r=achronop

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9a68c5f1ca0d
Make LOG macros clang-format friendlier. r=padenot

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9463a1a3d790
Add out-parameter to EnumerateDevicesImpl(); clearer semantics and flattens things. r=pehrsons

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/abe934612616
Less verbose code in MediaManager in places. r=pehrsons

Comment 9

[Pulsebot]

Backout by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/4ef33177d5ba
Backed out 4 changesets for bmsvc build bustage

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/995a65365aa1
Fix some nits and shorten MediaManager::GetUserMedia(). r=achronop

Comment 11

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/2f227a365a67
Make LOG macros clang-format friendlier. r=padenot

Comment 12

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/79b1f09eb5c5
Add out-parameter to EnumerateDevicesImpl(); clearer semantics and flattens things. r=pehrsons

Comment 13

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/dff4f40ef703
Less verbose code in MediaManager in places. r=pehrsons

Comment 14

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/mozilla-central/rev/995a65365aa1
https://hg.mozilla.org/mozilla-central/rev/2f227a365a67
https://hg.mozilla.org/mozilla-central/rev/79b1f09eb5c5
https://hg.mozilla.org/mozilla-central/rev/dff4f40ef703