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]
:
: J. Ryan Stinnett [:jryans] (use ni?)
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 | Splinter Review
debug build log parser (script) (2.92 KB, text/plain)
2011-08-20 05:37 PDT, Tim Taubert [:ttaubert]
no flags Details

Description User image 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 User image Tim Taubert [:ttaubert] 2011-08-20 02:05:59 PDT
Created attachment 554622 [details] [diff] [review]
patch v1
Comment 2 User image 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 User image 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 User image Tim Taubert [:ttaubert] 2011-08-20 10:24:11 PDT
http://hg.mozilla.org/integration/fx-team/rev/d827ae445f33
Comment 5 User image 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.