Rename NetworkDelegate first-party-only flag

net/url_request/url_request_http_job.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_request_http_job.h"
 
 #include "base/base_switches.h"
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/command_line.h"
@@ skipping 643 matching lines @@
   }
 }
 
 void URLRequestHttpJob::DoLoadCookies() {
   CookieOptions options;
   options.set_include_httponly();
 
   // TODO(mkwst): Drop this `if` once we decide whether or not to ship
   // first-party cookies: https://crbug.com/459154
   if (network_delegate() &&
-      network_delegate()->FirstPartyOnlyCookieExperimentEnabled())
+      network_delegate()->ExperimentalCookieFeaturesEnabled())

[mmenke, 2015/10/12 14:25:42]

Should be calling the On* method here (Old bug...)

[estark, 2015/10/12 14:48:08]

How so? The On* version is private.

     options.set_first_party_url(request_->first_party_for_cookies());
   else
     options.set_include_first_party_only();
 
   request_->context()->cookie_store()->GetCookiesWithOptionsAsync(
       request_->url(), options, base::Bind(&URLRequestHttpJob::OnCookiesLoaded,
                                            weak_factory_.GetWeakPtr()));
 }
 
 void URLRequestHttpJob::CheckCookiePolicyAndLoad(
@@ skipping 900 matching lines @@
   return override_response_headers_.get() ?
              override_response_headers_.get() :
              transaction_->GetResponseInfo()->headers.get();
 }
 
 void URLRequestHttpJob::NotifyURLRequestDestroyed() {
   awaiting_callback_ = false;
 }
 
 }  // namespace net