Summary: A recording initiated by the user should be shown immediately on completion. This does not include recordings started using console.record. Steps to Reproduce: 1. Open https://developer.mozilla.org/en-US/docs/Web/API/Canvas_API/Tutorial/Basic_animations 2. Open Canvas tab 3. Click record on the "A looping panorama" canvas 4. Wait for a couple frames, then hit stop Actual: => Recording is added to "View Recordings" Expected: => Recording view is shown Note: This only affects multi-frame recordings.
<rdar://problem/38479187>
Created attachment 335813 [details] Patch
Comment on attachment 335813 [details] Patch r=me. A top-level comment about *why* this fixes the issue would be nice. I had to do some debugging to learn that CanvasManager was dispatching RecordingStopped with `fromConsole` set to true, because this._recordingCanvas was being nulled out too early.
Created attachment 335832 [details] Patch
Comment on attachment 335832 [details] Patch Clearing flags on attachment: 335832 Committed r229620: <https://trac.webkit.org/changeset/229620>
All reviewed patches have been landed. Closing bug.
Comment on attachment 335832 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=335832&action=review > Source/WebInspectorUI/ChangeLog:20 > + in `stopRecording`, so this patch just prevents it from being nulled too early. You really took that review comment and ran with it!