Last Comment Bug 680649 - inspector leaks iframe [chrome://browser/content/inspector.html] until shutdown
: inspector leaks iframe [chrome://browser/content/inspector.html] until shutdown
Status: RESOLVED FIXED
:
Product: Firefox
Classification: Client Software
Component: Developer Tools (show other bugs)
: Trunk
: All All
: -- normal (vote)
: Firefox 9
Assigned To: Tim Taubert [:ttaubert]
:
Mentors:
Depends on:
Blocks: bc-leaks 671543
  Show dependency treegraph
 
Reported: 2011-08-20 02:02 PDT by Tim Taubert [:ttaubert]
Modified: 2011-08-21 23:52 PDT (History)
3 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
patch v1 (1.42 KB, patch)
2011-08-20 02:05 PDT, Tim Taubert [:ttaubert]
dcamp: review+
Details | Diff | Review
debug build log parser (script) (2.92 KB, text/plain)
2011-08-20 05:37 PDT, Tim Taubert [:ttaubert]
no flags Details

Description Tim Taubert [:ttaubert] 2011-08-20 02:02:23 PDT
leak statistics:

[browser/base/content/test/inspector/browser_inspector_registertools.js]
  1x [data:text/html,registertool%20tests%20for%20inspector]
  2x [chrome://browser/content/inspector.html]
  1x [about:blank]

[browser/base/content/test/inspector/browser_inspector_tab_switch.js]
  3x [chrome://browser/content/inspector.html]

[browser/base/content/test/inspector/browser_inspector_bug_665880.js]
  2x [chrome://browser/content/inspector.html]

[browser/base/content/test/inspector/browser_inspector_highlighter.js]
  1x [chrome://browser/content/inspector.html]

[browser/base/content/test/inspector/browser_inspector_iframeTest.js]
  1x [chrome://browser/content/inspector.html]

[browser/base/content/test/inspector/browser_inspector_initialization.js]
  1x [chrome://browser/content/inspector.html]

[browser/base/content/test/inspector/browser_inspector_scrolling.js]
  1x [chrome://browser/content/inspector.html]

[browser/base/content/test/inspector/browser_inspector_treePanel_output.js]
  1x [chrome://browser/content/inspector.html]

[browser/base/content/test/inspector/browser_inspector_treeSelection.js]
  1x [chrome://browser/content/inspector.html]
Comment 1 Tim Taubert [:ttaubert] 2011-08-20 02:05:59 PDT
Created attachment 554622 [details] [diff] [review]
patch v1
Comment 2 Panos Astithas [:past] 2011-08-20 04:07:07 PDT
Tim, how did you manage to get those statistics? If it's a tool, it seems pretty useful!
Comment 3 Tim Taubert [:ttaubert] 2011-08-20 05:37:52 PDT
Created attachment 554627 [details]
debug build log parser (script)

(In reply to Panos Astithas [:past] from comment #2)
> Tim, how did you manage to get those statistics? If it's a tool, it seems
> pretty useful!

That's a little Python script I wrote yesterday. It's pretty messy and my Python skills are not the best but it does the job! :)

It parses debug build logs (only) and counts created and discarded DOMWindows and assigns them to tests and urls to the DOMWindows. All the information is already contained in the build log so all it does is parse the file.

./parse.py debug-build-log.txt > leak-output.txt
Comment 4 Tim Taubert [:ttaubert] 2011-08-20 10:24:11 PDT
http://hg.mozilla.org/integration/fx-team/rev/d827ae445f33
Comment 5 Tim Taubert [:ttaubert] 2011-08-21 23:52:02 PDT
http://hg.mozilla.org/mozilla-central/rev/d827ae445f33

Note You need to log in before you can comment on or make changes to this bug.