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

Unified Diff: media/base/video_decoder_config.h

Issue 10832087: Remove VideoDecoderConfig.frame_rate_xxx() & VideoFrame:Get/SetDuration() (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address more CR comments. Created 8 years, 5 months 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 | « content/renderer/media/rtc_video_decoder.cc ('k') | media/base/video_decoder_config.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/base/video_decoder_config.h
diff --git a/media/base/video_decoder_config.h b/media/base/video_decoder_config.h
index 894eb241437b66f28e5a9cf53fedf0ffa9c76e93..9f5785c3ad44b463917f2e1f2a9390675cf68d82 100644
--- a/media/base/video_decoder_config.h
+++ b/media/base/video_decoder_config.h
@@ -72,7 +72,6 @@ class MEDIA_EXPORT VideoDecoderConfig {
VideoFrame::Format format,
const gfx::Size& coded_size,
const gfx::Rect& visible_rect,
- int frame_rate_numerator, int frame_rate_denominator,
int aspect_ratio_numerator, int aspect_ratio_denominator,
const uint8* extra_data, size_t extra_data_size);
@@ -84,7 +83,6 @@ class MEDIA_EXPORT VideoDecoderConfig {
VideoFrame::Format format,
const gfx::Size& coded_size,
const gfx::Rect& visible_rect,
- int frame_rate_numerator, int frame_rate_denominator,
int aspect_ratio_numerator, int aspect_ratio_denominator,
const uint8* extra_data, size_t extra_data_size,
bool record_stats);
@@ -121,14 +119,6 @@ class MEDIA_EXPORT VideoDecoderConfig {
// into account.
gfx::Size natural_size() const;
- // Frame rate in seconds expressed as a fraction.
- //
- // This information is required to properly timestamp video frames for
- // codecs that contain repeated frames, such as found in H.264's
- // supplemental enhancement information.
- int frame_rate_numerator() const;
- int frame_rate_denominator() const;
-
// Aspect ratio of the decoded video frame expressed as a fraction.
//
// TODO(scherkus): think of a better way to avoid having video decoders
@@ -151,9 +141,6 @@ class MEDIA_EXPORT VideoDecoderConfig {
gfx::Rect visible_rect_;
gfx::Size natural_size_;
- int frame_rate_numerator_;
- int frame_rate_denominator_;
-
int aspect_ratio_numerator_;
int aspect_ratio_denominator_;
« no previous file with comments | « content/renderer/media/rtc_video_decoder.cc ('k') | media/base/video_decoder_config.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698