Remove URLRequest::GetResponseCookies and URLRequestJob::GetResponseCookies.

chrome/browser/chromeos/policy/user_cloud_policy_manager_chromeos_unittest.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 "chrome/browser/chromeos/policy/user_cloud_policy_manager_chromeos.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/callback.h"
 #include "base/macros.h"
+#include "base/memory/ref_counted.h"
 #include "base/message_loop/message_loop.h"
 #include "base/run_loop.h"
 #include "base/sequenced_task_runner.h"
 #include "base/strings/string_util.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/test/test_simple_task_runner.h"
 #include "chrome/browser/chromeos/login/users/scoped_user_manager_enabler.h"
 #include "chrome/browser/chromeos/policy/user_cloud_policy_token_forwarder.h"
 #include "chrome/browser/chromeos/profiles/profile_helper.h"
 #include "chrome/browser/prefs/browser_prefs.h"
@@ skipping 16 matching lines @@
 #include "components/prefs/testing_pref_service.h"
 #include "components/signin/core/browser/fake_profile_oauth2_token_service.h"
 #include "components/signin/core/browser/profile_oauth2_token_service.h"
 #include "components/signin/core/browser/signin_manager.h"
 #include "components/syncable_prefs/pref_service_syncable.h"
 #include "components/user_manager/fake_user_manager.h"
 #include "content/public/test/test_browser_thread_bundle.h"
 #include "google_apis/gaia/gaia_auth_consumer.h"
 #include "google_apis/gaia/gaia_constants.h"
 #include "google_apis/gaia/gaia_urls.h"
+#include "net/http/http_response_headers.h"
 #include "net/url_request/test_url_fetcher_factory.h"
 #include "net/url_request/url_fetcher_delegate.h"
 #include "net/url_request/url_request_context_getter.h"
 #include "net/url_request/url_request_status.h"
 #include "policy/policy_constants.h"
 #include "policy/proto/device_management_backend.pb.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace em = enterprise_management;
 
 using testing::AnyNumber;
 using testing::AtLeast;
 using testing::Mock;
 using testing::_;
 
 namespace policy {
 
-const char kOAuthCodeCookie[] = "oauth_code=1234; Secure; HttpOnly";
+const char kOAuthCodeCookie[] = "Set-Cookie: oauth_code=1234; Secure; HttpOnly";

[mmenke, 2016/06/08 14:46:44]

Maybe rename this kSetOAuthCodeCookie?

[martijnc, 2016/06/08 16:08:10]

Done.

 
 const char kOAuth2TokenPairData[] =
     "{"
     "  \"refresh_token\": \"1234\","
     "  \"access_token\": \"5678\","
     "  \"expires_in\": 3600"
     "}";
 
 const char kOAuth2AccessTokenData[] =
     "{"
@@ skipping 157 matching lines @@
         .WillOnce(device_management_service_.CreateAsyncJob(&register_request));
 
     if (!has_request_token) {
       GaiaUrls* gaia_urls = GaiaUrls::GetInstance();
       net::TestURLFetcher* fetcher = NULL;
 
       // Issue the oauth_token cookie first.
       fetcher = PrepareOAuthFetcher(gaia_urls->client_login_to_oauth2_url());
       if (!fetcher)
         return NULL;
-      net::ResponseCookies cookies;
-      cookies.push_back(kOAuthCodeCookie);
 
-      fetcher->set_cookies(cookies);
+      scoped_refptr<net::HttpResponseHeaders> reponse_headers =
+          new net::HttpResponseHeaders("");
+      reponse_headers->AddHeader(kOAuthCodeCookie);
+      fetcher->set_response_headers(reponse_headers);
       fetcher->delegate()->OnURLFetchComplete(fetcher);
 
       // Issue the refresh token.
       fetcher = PrepareOAuthFetcher(gaia_urls->oauth2_token_url());
       if (!fetcher)
         return NULL;
       fetcher->SetResponseString(kOAuth2TokenPairData);
       fetcher->delegate()->OnURLFetchComplete(fetcher);
 
       // Issue the access token.
@@ skipping 341 matching lines @@
   EXPECT_TRUE(manager_->IsInitializationComplete(POLICY_DOMAIN_CHROME));
   EXPECT_TRUE(manager_->core()->client()->is_registered());
 
   // The refresh scheduler takes care of the initial fetch for unmanaged users.
   // Running the task runner issues the initial fetch.
   FetchPolicy(
       base::Bind(&base::TestSimpleTaskRunner::RunUntilIdle, task_runner_));
 }
 
 }  // namespace policy