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

Side by Side Diff: webkit/glue/media/video_renderer_impl.h

Issue 7932005: Reland r101418: Fix aspect ratio and clarify video frame dimensions (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 3 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « media/video/video_decode_engine.h ('k') | webkit/glue/media/video_renderer_impl.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef WEBKIT_GLUE_MEDIA_VIDEO_RENDERER_IMPL_H_ 5 #ifndef WEBKIT_GLUE_MEDIA_VIDEO_RENDERER_IMPL_H_
6 #define WEBKIT_GLUE_MEDIA_VIDEO_RENDERER_IMPL_H_ 6 #define WEBKIT_GLUE_MEDIA_VIDEO_RENDERER_IMPL_H_
7 7
8 #include "media/base/buffers.h" 8 #include "media/base/buffers.h"
9 #include "media/base/filters.h" 9 #include "media/base/filters.h"
10 #include "media/filters/video_renderer_base.h" 10 #include "media/filters/video_renderer_base.h"
(...skipping 50 matching lines...) Expand 10 before | Expand all | Expand 10 after
61 // These two members are used to determine if the |bitmap_| contains 61 // These two members are used to determine if the |bitmap_| contains
62 // an already converted image of the current frame. IMPORTANT NOTE: The 62 // an already converted image of the current frame. IMPORTANT NOTE: The
63 // value of |last_converted_frame_| must only be used for comparison purposes, 63 // value of |last_converted_frame_| must only be used for comparison purposes,
64 // and it should be assumed that the value of the pointer is INVALID unless 64 // and it should be assumed that the value of the pointer is INVALID unless
65 // it matches the pointer returned from GetCurrentFrame(). Even then, just 65 // it matches the pointer returned from GetCurrentFrame(). Even then, just
66 // to make sure, we compare the timestamp to be sure the bits in the 66 // to make sure, we compare the timestamp to be sure the bits in the
67 // |current_frame_bitmap_| are valid. 67 // |current_frame_bitmap_| are valid.
68 media::VideoFrame* last_converted_frame_; 68 media::VideoFrame* last_converted_frame_;
69 base::TimeDelta last_converted_timestamp_; 69 base::TimeDelta last_converted_timestamp_;
70 70
71 // The size of the video. 71 // The natural size of the video.
72 gfx::Size video_size_; 72 gfx::Size natural_size_;
73 73
74 // Whether we're logging video presentation timestamps (PTS). 74 // Whether we're logging video presentation timestamps (PTS).
75 bool pts_logging_; 75 bool pts_logging_;
76 76
77 DISALLOW_COPY_AND_ASSIGN(VideoRendererImpl); 77 DISALLOW_COPY_AND_ASSIGN(VideoRendererImpl);
78 }; 78 };
79 79
80 } // namespace webkit_glue 80 } // namespace webkit_glue
81 81
82 #endif // WEBKIT_GLUE_MEDIA_VIDEO_RENDERER_IMPL_H_ 82 #endif // WEBKIT_GLUE_MEDIA_VIDEO_RENDERER_IMPL_H_
OLDNEW
« no previous file with comments | « media/video/video_decode_engine.h ('k') | webkit/glue/media/video_renderer_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698