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

Unified Diff: webrtc/config.h

Issue 2060403002: Add task queue to Call. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@move_getpadding
Patch Set: Fix audio thread check when adding audio to bitrateallocator. Created 4 years, 4 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/config.h
diff --git a/webrtc/config.h b/webrtc/config.h
index 1550a9f8a13ac1b121c90b31a771eee56e12f1a6..d932eda1df8eee81a3d004518fcc9396a495d2c2 100644
--- a/webrtc/config.h
+++ b/webrtc/config.h
@@ -125,12 +125,20 @@ struct VideoStream {
};
struct VideoEncoderConfig {
+ public:
enum class ContentType {
kRealtimeVideo,
kScreen,
};
+ VideoEncoderConfig& operator=(VideoEncoderConfig&&) = default;
+ VideoEncoderConfig& operator=(const VideoEncoderConfig&) = delete;
+
+ // Mostly used by tests. Avoid creating copies if you can.
+ VideoEncoderConfig Copy() const { return VideoEncoderConfig(*this); }
+
VideoEncoderConfig();
+ VideoEncoderConfig(VideoEncoderConfig&&) = default;
~VideoEncoderConfig();
std::string ToString() const;
@@ -145,6 +153,11 @@ struct VideoEncoderConfig {
// unless the estimated bandwidth indicates that the link can handle it.
int min_transmit_bitrate_bps;
bool expect_encode_from_texture;
+
+ private:
+ // Access to the copy constructor is private to force use of the Copy()
+ // method for those exceptional cases where we do use it.
+ VideoEncoderConfig(const VideoEncoderConfig&) = default;
};
struct VideoDecoderH264Settings {

Powered by Google App Engine
This is Rietveld 408576698