Bug 1452143: Fix two more tests which don't run on debug try on a CLOSED TREE. r=me
authorEmilio Cobos Álvarez <emilio@crisal.io>
Tue, 17 Apr 2018 14:26:59 +0200
changeset 467630 a5aa0c6be992161aa110cfe91ad813a81a81767f
parent 467629 350013b0e02d8f89925d8749f5cbd884e1eb1032
child 467631 8ebe4719c2414dc32568d67a2889c48adb686031
push id9165
push userasasaki@mozilla.com
push dateThu, 26 Apr 2018 21:04:54 +0000
treeherdermozilla-beta@064c3804de2e [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersme
bugs1452143
milestone61.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 1452143: Fix two more tests which don't run on debug try on a CLOSED TREE. r=me MozReview-Commit-ID: B3kj7tzn3Oq
browser/base/content/test/static/browser_parsable_css.js
layout/style/test/test_parser_diagnostics_unprintables.html
--- a/browser/base/content/test/static/browser_parsable_css.js
+++ b/browser/base/content/test/static/browser_parsable_css.js
@@ -334,16 +334,17 @@ add_task(async function checkAllTheCSS()
   let hiddenFrame = new HiddenFrame();
   let win = await hiddenFrame.get();
   let iframe = win.document.createElementNS("http://www.w3.org/1999/xhtml", "html:iframe");
   win.document.documentElement.appendChild(iframe);
   let iframeLoaded = BrowserTestUtils.waitForEvent(iframe, "load", true);
   iframe.contentWindow.location = testFile;
   await iframeLoaded;
   let doc = iframe.contentWindow.document;
+  doc.docShell.cssErrorReportingEnabled = true;
 
   // Parse and remove all manifests from the list.
   // NOTE that this must be done before filtering out devtools paths
   // so that all chrome paths can be recorded.
   let manifestURIs = [];
   uris = uris.filter(uri => {
     if (uri.pathQueryRef.endsWith(".manifest")) {
       manifestURIs.push(uri);
--- a/layout/style/test/test_parser_diagnostics_unprintables.html
+++ b/layout/style/test/test_parser_diagnostics_unprintables.html
@@ -15,16 +15,18 @@
 <script type="application/javascript">
 // This test has intimate knowledge of how to get the CSS parser to
 // emit diagnostics that contain text under control of the user.
 // That's not the point of the test, though; the point is only that
 // *that text* is properly escaped.
 
 const isStylo = SpecialPowers.DOMWindowUtils.isStyledByServo;
 
+SpecialPowers.wrap(document).docShell.cssErrorReportingEnabled = true;
+
 // There is one "pattern" for each code path through the error reporter
 // that might need to escape some kind of user-supplied text.
 // Each "pattern" is tested once with each of the "substitution"s below:
 // <t>, <i>, and <s> are replaced by the t:, i:, and s: fields of
 // each substitution object in turn.
 let patterns = [
   // REPORT_UNEXPECTED_P (only ever used in contexts where identifier-like
   // escaping is appropriate)