Instead of calling JSON.stringify on the payload, we can format the output ourselves and try to show the data in its non-deduplicated state (after swizzling).
This is a great idea.
Created attachment 320383 [details] Patch
Comment on attachment 320383 [details] Patch Attachment 320383 [details] did not pass mac-ews (mac): Output: http://webkit-queues.webkit.org/results/4502493 New failing tests: inspector/canvas/recording-2d.html inspector/canvas/recording-webgl-snapshots.html
Created attachment 320386 [details] Archive of layout-test-results from ews100 for mac-elcapitan The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: ews100 Port: mac-elcapitan Platform: Mac OS X 10.11.6
Comment on attachment 320383 [details] Patch Attachment 320383 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.webkit.org/results/4502501 New failing tests: inspector/canvas/recording-webgl-snapshots.html
Created attachment 320388 [details] Archive of layout-test-results from ews106 for mac-elcapitan-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: ews106 Port: mac-elcapitan-wk2 Platform: Mac OS X 10.11.6
Comment on attachment 320383 [details] Patch Attachment 320383 [details] did not pass mac-debug-ews (mac): Output: http://webkit-queues.webkit.org/results/4502502 New failing tests: inspector/canvas/recording-2d.html inspector/canvas/recording-webgl-snapshots.html
Created attachment 320389 [details] Archive of layout-test-results from ews114 for mac-elcapitan The attached test failures were seen while running run-webkit-tests on the mac-debug-ews. Bot: ews114 Port: mac-elcapitan Platform: Mac OS X 10.11.6
Created attachment 320403 [details] Patch
Comment on attachment 320403 [details] Patch Attachment 320403 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.webkit.org/results/4507030 New failing tests: inspector/canvas/recording-2d.html
Created attachment 320405 [details] Archive of layout-test-results from ews105 for mac-elcapitan-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: ews105 Port: mac-elcapitan-wk2 Platform: Mac OS X 10.11.6
Comment on attachment 320403 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=320403&action=review Awesome! r=me > LayoutTests/inspector/canvas/resources/recording-utilities.js:71 > + if (action.parameters[i] != action._payloadParameters[i] && Number.isInteger(action._payloadParameters[i])) Style: Can we use `!==` here?
Comment on attachment 320403 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=320403&action=review > LayoutTests/inspector/canvas/resources/recording-utilities.js:81 > + for (let i = 0; i < action.trace.length; ++i) { You might want to use `k` instead of `i` again given you are already inside of an `i`. This would trigger "shadow" linter warnings, but it does behave fine right now, its just weird form.
Comment on attachment 320403 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=320403&action=review >> LayoutTests/inspector/canvas/resources/recording-utilities.js:71 >> + if (action.parameters[i] != action._payloadParameters[i] && Number.isInteger(action._payloadParameters[i])) > > Style: Can we use `!==` here? The main reason for not using `!==` is that `1 !== true`, which is what happens when we swizzle boolean values. I could add another case for that, but I figured that this was slightly cleaner. >> LayoutTests/inspector/canvas/resources/recording-utilities.js:81 >> + for (let i = 0; i < action.trace.length; ++i) { > > You might want to use `k` instead of `i` again given you are already inside of an `i`. This would trigger "shadow" linter warnings, but it does behave fine right now, its just weird form. Crap. Weird that ESLint didn't show it :(
Created attachment 320567 [details] Patch
Comment on attachment 320567 [details] Patch Attachment 320567 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.webkit.org/results/4527312 New failing tests: inspector/canvas/recording-2d.html
Created attachment 320574 [details] Archive of layout-test-results from ews107 for mac-elcapitan-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: ews107 Port: mac-elcapitan-wk2 Platform: Mac OS X 10.11.6
Created attachment 320730 [details] Patch
Comment on attachment 320730 [details] Patch Attachment 320730 [details] did not pass mac-debug-ews (mac): Output: http://webkit-queues.webkit.org/results/4542658 New failing tests: inspector/canvas/recording-webgl.html
Created attachment 320741 [details] Archive of layout-test-results from ews112 for mac-elcapitan The attached test failures were seen while running run-webkit-tests on the mac-debug-ews. Bot: ews112 Port: mac-elcapitan Platform: Mac OS X 10.11.6
Created attachment 320743 [details] Patch
Comment on attachment 320743 [details] Patch Attachment 320743 [details] did not pass mac-debug-ews (mac): Output: http://webkit-queues.webkit.org/results/4543612 New failing tests: inspector/canvas/recording-webgl.html
Created attachment 320751 [details] Archive of layout-test-results from ews114 for mac-elcapitan The attached test failures were seen while running run-webkit-tests on the mac-debug-ews. Bot: ews114 Port: mac-elcapitan Platform: Mac OS X 10.11.6
Created attachment 320757 [details] Patch
Created attachment 320856 [details] Patch
Comment on attachment 320856 [details] Patch Attachment 320856 [details] did not pass mac-debug-ews (mac): Output: http://webkit-queues.webkit.org/results/4552934 New failing tests: inspector/canvas/recording-webgl.html
Created attachment 320861 [details] Archive of layout-test-results from ews112 for mac-elcapitan The attached test failures were seen while running run-webkit-tests on the mac-debug-ews. Bot: ews112 Port: mac-elcapitan Platform: Mac OS X 10.11.6
Created attachment 320868 [details] Patch
Comment on attachment 320868 [details] Patch r=me!
Comment on attachment 320868 [details] Patch Clearing flags on attachment: 320868 Committed r222182: <http://trac.webkit.org/changeset/222182>
All reviewed patches have been landed. Closing bug.
<rdar://problem/34694214>