Changes to have chrome building with net.dll

webkit/blob/blob_url_request_job_unittest.cc

Index: webkit/blob/blob_url_request_job_unittest.cc
===================================================================
--- webkit/blob/blob_url_request_job_unittest.cc	(revision 87744)
+++ webkit/blob/blob_url_request_job_unittest.cc	(working copy)
@@ -69,6 +69,8 @@
 
    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()) {

[wtc, 2011/06/03 18:24:40]

I believe we can replace this with
    if (!request->is_pending()) {

See the following code in url_request_job.cc:

void URLRequestJob::NotifyDone(const URLRequestStatus &status) {
  DCHECK(!done_) << "Job sending done notification twice";
  if (done_)
    return;
  done_ = true;
  ^^^^^^^^^^^^^

  // Unless there was an error, we should have at least tried to handle
  // the response before getting here.
  DCHECK(has_handled_response_ || !status.is_success());

  // As with NotifyReadComplete, we need to take care to notice if we were
  // destroyed during a delegate callback.
  if (request_) {
    request_->set_is_pending(false);
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    ...

[rvargas (doing something else), 2011/06/03 18:47:42]

I added your comment to the bug report.

         RequestComplete();
         return;