Fix gaia logout URLFetcher memory leak

components/signin/core/browser/gaia_cookie_manager_service.cc

 // Copyright 2014 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 "components/signin/core/browser/gaia_cookie_manager_service.h"
 
 #include <queue>
 #include <vector>
 
 #include "base/json/json_reader.h"
@@ skipping 439 matching lines @@
   // be the last call in any other method.
   FOR_EACH_OBSERVER(Observer, observer_list_,
                     OnAddAccountToCookieCompleted(account_id, error));
 }
 
 void GaiaCookieManagerService::StartLogOutUrlFetch() {
   DCHECK(requests_.front().request_type() == GaiaCookieRequestType::LOG_OUT);
   VLOG(1) << "GaiaCookieManagerService::StartLogOutUrlFetch";
   GURL logout_url(GaiaUrls::GetInstance()->service_logout_url().Resolve(
       base::StringPrintf("?source=%s", source_.c_str())));
-  net::URLFetcher* fetcher =
-      net::URLFetcher::Create(logout_url, net::URLFetcher::GET, this);
-  fetcher->SetRequestContext(signin_client_->GetURLRequestContext());
-  fetcher->Start();
+  logout_url_request_.reset(
+      net::URLFetcher::Create(logout_url, net::URLFetcher::GET, this));
+  logout_url_request_->SetRequestContext(
+      signin_client_->GetURLRequestContext());
+  logout_url_request_->Start();
 }
 
 void GaiaCookieManagerService::OnUbertokenSuccess(
     const std::string& uber_token) {
   DCHECK(requests_.front().request_type() ==
       GaiaCookieRequestType::ADD_ACCOUNT);
   VLOG(1) << "GaiaCookieManagerService::OnUbertokenSuccess"
           << " account=" << requests_.front().account_id();
   fetcher_retries_ = 0;
   uber_token_ = uber_token;
@@ skipping 115 matching lines @@
   gaia_auth_fetcher_.reset(
       new GaiaAuthFetcher(this, source_,
                           signin_client_->GetURLRequestContext()));
   gaia_auth_fetcher_->StartListAccounts();
 }
 
 void GaiaCookieManagerService::OnURLFetchComplete(
     const net::URLFetcher* source) {
   DCHECK(requests_.front().request_type() == GaiaCookieRequestType::LOG_OUT);
   VLOG(1) << "GaiaCookieManagerService::OnURLFetchComplete";
+  scoped_ptr<net::URLFetcher> logout_url_request(logout_url_request_.Pass());
 
   if ((!source->GetStatus().is_success() ||
       source->GetResponseCode() != net::HTTP_OK) &&
       ++fetcher_retries_ < kMaxFetcherRetries) {
     fetcher_backoff_.InformOfRequest(false);
     fetcher_timer_.Start(
         FROM_HERE, fetcher_backoff_.GetTimeUntilRelease(), this,
         &GaiaCookieManagerService::StartLogOutUrlFetch);
     return;
   }
@@ skipping 29 matching lines @@
       case GaiaCookieRequestType::LOG_OUT:
         StartLogOutUrlFetch();
         break;
       case GaiaCookieRequestType::LIST_ACCOUNTS:
         uber_token_fetcher_.reset();
         StartFetchingListAccounts();
         break;
     };
   }
 }