Added URLRequestContext to constructor for URLRequest.

chrome_frame/test/test_server_test.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 <windows.h>
 #include <wininet.h>
 
 #include "base/basictypes.h"
 #include "base/bind.h"
 #include "base/path_service.h"
@@ skipping 45 matching lines @@
     return h_;
   }
 
  protected:
   HINTERNET h_;
 };
 
 class TestURLRequest : public net::URLRequest {
  public:
   TestURLRequest(const GURL& url, Delegate* delegate)
-      : net::URLRequest(url, delegate) {
-    set_context(new TestURLRequestContext());
+      : net::URLRequest(url, delegate, new TestURLRequestContext()) {

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

Looks like this previously leaked. Let's fix it to not leak, similar pattern to
the other cases.

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

Done.

   }
 };
 
 class UrlTaskChain {
  public:
   UrlTaskChain(const char* url, UrlTaskChain* next)
       : url_(url), next_(next) {
   }
 
   void Run() {
@@ skipping 94 matching lines @@
     EXPECT_EQ(file.accessed(), 1);
     EXPECT_EQ(redir.accessed(), 1);
 
     EXPECT_TRUE(person_task.response().find("Guthrie") != std::string::npos);
     EXPECT_TRUE(file_task.response().find("function") != std::string::npos);
     EXPECT_TRUE(redir_task.response().find("Destination") != std::string::npos);
   } else {
     ::TerminateThread(worker, ~0);
   }
 }