Closed Bug 785048 Opened 12 years ago Closed 11 years ago

Investigate if the manual cleanup of the script table can be removed

Categories

(DevTools :: Debugger, defect, P3)

defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: past, Unassigned)

Details

In TA_clearDebuggees (and originally in TA_disconnect) we have this bit of code to clean up the script table:

    // Unless we carefully take apart the scripts table this way, we end up
    // leaking documents. It would be nice to track this down carefully, once
    // we have the appropriate tools.
    for (let url in this._scripts) {
      delete this._scripts[url];
    }
    this._scripts = {};

One would assume that only the last assignment would suffice, but I had noted leaks without it.

We may be able to put about:ccdump or about:cc to good use in finding the cause of this leak and hopefully simplify the code here.
Priority: -- → P3
This was actually fixed in bug 740551.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.