Remove the restriction that ChromeURLRequestContextGetter needs to be release...

chrome/browser/sync/glue/http_bridge.cc

 // Copyright (c) 2006-2008 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 "chrome/browser/sync/glue/http_bridge.h"
 
 #include "base/message_loop.h"
 #include "base/string_util.h"
 #include "chrome/browser/chrome_thread.h"
 #include "chrome/browser/profile.h"
@@ skipping 28 matching lines @@
     context_->set_user_agent(user_agent_);
 
   return context_;
 }
 
 HttpBridgeFactory::HttpBridgeFactory(
     URLRequestContextGetter* baseline_context_getter) {
   DCHECK(baseline_context_getter != NULL);
   request_context_getter_ =
       new HttpBridge::RequestContextGetter(baseline_context_getter);
-  request_context_getter_->AddRef();
 }
 
 HttpBridgeFactory::~HttpBridgeFactory() {
-  if (request_context_getter_) {
-    // Clean up request context getter on IO thread.
-    bool posted = ChromeThread::ReleaseSoon(
-        ChromeThread::IO, FROM_HERE, request_context_getter_);
-    DCHECK(posted);
-    request_context_getter_ = NULL;
-  }
 }
 
 sync_api::HttpPostProviderInterface* HttpBridgeFactory::Create() {
   HttpBridge* http = new HttpBridge(request_context_getter_);
   http->AddRef();
   return http;
 }
 
 void HttpBridgeFactory::Destroy(sync_api::HttpPostProviderInterface* http) {
   static_cast<HttpBridge*>(http)->Release();
@@ skipping 39 matching lines @@
 }
 
 HttpBridge::HttpBridge(HttpBridge::RequestContextGetter* context_getter)
     : context_getter_for_request_(context_getter),
       url_poster_(NULL),
       created_on_loop_(MessageLoop::current()),
       request_completed_(false),
       request_succeeded_(false),
       http_response_code_(-1),
       http_post_completed_(false, false) {
-  context_getter_for_request_->AddRef();
 }
 
 HttpBridge::~HttpBridge() {
-  bool posted = ChromeThread::ReleaseSoon(
-      ChromeThread::IO, FROM_HERE, context_getter_for_request_);
-  DCHECK(posted);
 }
 
 void HttpBridge::SetUserAgent(const char* user_agent) {
   DCHECK_EQ(MessageLoop::current(), created_on_loop_);
   DCHECK(!request_completed_);
   context_getter_for_request_->set_user_agent(user_agent);
 }
 
 void HttpBridge::SetExtraRequestHeaders(const char * headers) {
   DCHECK(extra_headers_.empty())
@@ skipping 93 matching lines @@
   // URLFetcher, so it seems most natural / "polite" to let the stack unwind.
   MessageLoop::current()->DeleteSoon(FROM_HERE, url_poster_);
   url_poster_ = NULL;
 
   // Wake the blocked syncer thread in MakeSynchronousPost.
   // WARNING: DONT DO ANYTHING AFTER THIS CALL! |this| may be deleted!
   http_post_completed_.Signal();
 }
 
 }  // namespace browser_sync