Fix bug 84783: BlobURLRequestJobTest uses URLRequest::job()

webkit/blob/blob_url_request_job_unittest.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 <stack>
 #include <utility>
 
 #include "base/file_path.h"
 #include "base/file_util.h"
 #include "base/memory/ref_counted.h"
@@ skipping 51 matching lines @@
        if (bytes_read > 0)
          ReceiveData(request, bytes_read);
        else
          RequestComplete();
     }
 
     const std::string& response_data() const { return response_data_; }
 
    private:
     void ReadSome(net::URLRequest* request) {
-      // job() is not part of the URLRequest public API so it should not be
-      // used here. Bug 84783.
-      if (request->job()->is_done()) {
+      if (!request->is_pending()) {

[wtc, 2011/06/08 22:16:20]

I wonder if we should test !request->status().is_io_pending()
instead.  See lines 79-81 below.

         RequestComplete();
         return;
       }
 
       int bytes_read = 0;
       if (!request->Read(received_data_.get(), kBufferSize, &bytes_read)) {
         if (!request->status().is_io_pending()) {
           RequestComplete();
         }
         return;
@@ skipping 378 matching lines @@
 
 TEST_F(BlobURLRequestJobTest, TestExtraHeaders) {
   RunTestOnIOThread(&BlobURLRequestJobTest::TestExtraHeaders);
 }
 
 }  // namespace webkit_blob
 
 // BlobURLRequestJobTest is expected to always live longer than the
 // runnable methods.  This lets us call NewRunnableMethod on its instances.
 DISABLE_RUNNABLE_METHOD_REFCOUNT(webkit_blob::BlobURLRequestJobTest);