Bug 1383723 - remove unused mdn docs mochitest;r=jdescottes
authorJulian Descottes <jdescottes@mozilla.com>
Fri, 21 Jul 2017 23:18:39 +0200
changeset 421676 167ac4c7679b9df96920d5c2259226c00dc88e5b
parent 421675 17d7de773b6d3aecd88fdd7882b7d236b4f62e37
child 421677 495c6b6e45a784ee1fd39aab0a02eff3d526e1b0
push id1517
push userjlorenzo@mozilla.com
push dateThu, 14 Sep 2017 16:50:54 +0000
treeherdermozilla-release@3b41fd564418 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjdescottes
bugs1383723
milestone56.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 1383723 - remove unused mdn docs mochitest;r=jdescottes This mochitest was never added to browser.ini, and never ran on try. I tested it locally and the test still seems to run. However the mdn docs widget is now disabled by default and is actually not working very well when enabled. I propose to remove the file rather than slowing down our test suite for this. MozReview-Commit-ID: 4QEIdUQRgJZ
devtools/client/inspector/rules/test/browser_rules_css-docs-tooltip_closes-on-escape.js
deleted file mode 100644
--- a/devtools/client/inspector/rules/test/browser_rules_css-docs-tooltip_closes-on-escape.js
+++ /dev/null
@@ -1,52 +0,0 @@
-/* vim: set ft=javascript ts=2 et sw=2 tw=80: */
-/* Any copyright is dedicated to the Public Domain.
- http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/**
- * Test that the CssDocs tooltip of the ruleview can be closed when pressing the Escape
- * key.
- */
-
-"use strict";
-
-const {setBaseCssDocsUrl} =
-  require("devtools/client/shared/widgets/MdnDocsWidget");
-
-const PROPERTYNAME = "color";
-
-const TEST_URI = `
-  <html>
-    <body>
-      <div style="color: red">
-        Test "Show MDN Docs" closes on escape
-      </div>
-    </body>
-  </html>
-`;
-
-/**
- * Test that the tooltip is hidden when we press Escape
- */
-add_task(function* () {
-  yield addTab("data:text/html;charset=utf8," + encodeURIComponent(TEST_URI));
-  let {inspector, view} = yield openRuleView();
-  yield selectNode("div", inspector);
-
-  setBaseCssDocsUrl(URL_ROOT);
-
-  info("Retrieve a valid anchor for the CssDocs tooltip");
-  let {nameSpan} = getRuleViewProperty(view, "element", PROPERTYNAME);
-
-  info("Showing the MDN docs tooltip");
-  let cssDocs = view.tooltips.getTooltip("cssDocs");
-  let onShown = cssDocs.tooltip.once("shown");
-  cssDocs.show(nameSpan, PROPERTYNAME);
-  yield onShown;
-  ok(true, "The MDN docs tooltip was shown");
-
-  info("Simulate pressing the 'Escape' key");
-  let onHidden = cssDocs.tooltip.once("hidden");
-  EventUtils.sendKey("escape");
-  yield onHidden;
-  ok(true, "The MDN docs tooltip was hidden on pressing 'escape'");
-});