From 84bf1217eb9313b06cc9deb715f5d4774de8f865 Mon Sep 17 00:00:00 2001 From: "ajwong@chromium.org" Date: Fri, 23 Dec 2011 03:41:23 +0000 Subject: Fix review comments from wtc BUG=none TEST=none Review URL: http://codereview.chromium.org/9023021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115683 0039d316-1c4b-4281-b951-d872f2087c98 --- net/http/disk_cache_based_ssl_host_info.cc | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'net/http') diff --git a/net/http/disk_cache_based_ssl_host_info.cc b/net/http/disk_cache_based_ssl_host_info.cc index e99c9b0..eff8522 100644 --- a/net/http/disk_cache_based_ssl_host_info.cc +++ b/net/http/disk_cache_based_ssl_host_info.cc @@ -31,7 +31,7 @@ namespace net { // This is a hack. A better fix is to make it so that the disk_cache APIs // take a Callback to a mutator for setting the output value rather than // writing into a raw handle. Then the caller can just pass in a Callback -// bound to WeakPtr for itself. This callback would correct "no-op" itself +// bound to WeakPtr for itself. This callback would correctly "no-op" itself // when the DiskCacheBasedSSLHostInfo object is deleted. // // TODO(ajwong): Change disk_cache's API to return results via Callback. @@ -111,8 +111,8 @@ std::string DiskCacheBasedSSLHostInfo::key() const { return "sslhostinfo:" + hostname_; } -void DiskCacheBasedSSLHostInfo::OnIOComplete( - CacheOperationDataShim* unused, int rv) { +void DiskCacheBasedSSLHostInfo::OnIOComplete(CacheOperationDataShim* unused, + int rv) { rv = DoLoop(rv); if (rv != ERR_IO_PENDING && !user_callback_.is_null()) { CompletionCallback callback = user_callback_; @@ -171,7 +171,7 @@ int DiskCacheBasedSSLHostInfo::DoLoop(int rv) { int DiskCacheBasedSSLHostInfo::DoGetBackendComplete(int rv) { if (rv == OK) { - backend_ = data_shim_->backend, + backend_ = data_shim_->backend; state_ = OPEN; } else { state_ = WAIT_FOR_DATA_READY_DONE; -- cgit v1.1