1087187 - Reduce verbose logs in GStreamerReader::GetBuffered

Status: RESOLVED WONTFIX

(Core :: Audio/Video: Playback, defect, P5)

Description

[JW Wang [:jwwang] [:jw_wang]]

Got pipes of logs such as:
GStreamerReader(7f35402c9000) adding range [0.000000, 135.886190] for [0 23970324] size 25593096
GStreamerReader(7f35402c9000) adding range [0.000000, 135.886190] for [0 23970324] size 25593096
GStreamerReader(7f35402c9000) adding range [0.000000, 135.886190] for [0 23970324] size 25593096
GStreamerReader(7f35402c9000) adding range [0.000000, 135.886190] for [0 23970324] size 25593096
GStreamerReader(7f35402c9000) adding range [0.000000, 135.886190] for [0 23970324] size 25593096
GStreamerReader(7f35402c9000) complete range [0, 323.778333] for [0, 25593096]
GStreamerReader(7f35402c9000) complete range [0, 323.778333] for [0, 25593096]
GStreamerReader(7f35402c9000) complete range [0, 323.778333] for [0, 25593096]
GStreamerReader(7f35402c9000) complete range [0, 323.778333] for [0, 25593096]
GStreamerReader(7f35402c9000) complete range [0, 323.778333] for [0, 25593096]

when playing video with NSPR_LOG_MODULES="MediaDecoder:4".

We should reduce the log level of the messages in GStreamerReader::GetBuffered().

Comment 1

[Ralph Giles (:rillian)]

4 (PR_LOG_DEBUG) is already the most verbose logging level. Maybe a better fix is to change the log key, e.g. NSPR_LOG_MODLES="GstreamerReader:4" instead of MediaDecoder?

Comment 2

[JW Wang [:jwwang] [:jw_wang]]

I would like to use (PR_LOG_DEBUG+1) as http://hg.mozilla.org/mozilla-central/file/305d24174ace/content/media/mediasource/SourceBuffer.cpp#l31.

In fact there is bug 999898 to add another log level which is pended for a while. There are also several places in our codebase that just use (PR_LOG_DEBUG+1) or (5) for the verbose messages.

Comment 3

[Jean-Yves Avenard [:jya]]

gstreamer is going in bug 1234092