Make the request initiator Optional

url/origin.h

 // Copyright 2015 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 URL_ORIGIN_H_
 #define URL_ORIGIN_H_
 
 #include <stdint.h>
 
 #include <string>
 
+#include "base/memory/ref_counted.h"
 #include "base/strings/string16.h"
 #include "base/strings/string_piece.h"
 #include "url/scheme_host_port.h"
 #include "url/third_party/mozilla/url_parse.h"
 #include "url/url_canon.h"
 #include "url/url_constants.h"
 #include "url/url_export.h"
 
 class GURL;
 
@@ skipping 73 matching lines @@
   //
   // This constructor should be used in order to pass 'Origin' objects back and
   // forth over IPC (as transitioning through GURL would risk potentially
   // dangerous recanonicalization); other potential callers should prefer the
   // 'GURL'-based constructor.
   static Origin UnsafelyCreateOriginWithoutNormalization(
       base::StringPiece scheme,
       base::StringPiece host,
       uint16_t port);
 
-  ~Origin();
+  virtual ~Origin();
 
   // For unique origins, these return ("", "", 0).
   const std::string& scheme() const { return tuple_.scheme(); }
   const std::string& host() const { return tuple_.host(); }
   uint16_t port() const { return tuple_.port(); }
 
   bool unique() const { return unique_; }
 
   // An ASCII serialization of the Origin as per Section 6.2 of RFC 6454, with
   // the addition that all Origins with a 'file' scheme serialize to "file://".
@@ skipping 21 matching lines @@
   // std::map).
   bool operator<(const Origin& other) const;
 
  private:
   Origin(base::StringPiece scheme, base::StringPiece host, uint16_t port);
 
   SchemeHostPort tuple_;
   bool unique_;
 };
 
+// A version of Origin that supports ref count and can be passed in a
+// scoped_refptr. This allows to specify a null Origin (as opposed to a unique
+// origin).
+class URL_EXPORT NullableOrigin : public Origin,

[Mike West, 2016/10/11 08:50:31]

This feels like a stopgap that we're doing just because it's a pain in the butt
to make `Origin` itself refcounted. I'm fine with that as a temporary measure
(and because you shouldn't have to do work I said I was going to do and then
didn't...), but I'd at least like a `TODO(mkwst)` here to remove
`NullableOrigin` as future cleanup.

+                                  public base::RefCounted<NullableOrigin> {
+ public:
+  explicit NullableOrigin(const GURL& url);
+  NullableOrigin(const Origin& origin);
+
+ private:
+  friend base::RefCounted<NullableOrigin>;
+  ~NullableOrigin() override;
+};
+
 URL_EXPORT std::ostream& operator<<(std::ostream& out, const Origin& origin);
 
 URL_EXPORT bool IsSameOriginWith(const GURL& a, const GURL& b);
 
 }  // namespace url
 
 #endif  // URL_ORIGIN_H_