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

Side by Side Diff: webrtc/api/video/i420_buffer.h

Issue 2622263002: Delete deprecated and transitional stuff related to video frame refactoring. (Closed)
Patch Set: Fix include order. Add dependency in api/video_codecs/BUILD.gn. Created 3 years, 7 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
« no previous file with comments | « webrtc/api/mediastreaminterface.h ('k') | webrtc/api/video/i420_buffer.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 /* 1 /*
2 * Copyright (c) 2015 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2015 The WebRTC project authors. All Rights Reserved.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license 4 * Use of this source code is governed by a BSD-style license
5 * that can be found in the LICENSE file in the root of the source 5 * that can be found in the LICENSE file in the root of the source
6 * tree. An additional intellectual property rights grant can be found 6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may 7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree. 8 * be found in the AUTHORS file in the root of the source tree.
9 */ 9 */
10 10
(...skipping 35 matching lines...) Expand 10 before | Expand all | Expand 10 after
46 static void SetBlack(I420Buffer* buffer); 46 static void SetBlack(I420Buffer* buffer);
47 47
48 // Sets all three planes to all zeros. Used to work around for 48 // Sets all three planes to all zeros. Used to work around for
49 // quirks in memory checkers 49 // quirks in memory checkers
50 // (https://bugs.chromium.org/p/libyuv/issues/detail?id=377) and 50 // (https://bugs.chromium.org/p/libyuv/issues/detail?id=377) and
51 // ffmpeg (http://crbug.com/390941). 51 // ffmpeg (http://crbug.com/390941).
52 // TODO(nisse): Deprecated. Should be deleted if/when those issues 52 // TODO(nisse): Deprecated. Should be deleted if/when those issues
53 // are resolved in a better way. Or in the mean time, use SetBlack. 53 // are resolved in a better way. Or in the mean time, use SetBlack.
54 void InitializeData(); 54 void InitializeData();
55 55
56 // TODO(nisse): Deprecated, use static method instead.
57 void SetToBlack() { SetBlack(this); }
58
59 int width() const override; 56 int width() const override;
60 int height() const override; 57 int height() const override;
61 const uint8_t* DataY() const override; 58 const uint8_t* DataY() const override;
62 const uint8_t* DataU() const override; 59 const uint8_t* DataU() const override;
63 const uint8_t* DataV() const override; 60 const uint8_t* DataV() const override;
64 61
65 int StrideY() const override; 62 int StrideY() const override;
66 int StrideU() const override; 63 int StrideU() const override;
67 int StrideV() const override; 64 int StrideV() const override;
68 65
(...skipping 12 matching lines...) Expand all
81 int crop_width, 78 int crop_width,
82 int crop_height); 79 int crop_height);
83 80
84 // The common case of a center crop, when needed to adjust the 81 // The common case of a center crop, when needed to adjust the
85 // aspect ratio without distorting the image. 82 // aspect ratio without distorting the image.
86 void CropAndScaleFrom(const VideoFrameBuffer& src); 83 void CropAndScaleFrom(const VideoFrameBuffer& src);
87 84
88 // Scale all of |src| to the size of |this| buffer, with no cropping. 85 // Scale all of |src| to the size of |this| buffer, with no cropping.
89 void ScaleFrom(const VideoFrameBuffer& src); 86 void ScaleFrom(const VideoFrameBuffer& src);
90 87
91 // TODO(nisse): Deprecated, delete once downstream applications are updated.
92 // Returns a rotated versions of |src|. Native buffers are not
93 // supported. The reason this function doesn't return an I420Buffer,
94 // is that it returns |src| unchanged in case |rotation| is zero.
95 static rtc::scoped_refptr<VideoFrameBuffer> Rotate(
96 rtc::scoped_refptr<VideoFrameBuffer> src,
97 VideoRotation rotation);
98
99 protected: 88 protected:
100 I420Buffer(int width, int height); 89 I420Buffer(int width, int height);
101 I420Buffer(int width, int height, int stride_y, int stride_u, int stride_v); 90 I420Buffer(int width, int height, int stride_y, int stride_u, int stride_v);
102 91
103 ~I420Buffer() override; 92 ~I420Buffer() override;
104 93
105 private: 94 private:
106 const int width_; 95 const int width_;
107 const int height_; 96 const int height_;
108 const int stride_y_; 97 const int stride_y_;
109 const int stride_u_; 98 const int stride_u_;
110 const int stride_v_; 99 const int stride_v_;
111 const std::unique_ptr<uint8_t, AlignedFreeDeleter> data_; 100 const std::unique_ptr<uint8_t, AlignedFreeDeleter> data_;
112 }; 101 };
113 102
114 } // namespace webrtc 103 } // namespace webrtc
115 104
116 #endif // WEBRTC_API_VIDEO_I420_BUFFER_H_ 105 #endif // WEBRTC_API_VIDEO_I420_BUFFER_H_
OLDNEW
« no previous file with comments | « webrtc/api/mediastreaminterface.h ('k') | webrtc/api/video/i420_buffer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698