Decouple module loading initiator from ScriptElement
Created attachment 298460 [details] Patch
Created attachment 298545 [details] Patch
Created attachment 298546 [details] Patch
Comment on attachment 298546 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=298546&action=review This LGTM, but it's worth having somebody else who knows the WebCore bits more than I do give the thumbs up. > Source/JavaScriptCore/runtime/JSScriptInitiator.h:2 > + * Copyright (C) 2016 Yusuke Suzuki <utatane.tea@gmail.com> 2017 > Source/JavaScriptCore/runtime/ScriptInitiator.h:2 > + * Copyright (C) 2016 Yusuke Suzuki <utatane.tea@gmail.com> 2017 > Source/WebCore/ForwardingHeaders/runtime/JSScriptInitiator.h:2 > + * Copyright (C) 2016 Yusuke Suzuki <utatane.tea@gmail.com> 2017 > Source/WebCore/ForwardingHeaders/runtime/ScriptInitiator.h:2 > + * Copyright (C) 2016 Yusuke Suzuki <utatane.tea@gmail.com> 2017
Comment on attachment 298546 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=298546&action=review Thanks :) >> Source/JavaScriptCore/runtime/JSScriptInitiator.h:2 >> + * Copyright (C) 2016 Yusuke Suzuki <utatane.tea@gmail.com> > > 2017 Fixed. >> Source/JavaScriptCore/runtime/ScriptInitiator.h:2 >> + * Copyright (C) 2016 Yusuke Suzuki <utatane.tea@gmail.com> > > 2017 Fixed. >> Source/WebCore/ForwardingHeaders/runtime/JSScriptInitiator.h:2 >> + * Copyright (C) 2016 Yusuke Suzuki <utatane.tea@gmail.com> > > 2017 Fixed. >> Source/WebCore/ForwardingHeaders/runtime/ScriptInitiator.h:2 >> + * Copyright (C) 2016 Yusuke Suzuki <utatane.tea@gmail.com> > > 2017 Fixed.
Created attachment 298564 [details] Patch
Comment on attachment 298546 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=298546&action=review > Source/JavaScriptCore/ChangeLog:17 > + * runtime/JSScriptInitiator.cpp: Copied from Source/WebCore/dom/LoadableScript.cpp. > + (JSC::JSScriptInitiator::destroy): > + * runtime/JSScriptInitiator.h: Added. I'm not sure if "initiator" is a good name for this. How about ScriptLoader or ScriptFetcher? Or perhaps ScriptOrigin?
Comment on attachment 298564 [details] Patch r=me with my previous comments.
Comment on attachment 298546 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=298546&action=review Thank you for your review. >> Source/JavaScriptCore/ChangeLog:17 >> + * runtime/JSScriptInitiator.h: Added. > > I'm not sure if "initiator" is a good name for this. How about ScriptLoader or ScriptFetcher? > Or perhaps ScriptOrigin? I think `ScriptFetcher` is better since ScriptLoader is ambiguous with ModuleLoader thing in JSC.
Created attachment 298567 [details] Patch for landing
Committed r210585: <http://trac.webkit.org/changeset/210585>