Transfer SecurityOrigin overrides to Workers.

Source/core/workers/WorkerThreadStartupData.h

 /*
  * 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 29 matching lines @@
 #include "wtf/Noncopyable.h"
 
 namespace blink {
 
 class WorkerClients;
 
 class WorkerThreadStartupData FINAL : public NoBaseWillBeGarbageCollectedFinalized<WorkerThreadStartupData> {
     WTF_MAKE_NONCOPYABLE(WorkerThreadStartupData);
     WTF_MAKE_FAST_ALLOCATED_WILL_BE_REMOVED;
 public:
-    static PassOwnPtrWillBeRawPtr<WorkerThreadStartupData> create(const KURL& scriptURL, const String& userAgent, const String& sourceCode, WorkerThreadStartMode startMode, const String& contentSecurityPolicy, ContentSecurityPolicyHeaderType contentSecurityPolicyType, PassOwnPtrWillBeRawPtr<WorkerClients> workerClients)
+    static PassOwnPtrWillBeRawPtr<WorkerThreadStartupData> create(const KURL& scriptURL, const String& userAgent, const String& sourceCode, WorkerThreadStartMode startMode, const String& contentSecurityPolicy, ContentSecurityPolicyHeaderType contentSecurityPolicyType, const SecurityOrigin* starterOrigin, PassOwnPtrWillBeRawPtr<WorkerClients> workerClients)
     {
-        return adoptPtrWillBeNoop(new WorkerThreadStartupData(scriptURL, userAgent, sourceCode, startMode, contentSecurityPolicy, contentSecurityPolicyType, workerClients));
+        return adoptPtrWillBeNoop(new WorkerThreadStartupData(scriptURL, userAgent, sourceCode, startMode, contentSecurityPolicy, contentSecurityPolicyType, starterOrigin, workerClients));
     }
 
     ~WorkerThreadStartupData();
 
     KURL m_scriptURL;
     String m_userAgent;
     String m_sourceCode;
     WorkerThreadStartMode m_startMode;
     String m_contentSecurityPolicy;
     ContentSecurityPolicyHeaderType m_contentSecurityPolicyType;
+
+    // The SecurityOrigin of the Document creating a Worker may have
+    // been configured with extra policy privileges when it was created
+    // (e.g., enforce path-based file:// origins.)
+    // To have those be transferred to the origin of a new worker

[Mike West, 2014/09/29 07:37:22]

Nit: How about "To ensure that these are transferrer...". It would also be nice
to add a comment about object ownership.

[sof, 2014/09/29 09:42:28]

Oh yes, forgot to mention ownership assumptions.

Added - is the assumption reasonable to all?

+    // global scope, supply the Document's SecurityOrigin as the
+    // 'starter origin'.
+    //
+    // See SecurityOrigin::transferPrivileges() for details
+    // on the privileges that are inherited/transferred.
+    const SecurityOrigin* m_starterOrigin;
     OwnPtrWillBeMember<WorkerClients> m_workerClients;
 
     void trace(Visitor*);
 
 private:
-    WorkerThreadStartupData(const KURL& scriptURL, const String& userAgent, const String& sourceCode, WorkerThreadStartMode, const String& contentSecurityPolicy, ContentSecurityPolicyHeaderType contentSecurityPolicyType, PassOwnPtrWillBeRawPtr<WorkerClients>);
+    WorkerThreadStartupData(const KURL& scriptURL, const String& userAgent, const String& sourceCode, WorkerThreadStartMode, const String& contentSecurityPolicy, ContentSecurityPolicyHeaderType contentSecurityPolicyType, const SecurityOrigin* parentOrigin, PassOwnPtrWillBeRawPtr<WorkerClients>);

[Mike West, 2014/09/29 07:37:22]

s/parentOrigin/starterOrigin/. Or vice-versa, really. I'm not thrilled about
"starter origin". "parent origin" isn't much better though... *shrug* :)

[sof, 2014/09/29 09:42:28]

Oops, I decided not to go with 'parent origin' in the end ('parent' implies more
than 'starter'), but left this in. Removed.

 };
 
 } // namespace blink
 
 #endif // WorkerThreadStartupData_h