Remove URLRequest::GetResponseCookies and URLRequestJob::GetResponseCookies.

net/url_request/url_fetcher_impl_unittest.cc

Index: net/url_request/url_fetcher_impl_unittest.cc
diff --git a/net/url_request/url_fetcher_impl_unittest.cc b/net/url_request/url_fetcher_impl_unittest.cc
index efb17a4a89beb64d74f3731f7cc46ee690dafa78..853ea7169c33ca50d8b3eba5163c82403f16ab03 100644
--- a/net/url_request/url_fetcher_impl_unittest.cc
+++ b/net/url_request/url_fetcher_impl_unittest.cc
@@ -1521,7 +1521,7 @@ TEST_F(URLFetcherBadHTTPSTest, BadHTTPS) {
             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());