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

Unified Diff: content/browser/fileapi/fileapi_message_filter.cc

Issue 16294003: Update content/ to use scoped_refptr<T>::get() rather than implicit "operator T*" (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebased Created 7 years, 7 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: content/browser/fileapi/fileapi_message_filter.cc
diff --git a/content/browser/fileapi/fileapi_message_filter.cc b/content/browser/fileapi/fileapi_message_filter.cc
index 47009c28516a93693bd66f8b1e33e06e35227f66..d9af761e796a2452f8569de248d347c50d86f2b2 100644
--- a/content/browser/fileapi/fileapi_message_filter.cc
+++ b/content/browser/fileapi/fileapi_message_filter.cc
@@ -70,7 +70,7 @@ FileAPIMessageFilter::FileAPIMessageFilter(
request_context_(NULL),
blob_storage_context_(blob_storage_context) {
DCHECK(context_);
- DCHECK(request_context_getter_);
+ DCHECK(request_context_getter_.get());
DCHECK(blob_storage_context);
}
@@ -91,7 +91,7 @@ FileAPIMessageFilter::FileAPIMessageFilter(
void FileAPIMessageFilter::OnChannelConnected(int32 peer_pid) {
BrowserMessageFilter::OnChannelConnected(peer_pid);
- if (request_context_getter_) {
+ if (request_context_getter_.get()) {
DCHECK(!request_context_);
request_context_ = request_context_getter_->GetURLRequestContext();
request_context_getter_ = NULL;
@@ -757,13 +757,13 @@ void FileAPIMessageFilter::DidCreateSnapshot(
// when the filesystem has been granted permissions. This happens with:
// - Drive filesystems
// - Picasa filesystems
- DCHECK(snapshot_file ||
+ DCHECK(snapshot_file.get() ||
fileapi::SandboxMountPointProvider::IsSandboxType(url.type()) ||
url.type() == fileapi::kFileSystemTypeDrive ||
url.type() == fileapi::kFileSystemTypePicasa);
ChildProcessSecurityPolicyImpl::GetInstance()->GrantReadFile(
process_id_, platform_path);
- if (snapshot_file) {
+ if (snapshot_file.get()) {
// This will revoke all permissions for the file when the last ref
// of the file is dropped (assuming it's ok).
snapshot_file->AddFinalReleaseCallback(
@@ -771,7 +771,7 @@ void FileAPIMessageFilter::DidCreateSnapshot(
}
}
- if (snapshot_file) {
+ if (snapshot_file.get()) {
// This ref is held until OnDidReceiveSnapshotFile is called.
in_transit_snapshot_files_[request_id] = snapshot_file;
}
« no previous file with comments | « content/browser/fileapi/chrome_blob_storage_context.cc ('k') | content/browser/geolocation/device_data_provider.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698