Related bugs: - <https://webkit.org/b/190305> - <https://webkit.org/b/190325>
Created attachment 351888 [details] Patch
Comment on attachment 351888 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=351888&action=review r=me > LayoutTests/inspector/canvas/resources/shaderProgram-utilities.js:78 > + // This can't use `awaitEvent` since the promise resolution happens on the next tick.sss Whats up with the "tick.sss" at the end of this sentence? Just a typo?
Comment on attachment 351888 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=351888&action=review >> LayoutTests/inspector/canvas/resources/shaderProgram-utilities.js:78 >> + // This can't use `awaitEvent` since the promise resolution happens on the next tick.sss > > Whats up with the "tick.sss" at the end of this sentence? Just a typo? That was my attempt at Cmd+S on the bus 😅
Created attachment 351893 [details] Patch
Comment on attachment 351893 [details] Patch Clearing flags on attachment: 351893 Committed r236972: <https://trac.webkit.org/changeset/236972>
All reviewed patches have been landed. Closing bug.
<rdar://problem/45134558>