809950 - remove obvious static initializers from webrtc code

Status: RESOLVED FIXED

(Core :: WebRTC, defect)

Description

[Nathan Froyd [:froydnj]]

The logging code and the wrapper code macros both create static initializers
with abandon.  Fortunately they are easy to fix.

Comment 1

[Nathan Froyd [:froydnj]]

Attachment: fix mtransport logging macros to not introduce static initializers

It's tricky to figure out a way to use LogCtx pointers in a way that won't
introduce static initializers...so let's not.  Change the macros to use
something very close to the pattern used to fix bug 806618.

Comment 2

[Nathan Froyd [:froydnj]]

Attachment: fix webrtc signaling Wrapper code to not introduce static initializers

The Wrapper code declares statics; let's hide those behind a function so that
they don't get initialized at library import time, but only when the WebRTC
Wrapper code actually gets invoked.

Comment 3

[Nathan Froyd [:froydnj]]

remote:   https://hg.mozilla.org/integration/mozilla-inbound/rev/5905e2a217b9
remote:   https://hg.mozilla.org/integration/mozilla-inbound/rev/244ced46ac11

Comment 4

[(no longer active)]

https://hg.mozilla.org/mozilla-central/rev/5905e2a217b9
https://hg.mozilla.org/mozilla-central/rev/244ced46ac11