PlzNavigate: properly set the initiator of the navigation

content/browser/service_worker/service_worker_fetch_dispatcher.cc

Index: content/browser/service_worker/service_worker_fetch_dispatcher.cc
diff --git a/content/browser/service_worker/service_worker_fetch_dispatcher.cc b/content/browser/service_worker/service_worker_fetch_dispatcher.cc
index c074651cb42c469b58061bf9905ee12c50504dfb..2e2ee7d286a8c54f5905cbcb81cbffd1c24c0155 100644
--- a/content/browser/service_worker/service_worker_fetch_dispatcher.cc
+++ b/content/browser/service_worker/service_worker_fetch_dispatcher.cc
@@ -418,7 +418,9 @@ void ServiceWorkerFetchDispatcher::MaybeStartNavigationPreload(
   ResourceRequest request;
   request.method = original_request->method();
   request.url = original_request->url();
-  request.request_initiator = original_request->initiator();
+  request.request_initiator = original_request->initiator().has_value()

[horo, 2016/12/12 07:07:57]

Please add this TODO comment:
// TODO(horo): Set first_party_for_cookies to support Same-site Cookies.

MaybeStartNavigationPreload() doesn't set first_party_for_cookies, so same-site
cookies are not included in the navigation preload request. I will create a CL
with tests to support them.

[clamy, 2016/12/12 16:52:33]

Done.

+                                  ? original_request->initiator()
+                                  : url::Origin(original_request->url());
   request.referrer = GURL(original_request->referrer());
   request.referrer_policy = original_info->GetReferrerPolicy();
   request.visibility_state = original_info->GetVisibilityState();