Remaining cleanup (base::Bind): Replacing FileUtilProxy calls with new callback

webkit/blob/blob_url_request_job.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "webkit/blob/blob_url_request_job.h"
 
 #include "base/bind.h"
 #include "base/compiler_specific.h"
 #include "base/file_path.h"
 #include "base/file_util.h"
@@ skipping 47 matching lines @@
       item_index_(0),
       total_size_(0),
       current_item_offset_(0),
       remaining_bytes_(0),
       read_buf_offset_(0),
       read_buf_size_(0),
       read_buf_remaining_bytes_(0),
       bytes_to_read_(0),
       error_(false),
       headers_set_(false),
-      byte_range_set_(false),
-      ALLOW_THIS_IN_INITIALIZER_LIST(method_factory_(this)) {
+      byte_range_set_(false) {
   DCHECK(file_thread_proxy_);
 }
 
 BlobURLRequestJob::~BlobURLRequestJob() {
   // FileStream's destructor won't close it for us because we passed in our own
   // file handle.
   CloseStream();
 }
 
 void BlobURLRequestJob::Start() {
   // Continue asynchronously.
   MessageLoop::current()->PostTask(
       FROM_HERE,
-      method_factory_.NewRunnableMethod(&BlobURLRequestJob::DidStart));
+      base::Bind(&BlobURLRequestJob::DidStart,
+                 weak_factory_.GetWeakPtr()));
 }
 
 void BlobURLRequestJob::DidStart() {
   // We only support GET request per the spec.
   if (request()->method() != "GET") {
     NotifyFailure(net::ERR_METHOD_NOT_SUPPORTED);
     return;
   }
 
   // If the blob data is not present, bail out.
@@ skipping 10 matching lines @@
     stream_->Close();
     stream_.reset(NULL);
   }
 }
 
 void BlobURLRequestJob::Kill() {
   CloseStream();
 
   net::URLRequestJob::Kill();
   weak_factory_.InvalidateWeakPtrs();
-  method_factory_.RevokeAll();
 }
 
 void BlobURLRequestJob::ResolveFile(const FilePath& file_path) {
   base::FileUtilProxy::GetFileInfo(
         file_thread_proxy_, file_path,
         base::Bind(&BlobURLRequestJob::DidResolve, weak_factory_.GetWeakPtr()));
 }
 
 void BlobURLRequestJob::DidResolve(base::PlatformFileError rv,
                                    const base::PlatformFileInfo& file_info) {
@@ skipping 435 matching lines @@
         // We don't support multiple range requests in one single URL request,
         // because we need to do multipart encoding here.
         // TODO(jianli): Support multipart byte range requests.
         NotifyFailure(net::ERR_REQUEST_RANGE_NOT_SATISFIABLE);
       }
     }
   }
 }
 
 }  // namespace webkit_blob