Simplify XHR JS wrapper lifetime management and align with the specification: - https://xhr.spec.whatwg.org/#garbage-collection
Created attachment 392202 [details] Patch
Created attachment 392318 [details] Patch
Created attachment 392770 [details] Patch
Created attachment 392802 [details] Patch
Created attachment 392809 [details] Patch
Comment on attachment 392809 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=392809&action=review > Source/WebCore/ChangeLog:11 > + e now override ActiveDOMObject::hasPendingActivity() to match exactly the text Nit: e -> We? > Source/WebCore/xml/XMLHttpRequest.cpp:1167 > + || hasEventListeners(eventNames().loadEvent) > + || hasEventListeners(eventNames().timeoutEvent) Can we order these events in alphabetical/lexicological order?
Comment on attachment 392809 [details] Patch Oops, I didn't meant to set cq+
Created attachment 393061 [details] Patch
Comment on attachment 393061 [details] Patch Clearing flags on attachment: 393061 Committed r258159: <https://trac.webkit.org/changeset/258159>
All reviewed patches have been landed. Closing bug.
<rdar://problem/60239100>
Comment on attachment 393061 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=393061&action=review > Source/WebCore/xml/XMLHttpRequestProgressEventThrottle.h:36 > class EventTarget; Should have removed this.
(In reply to Darin Adler from comment #12) > Comment on attachment 393061 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=393061&action=review > > > Source/WebCore/xml/XMLHttpRequestProgressEventThrottle.h:36 > > class EventTarget; > > Should have removed this. Indeed, will follow up, thanks.
(In reply to Chris Dumez from comment #13) > (In reply to Darin Adler from comment #12) > > Comment on attachment 393061 [details] > > Patch > > > > View in context: > > https://bugs.webkit.org/attachment.cgi?id=393061&action=review > > > > > Source/WebCore/xml/XMLHttpRequestProgressEventThrottle.h:36 > > > class EventTarget; > > > > Should have removed this. > > Indeed, will follow up, thanks. <https://trac.webkit.org/changeset/258165>