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

Unified Diff: webkit/appcache/appcache_url_request_job_unittest.cc

Issue 11275088: Remove implicit scoped_refptr operator T* Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 8 years, 2 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
« no previous file with comments | « webkit/appcache/appcache_url_request_job.cc ('k') | webkit/appcache/mock_appcache_storage_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webkit/appcache/appcache_url_request_job_unittest.cc
diff --git a/webkit/appcache/appcache_url_request_job_unittest.cc b/webkit/appcache/appcache_url_request_job_unittest.cc
index 400998b2e8e94502219ef2e8287897fb610376b4..614fedfa50bc6708da1188d5c83fa326483ff70d 100644
--- a/webkit/appcache/appcache_url_request_job_unittest.cc
+++ b/webkit/appcache/appcache_url_request_job_unittest.cc
@@ -110,7 +110,8 @@ class AppCacheURLRequestJobTest : public testing::Test {
scoped_refptr<IOBuffer> wrapped_buffer(
new net::WrappedIOBuffer(received_data_->data() + amount_received_));
int bytes_read = 0;
- EXPECT_FALSE(request->Read(wrapped_buffer, kBlockSize, &bytes_read));
+ EXPECT_FALSE(
+ request->Read(wrapped_buffer.get(), kBlockSize, &bytes_read));
EXPECT_EQ(0, bytes_read);
}
@@ -250,7 +251,7 @@ class AppCacheURLRequestJobTest : public testing::Test {
void WriteBasicResponse() {
scoped_refptr<IOBuffer> body(new WrappedIOBuffer(kHttpBasicBody));
std::string raw_headers(kHttpBasicHeaders, arraysize(kHttpBasicHeaders));
- WriteResponse(MakeHttpResponseInfo(raw_headers), body,
+ WriteResponse(MakeHttpResponseInfo(raw_headers), body.get(),
strlen(kHttpBasicBody));
}
@@ -268,7 +269,7 @@ class AppCacheURLRequestJobTest : public testing::Test {
expected_write_result_ = GetHttpResponseInfoSize(head);
write_info_buffer_ = new HttpResponseInfoIOBuffer(head);
writer_->WriteInfo(
- write_info_buffer_,
+ write_info_buffer_.get(),
base::Bind(&AppCacheURLRequestJobTest::OnWriteInfoComplete,
base::Unretained(this)));
}
@@ -278,7 +279,7 @@ class AppCacheURLRequestJobTest : public testing::Test {
write_buffer_ = io_buffer;
expected_write_result_ = buf_len;
writer_->WriteData(
- write_buffer_, buf_len,
+ write_buffer_.get(), buf_len,
base::Bind(&AppCacheURLRequestJobTest::OnWriteComplete,
base::Unretained(this)));
}
@@ -288,7 +289,7 @@ class AppCacheURLRequestJobTest : public testing::Test {
read_buffer_ = io_buffer;
expected_read_result_ = buf_len;
reader_->ReadData(
- read_buffer_, buf_len,
+ read_buffer_.get(), buf_len,
base::Bind(&AppCacheURLRequestJobTest::OnReadComplete,
base::Unretained(this)));
}
@@ -544,7 +545,7 @@ class AppCacheURLRequestJobTest : public testing::Test {
// Start the request.
EXPECT_FALSE(job->has_been_started());
- mock_factory_job_ = job;
+ mock_factory_job_ = job.get();
request_->Start();
EXPECT_FALSE(mock_factory_job_);
EXPECT_TRUE(job->has_been_started());
@@ -603,7 +604,8 @@ class AppCacheURLRequestJobTest : public testing::Test {
for (int i = 0; i < 3; ++i, p += kBlockSize)
FillData(i + 1, p, kBlockSize);
std::string raw_headers(kHttpHeaders, arraysize(kHttpHeaders));
- WriteResponse(MakeHttpResponseInfo(raw_headers), body, kBlockSize * 3);
+ WriteResponse(
+ MakeHttpResponseInfo(raw_headers), body.get(), kBlockSize * 3);
}
void VerifyDeliverLargeAppCachedResponse() {
@@ -657,7 +659,7 @@ class AppCacheURLRequestJobTest : public testing::Test {
// Start the request.
EXPECT_FALSE(job->has_been_started());
- mock_factory_job_ = job;
+ mock_factory_job_ = job.get();
request_->Start();
EXPECT_FALSE(mock_factory_job_);
EXPECT_TRUE(job->has_been_started());
« no previous file with comments | « webkit/appcache/appcache_url_request_job.cc ('k') | webkit/appcache/mock_appcache_storage_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698