Web Inspector: Canvas: remove event listeners once a recording is ready

[Devin Rousso]

.

[Devin Rousso]

Created attachment 363626 [details]
Patch

[EWS Watchlist]

Comment on attachment 363626 [details]
Patch

Attachment 363626 [details] did not pass ios-sim-ews (ios-simulator-wk2):
Output: https://webkit-queues.webkit.org/results/11376383

New failing tests:
editing/selection/thai-word-at-document-end.html

[EWS Watchlist]

Created attachment 363636 [details]
Archive of layout-test-results from ews124 for ios-simulator-wk2

The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews124  Port: ios-simulator-wk2  Platform: Mac OS X 10.13.6

[Matt Baker]

Comment on attachment 363626 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=363626&action=review

r=me

> Source/WebInspectorUI/UserInterface/Models/Recording.js:536
> +        this._swizzle = null;

Style: this can go after this._processing = false. I don't think it's necessary to separate these by group.

[Devin Rousso]

Created attachment 363682 [details]
Patch

[WebKit Commit Bot]

Comment on attachment 363682 [details]
Patch

Clearing flags on attachment: 363682

Committed r242511: <https://trac.webkit.org/changeset/242511>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Radar WebKit Bug Importer]

<rdar://problem/48613617>