Added URLRequestContext to constructor for URLRequest.

remoting/host/url_fetcher.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 "remoting/host/url_fetcher.h"
 
 #include "base/location.h"
 #include "base/message_loop_proxy.h"
 #include "googleurl/src/gurl.h"
 #include "net/base/io_buffer.h"
@@ skipping 125 matching lines @@
     // We are done. Post a task to call |done_callback_|.
     delegate_message_loop_->PostTask(
         FROM_HERE, base::Bind(&UrlFetcher::Core::CallCallback, this, status,
                               request_->GetResponseCode()));
   }
 }
 
 void UrlFetcher::Core::DoStart() {
   DCHECK(network_task_runner_->BelongsToCurrentThread());
 
-  request_.reset(new net::URLRequest(url_, this));
-  request_->set_context(request_context_getter_->GetURLRequestContext());
+  request_.reset(new net::URLRequest(
+      url_,
+      this,
+      request_context_getter_->GetURLRequestContext()));

[Sergey Ulanov, 2012/06/20 00:51:51]

all arguments can fit on one line. This should be formatted as follows:
  request_.reset(new net::URLRequest(
      url_, this, request_context_getter_->GetURLRequestContext()));

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

Done.

 
   switch (method_) {
     case GET:
       break;
 
     case POST:
       DCHECK(!upload_content_.empty());
       DCHECK(!upload_content_type_.empty());
       request_->set_method("POST");
 
@@ skipping 53 matching lines @@
 
 void UrlFetcher::SetHeader(const std::string& key, const std::string& value) {
   core_->SetHeader(key, value);
 }
 
 void UrlFetcher::Start(const DoneCallback& done_callback) {
   core_->Start(done_callback);
 }
 
 }  // namespace remoting