Remove VideoDecoderConfig::aspect_ratio_xxx methods.

media/mp4/mp4_stream_parser.cc

Index: media/mp4/mp4_stream_parser.cc
diff --git a/media/mp4/mp4_stream_parser.cc b/media/mp4/mp4_stream_parser.cc
index c7a1c4dbafe985926d73624c67bfea5c7d7f7c87..10b2d43c49337ed21e5de237bccf470c828e8341 100644
--- a/media/mp4/mp4_stream_parser.cc
+++ b/media/mp4/mp4_stream_parser.cc
@@ -11,6 +11,7 @@
 #include "media/base/audio_decoder_config.h"
 #include "media/base/stream_parser_buffer.h"
 #include "media/base/video_decoder_config.h"
+#include "media/base/video_util.h"
 #include "media/mp4/box_definitions.h"
 #include "media/mp4/box_reader.h"
 #include "media/mp4/es_descriptor.h"
@@ -219,14 +220,16 @@ bool MP4StreamParser::ParseMoov(BoxReader* reader) {
       RCHECK(EmitKeyNeeded(entry.sinf.info.track_encryption));
 
       // TODO(strobe): Recover correct crop box
+      gfx::Size coded_size(entry.width, entry.height);
+      gfx::Rect visible_rect(coded_size);
+      gfx::Size natural_size = GetNaturalSize(visible_rect,
+                                              entry.pixel_aspect.h_spacing,
+                                              entry.pixel_aspect.v_spacing);
       video_config.Initialize(kCodecH264, H264PROFILE_MAIN,  VideoFrame::YV12,
-                              gfx::Size(entry.width, entry.height),
-                              gfx::Rect(0, 0, entry.width, entry.height),
-                              entry.pixel_aspect.h_spacing,
-                              entry.pixel_aspect.v_spacing,
+                              coded_size, visible_rect, natural_size,
                               // No decoder-specific buffer needed for AVC;
                               // SPS/PPS are embedded in the video stream
-                              NULL, 0, false);
+                              NULL, 0, true);

[acolwell GONE FROM CHROMIUM, 2012/08/01 17:29:23]

This being false was a bug.

       has_video_ = true;
       video_track_id_ = track->header.track_id;
     }