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] (on PTO, back Aug 29th)
:
Mentors:
Depends on:
Blocks: bc-leaks 671543
  Show dependency treegraph
 
Reported: 2011-08-20 02:02 PDT by Tim Taubert [:ttaubert] (on PTO, back Aug 29th)
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] (on PTO, back Aug 29th)
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] (on PTO, back Aug 29th)
no flags Details

Description Tim Taubert [:ttaubert] (on PTO, back Aug 29th) 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] (on PTO, back Aug 29th) 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] (on PTO, back Aug 29th) 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] (on PTO, back Aug 29th) 2011-08-20 10:24:11 PDT
http://hg.mozilla.org/integration/fx-team/rev/d827ae445f33
Comment 5 Tim Taubert [:ttaubert] (on PTO, back Aug 29th) 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.