DescriptionRemove HTMLParserScriptRunner's prefinalizer
We don't have to Dispose() PendingScripts, because:
- If HTMLParserScriptRunner is watching for load of a PendingScript, then
the PendingScript has a Member<HTMLParserScriptRunner> of |this|,
and thus at prefinalization the PendingScript is also to be garbage
collected. Dispose() is called by PendingScript's prefinalizer.
- Otherwise, what matters in PendingScript::Dispose() is
ScriptStreamer::Cancel() in ClassicPendingScript::DisposeInternal().
This CL might defer that ScriptStreamer::Cancel() to when
ClassicPendingScript is prefinalized, but this shouldn't cause anything
functionally wrong.
BUG=715309
Review-Url: https://codereview.chromium.org/2837363003
Cr-Commit-Position: refs/heads/master@{#467224}
Committed: https://chromium.googlesource.com/chromium/src/+/3d70ed2630a00fee3b5014b015ec765155885e54
Patch Set 1 #Patch Set 2 : Rebase #
Depends on Patchset: Dependent Patchsets: Messages
Total messages: 18 (13 generated)
|