Plug the leak in browser_toolbox_selected_tool_unavailable.js in a CLOSED TREE (bug 915448). r=testonly
authorPanos Astithas <past@mozilla.com>
Tue, 12 May 2015 15:58:53 +0300
changeset 243492 d5dd7ebbeece5695b47e8ea2b4e6564e4b7fa1db
parent 243491 3f78f06e37f6f92dc49ad0a53e9485fb8cd604ce
child 243493 5f6263cb1e504c462ba7f304f93b93b6d153f026
push id28740
push userkwierso@gmail.com
push dateTue, 12 May 2015 23:04:57 +0000
treeherdermozilla-central@de1fd9d0682a [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerstestonly
bugs915448
milestone40.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Plug the leak in browser_toolbox_selected_tool_unavailable.js in a CLOSED TREE (bug 915448). r=testonly
browser/devtools/framework/test/browser_toolbox_selected_tool_unavailable.js
--- a/browser/devtools/framework/test/browser_toolbox_selected_tool_unavailable.js
+++ b/browser/devtools/framework/test/browser_toolbox_selected_tool_unavailable.js
@@ -33,11 +33,13 @@ add_task(function*() {
 
   // Make the previously selected tool unavailable.
   testToolDefinition.isTargetSupported = () => false;
 
   target = TargetFactory.forTab(tab);
   toolbox = yield gDevTools.showToolbox(target);
   is(toolbox.currentToolId, "webconsole", "web console was selected");
 
+  yield gDevTools.closeToolbox(target);
   gDevTools.unregisterTool(testToolDefinition.id);
+  tab = toolbox = target = null;
   gBrowser.removeCurrentTab();
 });