1241690 - Reduce noisy error messages for unconnected PeerConnections

Status: RESOLVED FIXED

(Core :: WebRTC: Networking, defect, P3)

Comment 1

[Nils Ohlmeier [:drno]]

Attachment: MozReview Request: Bug 1241690: reduce logging output for unconnected PCs. r=bwc

Review commit: https://reviewboard.mozilla.org/r/31863/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/31863/

Comment 2

[Byron Campen [:bwc]]

Comment on attachment 8710752 [details]
MozReview Request: Bug 1241690: reduce logging output for unconnected PCs. r=bwc

https://reviewboard.mozilla.org/r/31863/#review28673

::: media/mtransport/nricemediastream.cpp:311
(Diff revision 1)
> +    return NS_ERROR_NOT_AVAILABLE;

I kinda feel like this should be NS_OK. There simply are no pairs.

Comment 3

[Nils Ohlmeier [:drno]]

Comment on attachment 8710752 [details]
MozReview Request: Bug 1241690: reduce logging output for unconnected PCs. r=bwc

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/31863/diff/1-2/

Comment 4

[Nils Ohlmeier [:drno]]

Comment on attachment 8710752 [details]
MozReview Request: Bug 1241690: reduce logging output for unconnected PCs. r=bwc

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/31863/diff/2-3/

Comment 5

[Nils Ohlmeier [:drno]]

Comment on attachment 8710752 [details]
MozReview Request: Bug 1241690: reduce logging output for unconnected PCs. r=bwc

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/31863/diff/3-4/

Comment 6

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/05f94b82da01

Comment 7

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/05f94b82da01