Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1107)

Unified Diff: media/filters/ffmpeg_demuxer.cc

Issue 2506533002: Use width,height for coded_{width,height} in AVStreamToVideoDecoderConfig (Closed)
Patch Set: Address comments. Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « media/ffmpeg/ffmpeg_common.cc ('k') | tools/metrics/histograms/histograms.xml » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/filters/ffmpeg_demuxer.cc
diff --git a/media/filters/ffmpeg_demuxer.cc b/media/filters/ffmpeg_demuxer.cc
index 78eba93f68b9c323b7145e310c16884ceaca6851..d13fd71c80fd529835b73d4499defd44f4a4229b 100644
--- a/media/filters/ffmpeg_demuxer.cc
+++ b/media/filters/ffmpeg_demuxer.cc
@@ -161,10 +161,6 @@ static void RecordVideoCodecStats(const VideoDecoderConfig& video_config,
UMA_HISTOGRAM_ENUMERATION("Media.VideoCodecProfile", video_config.profile(),
VIDEO_CODEC_PROFILE_MAX + 1);
}
- UMA_HISTOGRAM_COUNTS_10000("Media.VideoCodedWidth",
- video_config.coded_size().width());
- UmaHistogramAspectRatio("Media.VideoCodedAspectRatio",
- video_config.coded_size());
UMA_HISTOGRAM_COUNTS_10000("Media.VideoVisibleWidth",
video_config.visible_rect().width());
UmaHistogramAspectRatio("Media.VideoVisibleAspectRatio",
@@ -1509,8 +1505,6 @@ void FFmpegDemuxer::LogMetadata(AVFormatContext* avctx,
params.SetString("video_codec_name" + suffix, GetCodecName(video_codec));
params.SetInteger("width" + suffix, video_codec->width);
params.SetInteger("height" + suffix, video_codec->height);
- params.SetInteger("coded_width" + suffix, video_codec->coded_width);
- params.SetInteger("coded_height" + suffix, video_codec->coded_height);
params.SetString("time_base" + suffix,
base::StringPrintf("%d/%d", video_codec->time_base.num,
video_codec->time_base.den));
« no previous file with comments | « media/ffmpeg/ffmpeg_common.cc ('k') | tools/metrics/histograms/histograms.xml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698