Correctly set requestor origin for worker initiated requests.

third_party/WebKit/Source/core/loader/FrameFetchContext.cpp

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 735 matching lines @@
 
     if (request.firstPartyForCookies().isNull()) {
         request.setFirstPartyForCookies(m_document
             ? m_document->firstPartyForCookies()
             : SecurityOrigin::urlWithUniqueSecurityOrigin());
     }
 
     // Subresource requests inherit their requestor origin from |m_document| directly.
     // Top-level and nested frame types are taken care of in 'FrameLoadRequest()'.
     // Auxiliary frame types in 'createWindow()' and 'FrameLoader::load'.
-    if (request.frameType() == WebURLRequest::FrameTypeNone && !request.requestorOrigin()) {
+    //
+    // TODO(mkwst): It would be cleaner to adjust blink::ResourceRequest to
+    // initialize itself with a `nullptr` initiator so that this can be a simple
+    // `isNull()` check. https://crbug.com/625969
+    if (request.frameType() == WebURLRequest::FrameTypeNone && request.requestorOrigin()->isUnique()) {
         request.setRequestorOrigin(m_document->isSandboxed(SandboxOrigin)
             ? SecurityOrigin::create(m_document->url())
             : m_document->getSecurityOrigin());
     }
 }
 
 MHTMLArchive* FrameFetchContext::archive() const
 {
     ASSERT(!isMainFrame());
     // TODO(nasko): How should this work with OOPIF?
@@ skipping 64 matching lines @@
 }
 
 DEFINE_TRACE(FrameFetchContext)
 {
     visitor->trace(m_document);
     visitor->trace(m_documentLoader);
     FetchContext::trace(visitor);
 }
 
 } // namespace blink