Implement $Secure- cookie prefix

net/url_request/url_request_http_job.cc

Index: net/url_request/url_request_http_job.cc
diff --git a/net/url_request/url_request_http_job.cc b/net/url_request/url_request_http_job.cc
index 3cace9060cfbdaa6b7cb699596eb4df8914b47f7..263f5020ffce5ad22d89b83466efac8e736ade00 100644
--- a/net/url_request/url_request_http_job.cc
+++ b/net/url_request/url_request_http_job.cc
@@ -749,6 +749,8 @@ void URLRequestHttpJob::SaveNextCookie() {
     CookieOptions options;
     options.set_include_httponly();
     options.set_server_time(response_date_);
+    if (network_delegate()->AreExperimentalCookieFeaturesEnabled())

[droger, 2015/12/01 13:10:23]

Is it supported to have a null delegate?
I see that a lot of places test for the existence of the delegate before calling
it, but this is not done here.
See line 663 above for example.

There is some code (ios/crnet/crnet_environment) that sets up the network stack
without network delegate, and it now crashes here. Should I use a dummy delegate
there, or should we fix this call to support the null delegate?

[mmenke, 2015/12/01 15:55:53]

Hrm...we generally allow a NULL delegate.  The fact that apparently not a lot of
tests use HTTP requests with NULL delegates, so this wasn't caught indicates
that perhaps we should rethink supporting the NULL delegate case.

For now, though, this code should be fixed.

+      options.set_enforce_prefixes();
 
     CookieStore::SetCookiesCallback callback(base::Bind(
         &URLRequestHttpJob::OnCookieSaved, weak_factory_.GetWeakPtr(),