Teach URLRequest about initiator checks for First-Party-Only cookies.

net/url_request/url_request.h

 // 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.
 
 #ifndef NET_URL_REQUEST_URL_REQUEST_H_
 #define NET_URL_REQUEST_URL_REQUEST_H_
 
 #include <stdint.h>
 
 #include <string>
@@ skipping 24 matching lines @@
 #include "url/gurl.h"
 
 namespace base {
 class Value;
 
 namespace debug {
 class StackTrace;
 }  // namespace debug
 }  // namespace base
 
+namespace url {
+class Origin;
+}
+
 namespace net {
 
 class ChunkedUploadDataStream;
 class CookieOptions;
 class HostPortPair;
 class IOBuffer;
 struct LoadTimingInfo;
 struct RedirectInfo;
 class SSLCertRequestInfo;
 class SSLInfo;
@@ skipping 220 matching lines @@
   void set_first_party_for_cookies(const GURL& first_party_for_cookies);
 
   // The first-party URL policy to apply when updating the first party URL
   // during redirects. The first-party URL policy may only be changed before
   // Start() is called.
   FirstPartyURLPolicy first_party_url_policy() const {
     return first_party_url_policy_;
   }
   void set_first_party_url_policy(FirstPartyURLPolicy first_party_url_policy);
 
+  // The origin of the context which initiated the request.
+  const url::Origin initiator() const { return initiator_; }

[mmenke, 2015/10/21 15:36:38]

const url::Origin&

[mmenke, 2015/10/21 15:36:38]

I'm concerned that I have no idea what the difference between initiator and
first_party_for_cookies is, and the code here has no pointers to definitions for
either.  Can we add some pointers to RFCs?

[Mike West, 2015/10/22 13:17:02]

Done and done.

+  // This method may only be called before Start().
+  void set_initiator(const url::Origin& initiator);
+
   // The request method, as an uppercase string.  "GET" is the default value.
   // The request method may only be changed before Start() is called and
   // should only be assigned an uppercase value.
   const std::string& method() const { return method_; }
   void set_method(const std::string& method);
 
+  // True if the request method is "safe" (per section 4.2.1 of RFC 7231).
+  bool is_safe_method() const;
+
   // The referrer URL for the request.  This header may actually be suppressed
   // from the underlying network request for security reasons (e.g., a HTTPS
   // URL will not be sent as the referrer for a HTTP request).  The referrer
   // may only be changed before Start() is called.
   const std::string& referrer() const { return referrer_; }
   // Referrer is sanitized to remove URL fragment, user name and password.
   void SetReferrer(const std::string& referrer);
 
   // The referrer policy to apply when updating the referrer during redirects.
   // The referrer policy may only be changed before Start() is called.
@@ skipping 451 matching lines @@
   BoundNetLog net_log_;
 
   scoped_refptr<URLRequestJob> job_;
   scoped_ptr<UploadDataStream> upload_data_stream_;
   // TODO(mmenke):  Make whether or not an upload is chunked transparent to the
   // URLRequest.
   ChunkedUploadDataStream* upload_chunked_data_stream_;
 
   std::vector<GURL> url_chain_;
   GURL first_party_for_cookies_;
+  url::Origin initiator_;
   GURL delegate_redirect_url_;
   std::string method_;  // "GET", "POST", etc. Should be all uppercase.
   std::string referrer_;
   ReferrerPolicy referrer_policy_;
   FirstPartyURLPolicy first_party_url_policy_;
   HttpRequestHeaders extra_request_headers_;
   int load_flags_;  // Flags indicating the request type for the load;
                     // expected values are LOAD_* enums above.
 
   // Never access methods of the |delegate_| directly. Always use the
@@ skipping 82 matching lines @@
   HostPortPair proxy_server_;
 
   scoped_ptr<const base::debug::StackTrace> stack_trace_;
 
   DISALLOW_COPY_AND_ASSIGN(URLRequest);
 };
 
 }  // namespace net
 
 #endif  // NET_URL_REQUEST_URL_REQUEST_H_