Bug 1138976 - Leave the toolbox destruction to the head.js cleanup method when running markup view event tooltip tests. r=pbrosset, a=test-only FIREFOX_BETA_40_BASE
authorSami Jaktholm <sjakthol@outlook.com>
Sat, 20 Jun 2015 14:28:49 +0300
changeset 275214 d77cf39268848f8a7e9b38082348b6cd4d1f3b5e
parent 275213 318d4b92646dcb6e8f5f9da3c812e21a8dcf552e
child 275215 9d83ab013ab61c7f6e987bf0e7cbab1a1aed1ca8
push id863
push userraliiev@mozilla.com
push dateMon, 03 Aug 2015 13:22:43 +0000
treeherdermozilla-release@f6321b14228d [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerspbrosset, test-only
bugs1138976
milestone40.0a2
Bug 1138976 - Leave the toolbox destruction to the head.js cleanup method when running markup view event tooltip tests. r=pbrosset, a=test-only
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