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

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: Rebased Created 4 years, 5 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 b2dfe2d272642fa5b8f73bd4a0e75b4a3c4925d1..cc06fbc14b62b1ae6b87b716d9e0a6bc0b40bc98 100644
--- a/webrtc/config.h
+++ b/webrtc/config.h
@@ -124,12 +124,25 @@ struct VideoStream {
};
struct VideoEncoderConfig {
+ private:
stefan-webrtc 2016/07/08 15:56:41 Put private on the bottom instead
perkj_webrtc 2016/07/11 11:41:07 Done.
+ // 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;
+
+ 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;

Powered by Google App Engine
This is Rietveld 408576698