Fix memory leaks in tests.

With the new filesystem URL parser, the invalid URLs used in the InvalidURL test
are now recognized as invalid far earlier, so the job wasn't even getting
requested from the factory, and so wasn't getting deleted.

BUG=121571
TEST=FileSystem[Dir]URLRequestJobTest.InvalidURL under valgrind


Review URL: http://codereview.chromium.org/9956121

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@130741 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 8 insertions, 0 deletions

diff --git a/webkit/fileapi/file_system_url_request_job_unittest.cc b/webkit/fileapi/file_system_url_request_job_unittest.cc
index 16986e2..b8151ac 100644
--- a/webkit/fileapi/file_system_url_request_job_unittest.cc
+++ b/webkit/fileapi/file_system_url_request_job_unittest.cc
@@ -98,6 +98,7 @@ class FileSystemURLRequestJobTest : public testing::Test {
 
   virtual void TearDown() {
     net::URLRequest::Deprecated::RegisterProtocolFactory("filesystem", NULL);
+    ClearUnusedJob();
   }
 
   void OnValidateFileSystem(base::PlatformFileError result) {
@@ -194,6 +195,13 @@ class FileSystemURLRequestJobTest : public testing::Test {
     return temp;
   }
 
+  static void ClearUnusedJob() {
+    if (job_) {
+      scoped_refptr<net::URLRequestJob> deleter = job_;
+      job_ = NULL;
+    }
+  }
+
   // Put the message loop at the top, so that it's the last thing deleted.
   MessageLoop message_loop_;