Make the request initiator Optional

content/child/web_url_loader_impl.cc

 // Copyright 2014 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 "content/child/web_url_loader_impl.h"
 
 #include <openssl/ssl.h>
 #include <stdint.h>
 
 #include <algorithm>
@@ skipping 511 matching lines @@
   // creating the GURLs.
 
   // TODO(horo): Check credentials flag is unset when credentials mode is omit.
   //             Check credentials flag is set when credentials mode is include.
 
   std::unique_ptr<ResourceRequest> resource_request(new ResourceRequest);
 
   resource_request->method = method;
   resource_request->url = url;
   resource_request->first_party_for_cookies = request.firstPartyForCookies();
-  resource_request->request_initiator = request.requestorOrigin();
+  if (!request.requestorOrigin().isNull()) {
+    resource_request->request_initiator =
+        new url::NullableOrigin(request.requestorOrigin());

[nasko, 2016/10/10 22:28:11]

Isn't the goal of a nullable origin to have it always present, but being able to
have a null value? In this case I wouldn't expect to have a conditional
statement around this assignment.

+  }
   resource_request->referrer = referrer_url;
 
   referrer_policy_ = request.referrerPolicy();
   resource_request->referrer_policy = referrer_policy_;
 
   resource_request->headers = GetWebURLRequestHeaders(request);
   resource_request->load_flags = GetLoadFlagsForWebURLRequest(request);
   // origin_pid only needs to be non-zero if the request originates outside
   // the render process, so we can use requestorProcessID even for requests
   // from in-process plugins.
@@ skipping 726 matching lines @@
     response->clearHTTPHeaderField(webStringName);
     while (response_headers->EnumerateHeader(&iterator, name, &value)) {
       response->addHTTPHeaderField(webStringName,
                                    WebString::fromLatin1(value));
     }
   }
   return true;
 }
 
 }  // namespace content