There's no reason to print the exact dataURL, especially when the test isn't actually testing the content (e.g. the recording tests care more about the fact that content was sent, not necessarily what that content was).
<rdar://problem/48248697>
Created attachment 363165 [details] Patch
Comment on attachment 363165 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=363165&action=review r=me, nice change. > LayoutTests/inspector/canvas/requestContent-2d.html:28 > + .then(({content}) => InspectorTest.expectGreaterThan(content.length, "data:image/png;base64,".length, "The content should not be empty.")) I'd drop the article from all of these: "Content should not be empty." > LayoutTests/inspector/canvas/setShaderProgramHighlighted-expected.txt:12 > +PASS: Unhighlighting a shader program should draw differently. Can this just be "Unhighlighting a shader program should change the content", or does the distinction matter?
Created attachment 363232 [details] Patch
Comment on attachment 363232 [details] Patch Clearing flags on attachment: 363232 Committed r242225: <https://trac.webkit.org/changeset/242225>
All reviewed patches have been landed. Closing bug.