Closed Bug 500615 Opened 15 years ago Closed 15 years ago

[HTML5] Recycle continue event objects

Categories

(Core :: DOM: HTML Parser, defect)

Product:
Core
Component:
DOM: HTML Parser
Version:
Other Branch
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED INVALID

People

(Reporter: hsivonen, Unassigned)

Details

From bug 487949:
> >+  // This creates a reference cycle between this and the event that is
> >+  // broken when the event fires.
> >+  nsCOMPtr<nsIRunnable> event = new nsHtml5ParserContinueEvent(this);
> >+  if (NS_FAILED(NS_DispatchToCurrentThread(event))) {
> >+      NS_WARNING("failed to dispatch parser continuation event");
> >+  } else {
> >+      mContinueEvent = event;
> >+  }
> 
> Probably worth keeping the event object around and reuse it multiple times.

(Need to deal with the reference cycle issues.)
The continue event is gone.
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.