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

Unified Diff: webrtc/modules/video_coding/utility/quality_scaler.h

Issue 2309743002: Move the QP scaling thresholds to the relevant encoders (Closed)
Patch Set: Remove redundant call Created 4 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 side-by-side diff with in-line comments
Download patch
Index: webrtc/modules/video_coding/utility/quality_scaler.h
diff --git a/webrtc/modules/video_coding/utility/quality_scaler.h b/webrtc/modules/video_coding/utility/quality_scaler.h
index c1ae50b06e138b365b190acb61c1a3a66bcc7e92..c0f94409ce30ee0d5b46260857bc42f696f72c01 100644
--- a/webrtc/modules/video_coding/utility/quality_scaler.h
+++ b/webrtc/modules/video_coding/utility/quality_scaler.h
@@ -11,6 +11,7 @@
#ifndef WEBRTC_MODULES_VIDEO_CODING_UTILITY_QUALITY_SCALER_H_
#define WEBRTC_MODULES_VIDEO_CODING_UTILITY_QUALITY_SCALER_H_
+#include "webrtc/common_types.h"
#include "webrtc/common_video/include/i420_buffer_pool.h"
#include "webrtc/modules/video_coding/utility/moving_average.h"
@@ -23,6 +24,11 @@ class QualityScaler {
};
QualityScaler();
+ void Init(VideoCodecType codec_type,
+ int initial_bitrate_kbps,
+ int width,
+ int height,
+ int fps);
void Init(int low_qp_threshold,
int high_qp_threshold,
int initial_bitrate_kbps,
@@ -38,15 +44,6 @@ class QualityScaler {
const rtc::scoped_refptr<VideoFrameBuffer>& frame);
int downscale_shift() const { return downscale_shift_; }
- // QP is obtained from VP8-bitstream for HW, so the QP corresponds to the
- // bitstream range of [0, 127] and not the user-level range of [0,63].
- static const int kLowVp8QpThreshold;
- static const int kBadVp8QpThreshold;
-
- // H264 QP is in the range [0, 51].
- static const int kLowH264QpThreshold;
- static const int kBadH264QpThreshold;
-
private:
void ClearSamples();
void ScaleUp();
« no previous file with comments | « webrtc/modules/video_coding/codecs/vp8/vp8_impl.cc ('k') | webrtc/modules/video_coding/utility/quality_scaler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698