Bug 1138976 - Leave the toolbox destruction to the head.js cleanup method when running markup view event tooltip tests. r=pbrosset
authorSami Jaktholm <sjakthol@outlook.com>
Sat, 20 Jun 2015 14:28:49 +0300
changeset 280992 a55da7f74011cb055300ac89f08cffcf1f3c9efe
parent 280991 76a609fd0c81755cf86aa3524073b1c155c46bac
child 280993 d3f0281f73572af565f2f777af91eb43621d3996
push id4932
push userjlund@mozilla.com
push dateMon, 10 Aug 2015 18:23:06 +0000
treeherdermozilla-beta@6dd5a4f5f745 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerspbrosset
bugs1138976
milestone41.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
Bug 1138976 - Leave the toolbox destruction to the head.js cleanup method when running markup view event tooltip tests. r=pbrosset
browser/devtools/markupview/test/helper_events_test_runner.js
--- a/browser/devtools/markupview/test/helper_events_test_runner.js
+++ b/browser/devtools/markupview/test/helper_events_test_runner.js
@@ -10,18 +10,16 @@ function* runEventPopupTests() {
   let {inspector} = yield addTab(TEST_URL).then(openInspector);
 
   yield inspector.markup.expandAll();
 
   for (let {selector, expected} of TEST_DATA) {
     yield checkEventsForNode(selector, expected, inspector);
   }
 
-  gBrowser.removeCurrentTab();
-
   // Wait for promises to avoid leaks when running this as a single test.
   // We need to do this because we have opened a bunch of popups and don't them
   // to affect other test runs when they are GCd.
   yield promiseNextTick();
 }
 
 /**
  * Generator function that takes a selector and expected results and returns