media-internals: Differentiate error, info, and debug log messages

media/formats/mpeg/adts_stream_parser.cc

Index: media/formats/mpeg/adts_stream_parser.cc
diff --git a/media/formats/mpeg/adts_stream_parser.cc b/media/formats/mpeg/adts_stream_parser.cc
index beb94350c9c3267a55a30f44cd0609df8f301bfb..062a88940465b1c03b28a5687bd0cd69dbcf5127 100644
--- a/media/formats/mpeg/adts_stream_parser.cc
+++ b/media/formats/mpeg/adts_stream_parser.cc
@@ -69,12 +69,13 @@ int ADTSStreamParser::ParseFrameHeader(const uint8* data,
   if (sync != 0xfff || layer != 0 || frame_length < bytes_read ||
       sample_rate_index >= kADTSFrequencyTableSize ||
       channel_layout_index >= kADTSChannelLayoutTableSize) {
-    MEDIA_LOG(log_cb()) << "Invalid header data :" << std::hex
-                        << " sync 0x" << sync
-                        << " version 0x" << version
-                        << " layer 0x" << layer
-                        << " sample_rate_index 0x" << sample_rate_index
-                        << " channel_layout_index 0x" << channel_layout_index;
+    MEDIA_LOG(log_cb(), ERROR) << "Invalid header data :" << std::hex

[DaleCurtis, 2015/03/31 00:12:26]

Hmm are these truly errors? They won't break Append() just cause it to skip the
frame.

[wolenetz, 2015/03/31 00:30:11]

You're right; not ERROR in all cases. I'll change to DEBUG.

[wolenetz, 2015/03/31 19:20:22]

Done.

+                               << " sync 0x" << sync
+                               << " version 0x" << version
+                               << " layer 0x" << layer
+                               << " sample_rate_index 0x" << sample_rate_index
+                               << " channel_layout_index 0x"
+                               << channel_layout_index;
     return -1;
   }