Simplify ScriptExecutionContext::vm().
Created attachment 411992 [details] Patch
Committed r268801: <https://trac.webkit.org/changeset/268801> All reviewed patches have been landed. Closing bug and clearing flags on attachment 411992 [details].
<rdar://problem/70532783>
Comment on attachment 411992 [details] Patch I feel like this would be even better as a pure-virtual function that the subclasses would implement. That would avoid one virtual function call in the worker/worklet case as well.
(In reply to Sam Weinig from comment #4) > Comment on attachment 411992 [details] > Patch > > I feel like this would be even better as a pure-virtual function that the > subclasses would implement. That would avoid one virtual function call in > the worker/worklet case as well. Good idea. I will look into this shortly.
(In reply to Chris Dumez from comment #5) > (In reply to Sam Weinig from comment #4) > > Comment on attachment 411992 [details] > > Patch > > > > I feel like this would be even better as a pure-virtual function that the > > subclasses would implement. That would avoid one virtual function call in > > the worker/worklet case as well. > > Good idea. I will look into this shortly. 👍
(In reply to Sam Weinig from comment #6) > (In reply to Chris Dumez from comment #5) > > (In reply to Sam Weinig from comment #4) > > > Comment on attachment 411992 [details] > > > Patch > > > > > > I feel like this would be even better as a pure-virtual function that the > > > subclasses would implement. That would avoid one virtual function call in > > > the worker/worklet case as well. > > > > Good idea. I will look into this shortly. > > 👍 https://bugs.webkit.org/show_bug.cgi?id=218057