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

Unified Diff: services/media/framework/parts/file_reader.cc

Issue 1686363002: Motown: ffmpeg implementations of framework 'parts' (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: Changed the way AVBuffer allocation/deallocation is done in the ffmpeg audio decoder. Created 4 years, 10 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: services/media/framework/parts/file_reader.cc
diff --git a/services/media/framework/parts/file_reader.cc b/services/media/framework/parts/file_reader.cc
index 6c68d05445e4d4aaedacc04d82bbc9fb9a4a4444..623e1e41140ec194451dbdc6e553cb86e9526177 100644
--- a/services/media/framework/parts/file_reader.cc
+++ b/services/media/framework/parts/file_reader.cc
@@ -40,7 +40,7 @@ Result FileReader::Init(const GURL& gurl) {
return Result::kOk;
}
-size_t FileReader::Read(uint8* buffer, int bytes_to_read) {
+size_t FileReader::Read(uint8_t* buffer, size_t bytes_to_read) {
return fread(buffer, 1, bytes_to_read, file_);
}
@@ -48,7 +48,7 @@ int64_t FileReader::GetPosition() const {
return ftell(file_);
}
-int64_t FileReader::SetPosition(int64 position) {
+int64_t FileReader::SetPosition(int64_t position) {
if (fseek(file_, position, SEEK_SET) < 0) {
return -1;
}

Powered by Google App Engine
This is Rietveld 408576698