Remove URLRequest::GetResponseCookies and URLRequestJob::GetResponseCookies.

net/url_request/url_fetcher_impl_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 "net/url_request/url_fetcher_impl.h"
 
 #include <stdint.h>
 #include <string.h>
 
 #include <algorithm>
@@ skipping 1503 matching lines @@
 TEST_F(URLFetcherBadHTTPSTest, BadHTTPS) {
   WaitingURLFetcherDelegate delegate;
   delegate.CreateFetcher(test_server_->GetURL(kDefaultResponsePath),
                          URLFetcher::GET, CreateSameThreadContextGetter());
   delegate.StartFetcherAndWait();
 
   EXPECT_EQ(URLRequestStatus::CANCELED,
             delegate.fetcher()->GetStatus().status());
   EXPECT_EQ(ERR_ABORTED, delegate.fetcher()->GetStatus().error());
   EXPECT_EQ(-1, delegate.fetcher()->GetResponseCode());
-  EXPECT_TRUE(delegate.fetcher()->GetCookies().empty());
+  EXPECT_FALSE(delegate.fetcher()->GetResponseHeaders());

[martijnc, 2016/06/07 21:17:12]

Because the request is expected to fail it has no response headers so there
can't be response cookies either. This change makes the test more strict though.
Would it be better to just remove this line?

   std::string data;
   EXPECT_TRUE(delegate.fetcher()->GetResponseAsString(&data));
   EXPECT_TRUE(data.empty());
 }
 
 }  // namespace
 
 }  // namespace net