Fixes a duplication and bad name choice for logging media information.

media/filters/ffmpeg_demuxer.cc

Index: media/filters/ffmpeg_demuxer.cc
diff --git a/media/filters/ffmpeg_demuxer.cc b/media/filters/ffmpeg_demuxer.cc
index 30cb0c0029857c8f19cf0b0d33dc75c143635163..07bb77b9b42949b19a4a778e1f33dde3396bc4ca 100644
--- a/media/filters/ffmpeg_demuxer.cc
+++ b/media/filters/ffmpeg_demuxer.cc
@@ -595,8 +595,6 @@ void FFmpegDemuxer::OnFindStreamInfoDone(const PipelineStatusCB& status_cb,
       media_log_->SetStringProperty("audio_codec_name", codec->name);
     }
 
-    media_log_->SetIntegerProperty("audio_sample_rate",
-                                   audio_codec->sample_rate);

[Ty Overby, 2013/09/05 22:10:57]

audio_codec->sample_rate gives you the same information as
audio_config.samples_per_second() which we already log.  

My reason for choosing to use audio_config is that audio_config is  maintained
by chrome audio while audio_codec is from ffmpeg.

     media_log_->SetIntegerProperty("audio_channels_count",
                                    audio_codec->channels);
     media_log_->SetIntegerProperty("audio_samples_per_second",
@@ -637,7 +635,7 @@ void FFmpegDemuxer::OnFindStreamInfoDone(const PipelineStatusCB& status_cb,
 
   media_log_->SetDoubleProperty("max_duration", max_duration.InSecondsF());
   media_log_->SetDoubleProperty("start_time", start_time_.InSecondsF());
-  media_log_->SetDoubleProperty("filesize_in_bytes",
+  media_log_->SetDoubleProperty("total_bytes",

[Ty Overby, 2013/09/05 22:10:57]

"total_bytes" is used by the network infrastructure, so this will match with the
name provided by the network.

[scherkus (not reviewing), 2013/09/06 20:47:57]

I would remove logging this if the it's already handled by the networking layer
instead of having to match the key name

[Ty Overby, 2013/09/09 17:29:04]

Sometimes the networking layer doesn't report it, (I think this happens when
there is a cache?).

[scherkus (not reviewing), 2013/09/09 17:56:50]

ah I thought by network infrastructure you were referring to
Buffered{DataSource,ResourceLoader}, which is where filesize_in_bytes comes from
but it looks like we don't have much logging there yet

disregard my comment then

                                 static_cast<double>(filesize_in_bytes));
   media_log_->SetIntegerProperty("bitrate", bitrate_);