Added URLRequestContext to constructor for URLRequest.

webkit/fileapi/file_system_dir_url_request_job_unittest.cc

 // Copyright (c) 2012 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/fileapi/file_system_dir_url_request_job.h"
 
 #include <string>
 
 #include "base/file_path.h"
 #include "base/format_macros.h"
@@ skipping 67 matching lines @@
     ClearUnusedJob();
   }
 
   void OnValidateFileSystem(base::PlatformFileError result) {
     ASSERT_EQ(base::PLATFORM_FILE_OK, result);
   }
 
   void TestRequestHelper(const GURL& url, bool run_to_completion) {
     delegate_.reset(new TestDelegate());
     delegate_->set_quit_on_redirect(true);
-    request_.reset(new net::URLRequest(url, delegate_.get()));
+    request_.reset(new net::URLRequest(url,
+                                       delegate_.get(),
+                                       &empty_context_));
     job_ = new FileSystemDirURLRequestJob(request_.get(),
                                           file_system_context_.get());
 
     request_->Start();
     ASSERT_TRUE(request_->is_pending());  // verify that we're starting async
     if (run_to_completion)
       MessageLoop::current()->Run();
   }
 
   void TestRequest(const GURL& url) {
@@ skipping 105 matching lines @@
     return file_system_context_->sandbox_provider()->GetFileUtil();
   }
 
   // Put the message loop at the top, so that it's the last thing deleted.
   MessageLoop message_loop_;
   // Delete all MessageLoopProxy objects before the MessageLoop, to help prevent
   // leaks caused by tasks posted during shutdown.
   scoped_refptr<base::MessageLoopProxy> file_thread_proxy_;
 
   ScopedTempDir temp_dir_;
+  net::URLRequestContext empty_context_;
   scoped_ptr<net::URLRequest> request_;

[erikwright (departed), 2012/06/19 21:06:22]

Is it bad that delegate_ is below request_?

[shalev, 2012/06/20 20:13:12]

Done.

   scoped_ptr<TestDelegate> delegate_;
   scoped_refptr<quota::MockSpecialStoragePolicy> special_storage_policy_;
   scoped_refptr<FileSystemContext> file_system_context_;
   base::WeakPtrFactory<FileSystemDirURLRequestJobTest> weak_factory_;
 
   static net::URLRequestJob* job_;
 };
 
 // static
 net::URLRequestJob* FileSystemDirURLRequestJobTest::job_ = NULL;
@@ skipping 58 matching lines @@
   CreateDirectory("foo");
   TestRequestNoRun(CreateFileSystemURL("foo/"));
   // Run StartAsync() and only StartAsync().
   MessageLoop::current()->DeleteSoon(FROM_HERE, request_.release());
   MessageLoop::current()->RunAllPending();
   // If we get here, success! we didn't crash!
 }
 
 }  // namespace (anonymous)
 }  // namespace fileapi