Closed
Bug 734172
Opened 13 years ago
Closed 12 years ago
browser_save_resend_postdata.js and browser_aboutCrashes.js erroneously blamed for leaking 2 DOMWindows and 1 DocShell
Categories
(Testing :: Mochitest, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
mozilla15
People
(Reporter: sgautherie, Assigned: dao)
References
(Blocks 1 open bug, )
Details
Attachments
(1 file)
(deleted),
patch
|
ttaubert
:
review+
|
Details | Diff | Splinter Review |
https://tbpl.mozilla.org/php/getParsedLog.php?id=9910719&tree=Firefox&full=1 Rev3 WINNT 6.1 mozilla-central debug test mochitest-other on 2012-03-08 06:30:34 PST for push 0337458a08ba { [toolkit/content/tests/browser/browser_save_resend_postdata.js] 1 window(s) [url = about:blank] 1 docShell(s) } http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1331186151.1331190870.3486.gz&fulltext=1 Linux comm-central-trunk debug test mochitest-other on 2012/03/07 21:55:51 { [toolkit/content/tests/browser/browser_save_resend_postdata.js] 1 window(s) [url = about:blank] 1 docShell(s) }
Reporter | ||
Comment 1•12 years ago
|
||
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1336024558.1336031502.20286.gz&fulltext=1 WINNT 5.2 comm-central-trunk debug test mochitest-other on 2012/05/02 22:55:58
Assignee | ||
Updated•12 years ago
|
Severity: major → normal
Assignee | ||
Updated•12 years ago
|
Component: Download Manager → BrowserTest
Product: Toolkit → Testing
QA Contact: download.manager → browsertest
Summary: browser_save_resend_postdata.js leaks 1 DOMWindow and 1 DocShell → browser_save_resend_postdata.js and browser_aboutCrashes.js erroneously blamed for leaking 2 DOMWindows and 1 DocShell
Assignee | ||
Comment 3•12 years ago
|
||
Updated•12 years ago
|
Attachment #624313 -
Flags: review?(ttaubert) → review+
Comment 4•12 years ago
|
||
Comment on attachment 624313 [details] [diff] [review] patch Review of attachment 624313 [details] [diff] [review]: ----------------------------------------------------------------- Please note that while I'm fine with those changes I'm technically not the right one for reviewing mochitest suites changes.
Assignee | ||
Comment 5•12 years ago
|
||
(In reply to Tim Taubert [:ttaubert] from comment #4) > Please note that while I'm fine with those changes I'm technically not the > right one for reviewing mochitest suites changes. I think this change is simple enough and not really interesting outside the context of leaks. http://hg.mozilla.org/integration/mozilla-inbound/rev/c220dcb86e4e
Target Milestone: --- → mozilla15
Assignee | ||
Comment 6•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/c220dcb86e4e https://hg.mozilla.org/mozilla-central/rev/908e9a9acabd
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 7•12 years ago
|
||
SM 2.12a1: http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1337320632.1337325277.11695.gz&fulltext=1 WINNT 5.2 comm-central-trunk debug test mochitest-other on 2012/05/17 22:57:12 V.Fixed
Status: RESOLVED → VERIFIED
Flags: in-testsuite-
Updated•7 years ago
|
Component: BrowserTest → Mochitest
You need to log in
before you can comment on or make changes to this bug.
Description
•