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

Unified Diff: chrome/browser/media_galleries/fileapi/mtp_file_stream_reader.cc

Issue 1815363002: Add RetainedRef uses where needed. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 9 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: chrome/browser/media_galleries/fileapi/mtp_file_stream_reader.cc
diff --git a/chrome/browser/media_galleries/fileapi/mtp_file_stream_reader.cc b/chrome/browser/media_galleries/fileapi/mtp_file_stream_reader.cc
index 8b728a999b3c5fb4914990d0f83a9278045cd0bf..eeba117953deb4867fb9b7410db861943c823406 100644
--- a/chrome/browser/media_galleries/fileapi/mtp_file_stream_reader.cc
+++ b/chrome/browser/media_galleries/fileapi/mtp_file_stream_reader.cc
@@ -103,17 +103,12 @@ int MTPFileStreamReader::Read(net::IOBuffer* buf, int buf_len,
header_buf_len = net::kMaxBytesToSniff;
}
- ReadBytes(url_,
- header_buf.get(),
- 0,
- header_buf_len,
- base::Bind(&MTPFileStreamReader::FinishValidateMediaHeader,
- weak_factory_.GetWeakPtr(),
- header_buf,
- make_scoped_refptr(buf),
- buf_len,
- callback),
- callback);
+ ReadBytes(
+ url_, header_buf.get(), 0, header_buf_len,
+ base::Bind(&MTPFileStreamReader::FinishValidateMediaHeader,
+ weak_factory_.GetWeakPtr(), base::RetainedRef(header_buf),
+ base::RetainedRef(buf), buf_len, callback),
+ callback);
return net::ERR_IO_PENDING;
}

Powered by Google App Engine
This is Rietveld 408576698