[Webkit-unassigned] [Bug 128532] [MSE] Move PublicURLManager shutdown logic so ActiveDOMObjects associated with public URLs won't leak.
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Tue Feb 11 08:58:41 PST 2014
https://bugs.webkit.org/show_bug.cgi?id=128532
--- Comment #8 from Byungseon Shin <sun.shin at lge.com> 2014-02-11 08:55:57 PST ---
(From update of attachment 223850)
View in context: https://bugs.webkit.org/attachment.cgi?id=223850&action=review
>> Source/WebCore/ChangeLog:6
>> + Reviewed by NOBODY (OOPS!).
>
> I was really confused until I read the bug report. It would be nice to add some text to the ChangeLog which says what this fix does. Something like:
>
> This fixes a leak of DOM objects by breaking the circular reference between Document, PublicURLManager, and MediaSource. Instead of clearing PublicURLManager at destruction-time, which is delayed indefinitely because of the circular reference, clear the PublicURLManager during ActiveDOMObject::stop().
@Jer, thanks for the comment. I have applied new patch.
>> Source/WebCore/html/PublicURLManager.h:48
>> + static PassOwnPtr<PublicURLManager> create(ScriptExecutionContext*);
>
> We should fix this function to return a std::unique_ptr<>, but we don't have to do it in this patch.
OK, I'll prepare a new patch for it.
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list