Remove runtime setting for enabling/disabling the isSecureContext API. It should always be enabled at this point.
Created attachment 408932 [details] Patch
Created attachment 408934 [details] Patch
Created attachment 408942 [details] Patch
Comment on attachment 408942 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=408942&action=review r=me assuming tests pass > Source/WebCore/page/WindowOrWorkerGlobalScope.idl:64 > + // microtask queuing. Capitalize microtask if we are using sentence style? Also, why does a single function "queue microtask" need a comment saying "microtask queuing"? I’m not sure of the value of these section comments.
(In reply to Darin Adler from comment #4) > Comment on attachment 408942 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=408942&action=review > > r=me assuming tests pass > > > Source/WebCore/page/WindowOrWorkerGlobalScope.idl:64 > > + // microtask queuing. > > Capitalize microtask if we are using sentence style? Also, why does a single > function "queue microtask" need a comment saying "microtask queuing"? I’m > not sure of the value of these section comments. They come from the spec text. I've mostly been keeping them to make diffing against the spec text easier.
Committed r267164: <https://trac.webkit.org/changeset/267164> All reviewed patches have been landed. Closing bug and clearing flags on attachment 408942 [details].
<rdar://problem/69017378>