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

Unified Diff: webrtc/modules/video_coding/utility/ivf_file_writer.cc

Issue 2054373002: FileWrapper[Impl] modifications and actually remove the "Impl" class. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Fix compile error in func_test_manager.cc Created 4 years, 6 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/ivf_file_writer.cc
diff --git a/webrtc/modules/video_coding/utility/ivf_file_writer.cc b/webrtc/modules/video_coding/utility/ivf_file_writer.cc
index 97f1da30e40a1b7fe7b8ad07fd4bd1c684c835b6..a215d4c26c1c62eeed01e58c5ccce3094b1693fa 100644
--- a/webrtc/modules/video_coding/utility/ivf_file_writer.cc
+++ b/webrtc/modules/video_coding/utility/ivf_file_writer.cc
@@ -139,7 +139,7 @@ bool IvfFileWriter::InitFromFirstFrame(const EncodedImage& encoded_image) {
}
bool IvfFileWriter::WriteFrame(const EncodedImage& encoded_image) {
- RTC_DCHECK(file_->Open());
+ RTC_DCHECK(file_->is_open());
if (num_frames_ == 0 && !InitFromFirstFrame(encoded_image))
return false;
@@ -178,7 +178,7 @@ bool IvfFileWriter::WriteFrame(const EncodedImage& encoded_image) {
}
bool IvfFileWriter::Close() {
- if (!file_->Open())
+ if (!file_->is_open())
return false;
if (num_frames_ == 0) {
@@ -190,7 +190,9 @@ bool IvfFileWriter::Close() {
return true;
}
- return WriteHeader() && (file_->CloseFile() == 0);
+ bool ret = WriteHeader();
+ file_->CloseFile();
+ return ret;
}
} // namespace webrtc

Powered by Google App Engine
This is Rietveld 408576698