ScriptFetcher should be notified directly from the module pipeline.
Created attachment 324224 [details] Patch
Comment on attachment 324224 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=324224&action=review > Source/JavaScriptCore/runtime/JSModuleLoader.cpp:312 > + Identifier moduleKey = jsValueToModuleKey(exec, key); could use auto here.
Comment on attachment 324224 [details] Patch Attachment 324224 [details] did not pass mac-ews (mac): Output: http://webkit-queues.webkit.org/results/4919473 New failing tests: http/tests/security/contentSecurityPolicy/1.1/module-scriptnonce-redirect.html js/dom/modules/module-execution-error-should-be-propagated-to-onerror.html http/tests/security/module-no-mime-type.html
Created attachment 324231 [details] Archive of layout-test-results from ews103 for mac-elcapitan The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: ews103 Port: mac-elcapitan Platform: Mac OS X 10.11.6
Comment on attachment 324224 [details] Patch Attachment 324224 [details] did not pass ios-sim-ews (ios-simulator-wk2): Output: http://webkit-queues.webkit.org/results/4919564 New failing tests: http/tests/security/contentSecurityPolicy/1.1/module-scriptnonce-redirect.html js/dom/modules/module-execution-error-should-be-propagated-to-onerror.html http/tests/security/module-no-mime-type.html
Created attachment 324234 [details] Archive of layout-test-results from ews123 for ios-simulator-wk2 The attached test failures were seen while running run-webkit-tests on the ios-sim-ews. Bot: ews123 Port: ios-simulator-wk2 Platform: Mac OS X 10.12.6
Comment on attachment 324224 [details] Patch Attachment 324224 [details] did not pass mac-debug-ews (mac): Output: http://webkit-queues.webkit.org/results/4919579 New failing tests: http/tests/security/contentSecurityPolicy/1.1/module-scriptnonce-redirect.html js/dom/modules/module-execution-error-should-be-propagated-to-onerror.html http/tests/security/module-no-mime-type.html
Created attachment 324235 [details] Archive of layout-test-results from ews112 for mac-elcapitan The attached test failures were seen while running run-webkit-tests on the mac-debug-ews. Bot: ews112 Port: mac-elcapitan Platform: Mac OS X 10.11.6
Comment on attachment 324224 [details] Patch Attachment 324224 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.webkit.org/results/4923102 New failing tests: http/tests/security/contentSecurityPolicy/1.1/module-scriptnonce-redirect.html js/dom/modules/module-execution-error-should-be-propagated-to-onerror.html http/tests/security/module-no-mime-type.html
Created attachment 324291 [details] Archive of layout-test-results from ews107 for mac-elcapitan-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: ews107 Port: mac-elcapitan-wk2 Platform: Mac OS X 10.11.6
Committed r223744: <https://trac.webkit.org/changeset/223744>
Committed r223750: <https://trac.webkit.org/changeset/223750>
Committed r223751: <https://trac.webkit.org/changeset/223751>
These patches have caused several existing and newly added test to fail consistently: http/tests/security/contentSecurityPolicy/1.1/module-scriptnonce-redirect.html http/tests/security/module-no-mime-type.html js/dom/modules/import-from-javascript-url.html History (in same order): https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=http%2Ftests%2Fsecurity%2FcontentSecurityPolicy%2F1.1%2Fmodule-scriptnonce-redirect.html https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=http%2Ftests%2Fsecurity%2Fmodule-no-mime-type.html https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=js%2Fdom%2Fmodules%2Fimport-from-javascript-url.html build: https://build.webkit.org/results/Apple%20High%20Sierra%20Debug%20WK2%20(Tests)/r223757%20(433)/results.html
Re-opened since this is blocked by bug 178594