Move the worker script loading code to the worker process (phase:1/5)

Source/web/SharedWorkerRepositoryClientImpl.cpp

 /*
  * Copyright (C) 2009 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 141 matching lines @@
     return reinterpret_cast<WebSharedWorkerRepositoryClient::DocumentID>(document);
 }
 
 void SharedWorkerRepositoryClientImpl::connect(PassRefPtr<SharedWorker> worker, PassOwnPtr<WebMessagePortChannel> port, const KURL& url, const String& name, ExceptionState& exceptionState)
 {
     ASSERT(m_client);
 
     // No nested workers (for now) - connect() should only be called from document context.
     ASSERT(worker->executionContext()->isDocument());
     Document* document = toDocument(worker->executionContext());
-    OwnPtr<WebSharedWorkerConnector> webWorkerConnector = adoptPtr(m_client->createSharedWorkerConnector(url, name, getId(document)));
+    OwnPtr<WebSharedWorkerConnector> webWorkerConnector = adoptPtr(m_client->createSharedWorkerConnector(url, name, getId(document), worker->executionContext()->contentSecurityPolicy()->deprecatedHeader(), static_cast<blink::WebContentSecurityPolicyType>(worker->executionContext()->contentSecurityPolicy()->deprecatedHeaderType())));

[kinuko, 2013/12/20 05:07:12]

It's a bit suspected that we really need to send this to SharedWorker, though
without this some test fails..

[horo, 2013/12/20 06:30:24]

In the current source code these content security policy and type are used in
SharedWorkerGlobalScope.
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/WebKit...
So I think we should send them, not to change the behaviour in
SharedWorkerGlobalScope now.

[kinuko, 2013/12/20 06:43:21]

Right, I wanted to note that this may be changed and we may start using
SharedWorker's own CSP policy when we have new agreement (see:
http://lists.w3.org/Archives/Public/public-webappsec/2013Nov/0008.html)

     if (!webWorkerConnector) {
         // Existing worker does not match this url, so return an error back to the caller.
         exceptionState.throwDOMException(URLMismatchError, "The location of the SharedWorker named '" + name + "' does not exactly match the provided URL ('" + url.elidedString() + "').");
         return;
     }
 
     // The loader object manages its own lifecycle (and the lifecycles of the two worker objects).
     // It will free itself once loading is completed.
     SharedWorkerScriptLoader* loader = new SharedWorkerScriptLoader(worker, url, name, port, webWorkerConnector.release());
     loader->load();
 }
 
 void SharedWorkerRepositoryClientImpl::documentDetached(Document* document)
 {
     ASSERT(m_client);
     m_client->documentDetached(getId(document));
 }
 
 SharedWorkerRepositoryClientImpl::SharedWorkerRepositoryClientImpl(WebSharedWorkerRepositoryClient* client)
     : m_client(client)
 {
 }
 
 } // namespace blink