Tim Taubert [:ttaubert] (inactive) Assignee
	Description • 10 years ago

Those are regressions from bug 886570.

	Tim Taubert [:ttaubert] (inactive) Assignee
	Comment 2 • 10 years ago

https://tbpl.mozilla.org/?tree=Try&rev=ac9f5bfcb788

	Tim Taubert [:ttaubert] (inactive) Assignee
	Comment 3 • 10 years ago

Sorry.

https://tbpl.mozilla.org/?tree=Try&rev=f4b7fe55d07e

	Tim Taubert [:ttaubert] (inactive) Assignee
	Comment 4 • 10 years ago

The one leak we see so far is reproducible locally for me:

toolkit/mozapps/extensions/test/browser/browser_openH264.js | leaked 2 window(s) until shutdown [url = about:addons]

	Tim Taubert [:ttaubert] (inactive) Assignee
	Comment 5 • 10 years ago

Try push with a fix for bug 1052409:

https://tbpl.mozilla.org/?tree=Try&rev=e1d01b882103

	Ahmed Kachkach [:akachkach]
	Comment 6 • 10 years ago

Comment on attachment 8471520 [details] [diff] [review]
0001-Bug-1052371-Fix-m-bc-shutdown-leak-detector.patch, v2

Review of attachment 8471520 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good to me! The "suite_end" part is probably due to the fact that we don't process the messages generated through Python by those handlers (and that was previously generated by the JS side).
I went ahead and checked that "TEST-START | Shutdown" is still printed by mochitest-plain, mochitest-chrome and robocop.

	Tim Taubert [:ttaubert] (inactive) Assignee
	Comment 7 • 10 years ago

https://hg.mozilla.org/integration/fx-team/rev/d04a9bf0df24

	Tim Taubert [:ttaubert] (inactive) Assignee
	Comment 8 • 10 years ago

Oops, Android mochitests were failing. Pushed a tiny bustage fix:

https://hg.mozilla.org/integration/fx-team/rev/9b26ce378971

	Tim Taubert [:ttaubert] (inactive) Assignee
	Comment 9 • 10 years ago

Had to back out unfortunately. DevTools tests leak the world :(

https://hg.mozilla.org/integration/fx-team/rev/e5249cb294a8
https://hg.mozilla.org/integration/fx-team/rev/a60a276fb205

	Tim Taubert [:ttaubert] (inactive) Assignee
	Comment 10 • 10 years ago

Here is a leak log:

https://tbpl.mozilla.org/php/getParsedLog.php?id=45772397&tree=Fx-Team

	Tim Taubert [:ttaubert] (inactive) Assignee
	Comment 11 • 10 years ago

New try push with the patch from bug 1052756:

https://tbpl.mozilla.org/?tree=Try&rev=469635719a6b

	Ryan VanderMeulen [:RyanVM]
	Comment 12 • 10 years ago

https://hg.mozilla.org/mozilla-central/rev/c865443dfb0c

	Ryan VanderMeulen [:RyanVM]
	Comment 13 • 10 years ago

https://hg.mozilla.org/releases/mozilla-aurora/rev/540ac322aac8