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

Unified Diff: webrtc/modules/video_coding/main/source/encoded_frame.cc

Issue 1371043003: Unify FrameType and VideoFrameType. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Created 5 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/main/source/encoded_frame.cc
diff --git a/webrtc/modules/video_coding/main/source/encoded_frame.cc b/webrtc/modules/video_coding/main/source/encoded_frame.cc
index 0fa44250d7e92055cb864f5bd3268a409a3bcd3a..c0b3fafb03c9c0314f67ad7fd92c00425d4f4e0d 100644
--- a/webrtc/modules/video_coding/main/source/encoded_frame.cc
+++ b/webrtc/modules/video_coding/main/source/encoded_frame.cc
@@ -226,40 +226,4 @@ void VCMEncodedFrame::VerifyAndAllocate(size_t minimumSize)
}
}
-webrtc::FrameType VCMEncodedFrame::ConvertFrameType(VideoFrameType frameType)
-{
- switch(frameType) {
- case kKeyFrame:
- return kVideoFrameKey;
- case kDeltaFrame:
- return kVideoFrameDelta;
- case kSkipFrame:
- return kFrameEmpty;
- default:
- return kVideoFrameDelta;
- }
-}
-
-VideoFrameType VCMEncodedFrame::ConvertFrameType(webrtc::FrameType frame_type) {
- switch (frame_type) {
- case kVideoFrameKey:
- return kKeyFrame;
- case kVideoFrameDelta:
- return kDeltaFrame;
- default:
- assert(false);
- return kDeltaFrame;
- }
-}
-
-void VCMEncodedFrame::ConvertFrameTypes(
- const std::vector<webrtc::FrameType>& frame_types,
- std::vector<VideoFrameType>* video_frame_types) {
- assert(video_frame_types);
- video_frame_types->reserve(frame_types.size());
- for (size_t i = 0; i < frame_types.size(); ++i) {
- (*video_frame_types)[i] = ConvertFrameType(frame_types[i]);
- }
-}
-
}

Powered by Google App Engine
This is Rietveld 408576698