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

Side by Side Diff: webrtc/video/vie_encoder.h

Issue 2853463004: Don't reinit encoder when rotation changes. (Closed)
Patch Set: Add trivial unit tests. 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
OLDNEW
1 /* 1 /*
2 * Copyright (c) 2012 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2012 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 115 matching lines...) Expand 10 before | Expand all | Expand 10 after
126 126
127 private: 127 private:
128 class ConfigureEncoderTask; 128 class ConfigureEncoderTask;
129 class EncodeTask; 129 class EncodeTask;
130 class VideoSourceProxy; 130 class VideoSourceProxy;
131 131
132 class VideoFrameInfo { 132 class VideoFrameInfo {
133 public: 133 public:
134 VideoFrameInfo(int width, 134 VideoFrameInfo(int width,
135 int height, 135 int height,
136 VideoRotation rotation,
137 bool is_texture) 136 bool is_texture)
138 : width(width), 137 : width(width),
139 height(height), 138 height(height),
140 rotation(rotation),
141 is_texture(is_texture) {} 139 is_texture(is_texture) {}
142 int width; 140 int width;
143 int height; 141 int height;
144 VideoRotation rotation;
145 bool is_texture; 142 bool is_texture;
146 int pixel_count() const { return width * height; } 143 int pixel_count() const { return width * height; }
147 }; 144 };
148 145
149 void ConfigureEncoderOnTaskQueue(VideoEncoderConfig config, 146 void ConfigureEncoderOnTaskQueue(VideoEncoderConfig config,
150 size_t max_data_payload_length, 147 size_t max_data_payload_length,
151 bool nack_enabled); 148 bool nack_enabled);
152 void ReconfigureEncoder(); 149 void ReconfigureEncoder();
153 150
154 void ConfigureQualityScaler(); 151 void ConfigureQualityScaler();
(...skipping 103 matching lines...) Expand 10 before | Expand all | Expand 10 after
258 // All public methods are proxied to |encoder_queue_|. It must must be 255 // All public methods are proxied to |encoder_queue_|. It must must be
259 // destroyed first to make sure no tasks are run that use other members. 256 // destroyed first to make sure no tasks are run that use other members.
260 rtc::TaskQueue encoder_queue_; 257 rtc::TaskQueue encoder_queue_;
261 258
262 RTC_DISALLOW_COPY_AND_ASSIGN(ViEEncoder); 259 RTC_DISALLOW_COPY_AND_ASSIGN(ViEEncoder);
263 }; 260 };
264 261
265 } // namespace webrtc 262 } // namespace webrtc
266 263
267 #endif // WEBRTC_VIDEO_VIE_ENCODER_H_ 264 #endif // WEBRTC_VIDEO_VIE_ENCODER_H_
OLDNEW
« no previous file with comments | « webrtc/modules/video_coding/codecs/vp9/test/vp9_impl_unittest.cc ('k') | webrtc/video/vie_encoder.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698