REGRESSION (r262643): DumpRenderTree at com.apple.WebCore: WebCore::Document::prepareCanvasesForDisplayIfNeeded
rdar://64260400
Created attachment 401954 [details] Patch
Comment on attachment 401954 [details] Patch I think you should split this into two patches.
Created attachment 402049 [details] Patch
Comment on attachment 402049 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=402049&action=review > Source/WebCore/dom/Document.h:1601 > + void clearCanvasPreparation(HTMLCanvasElement*); I don't like how we have multiple ways we do this. There's generic Element::didMove... stuff, there's HTMLMediaElement::unregisterWithDocument(), Document::unregisterForCaptionPreferencesChangedCallbacks(), Document::unregisterArticleElement() etc. We should add one overridable hook for elements that need to do custom document removal. Maybe not in this patch.
Committed r263128: <https://trac.webkit.org/changeset/263128>