Make the request initiator Optional

net/url_request/url_request.h

Index: net/url_request/url_request.h
diff --git a/net/url_request/url_request.h b/net/url_request/url_request.h
index 0c413f7e50c095266da3a7fd6239abdad45bcc0a..21258bf93a66b05f077c43d96ab703f3f7130dbc 100644
--- a/net/url_request/url_request.h
+++ b/net/url_request/url_request.h
@@ -14,6 +14,7 @@
 #include "base/debug/leak_tracker.h"
 #include "base/logging.h"
 #include "base/macros.h"
+#include "base/optional.h"
 #include "base/strings/string16.h"
 #include "base/supports_user_data.h"
 #include "base/threading/non_thread_safe.h"
@@ -303,9 +304,9 @@ class NET_EXPORT URLRequest : NON_EXPORTED_BASE(public base::NonThreadSafe),
   //
   // This value is used to perform the cross-origin check specified in Section
   // 4.3 of https://tools.ietf.org/html/draft-west-first-party-cookies.
-  const url::Origin& initiator() const { return initiator_; }
+  const base::Optional<url::Origin>& initiator() const { return initiator_; }
   // This method may only be called before Start().

[mmenke, 2016/11/03 17:58:58]

Is it worth mentioning the difference between a unique and invalid initiator?

[clamy, 2016/11/04 16:03:50]

Done.

-  void set_initiator(const url::Origin& initiator);
+  void set_initiator(const base::Optional<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
@@ -761,7 +762,7 @@ class NET_EXPORT URLRequest : NON_EXPORTED_BASE(public base::NonThreadSafe),
 
   std::vector<GURL> url_chain_;
   GURL first_party_for_cookies_;
-  url::Origin initiator_;
+  base::Optional<url::Origin> initiator_;
   GURL delegate_redirect_url_;
   std::string method_;  // "GET", "POST", etc. Should be all uppercase.
   std::string referrer_;