Bug 1397651 - Update tests within devtools/ to comply with new toplevel data: URI navigation policy. r=bgrinstead
authorChristoph Kerschbaumer <ckerschb@christophkerschbaumer.com>
Thu, 07 Sep 2017 20:10:59 +0200
changeset 429111 860d5124faec334197b427dd888a40e4d291528a
parent 429110 7922386758a48b3eceef1291222e17de610e610a
child 429112 10c02a1f786df8f515e2678a7dd36ea22bf9c385
push id7761
push userjlund@mozilla.com
push dateFri, 15 Sep 2017 00:19:52 +0000
treeherdermozilla-beta@c38455951db4 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersbgrinstead
bugs1397651
milestone57.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 1397651 - Update tests within devtools/ to comply with new toplevel data: URI navigation policy. r=bgrinstead
devtools/client/inspector/markup/test/browser.ini
devtools/client/inspector/markup/test/browser_markup_update-on-navigtion.js
devtools/client/inspector/markup/test/doc_markup_update-on-navigtion_1.html
devtools/client/inspector/markup/test/doc_markup_update-on-navigtion_2.html
devtools/shared/heapsnapshot/tests/mochitest/file_saveHeapSnapshot_e10s_01.html
devtools/shared/heapsnapshot/tests/mochitest/mochitest.ini
devtools/shared/heapsnapshot/tests/mochitest/test_saveHeapSnapshot_e10s_01.html
--- a/devtools/client/inspector/markup/test/browser.ini
+++ b/devtools/client/inspector/markup/test/browser.ini
@@ -33,16 +33,18 @@ support-files =
   doc_markup_search.html
   doc_markup_svg_attributes.html
   doc_markup_toggle.html
   doc_markup_tooltip.png
   doc_markup_void_elements.html
   doc_markup_void_elements.xhtml
   doc_markup_whitespace.html
   doc_markup_xul.xul
+  doc_markup_update-on-navigtion_1.html
+  doc_markup_update-on-navigtion_2.html
   events_bundle.js
   events_bundle.js.map
   events_original.js
   head.js
   helper_attributes_test_runner.js
   helper_diff.js
   helper_events_test_runner.js
   helper_markup_accessibility_navigation.js
--- a/devtools/client/inspector/markup/test/browser_markup_update-on-navigtion.js
+++ b/devtools/client/inspector/markup/test/browser_markup_update-on-navigtion.js
@@ -1,18 +1,17 @@
 /* vim: set ts=2 et sw=2 tw=80: */
 /* Any copyright is dedicated to the Public Domain.
  * http://creativecommons.org/publicdomain/zero/1.0/ */
 "use strict";
 
 // Test that markup view handles page navigation correctly.
 
-const SCHEMA = "data:text/html;charset=UTF-8,";
-const URL_1 = SCHEMA + "<div id='one' style='color:red;'>ONE</div>";
-const URL_2 = SCHEMA + "<div id='two' style='color:green;'>TWO</div>";
+const URL_1 = URL_ROOT + "doc_markup_update-on-navigtion_1.html";
+const URL_2 = URL_ROOT + "doc_markup_update-on-navigtion_2.html";
 
 add_task(function* () {
   let {inspector, testActor} = yield openInspectorForURL(URL_1);
 
   assertMarkupViewIsLoaded();
   yield selectNode("#one", inspector);
 
   let willNavigate = inspector.target.once("will-navigate");
new file mode 100644
--- /dev/null
+++ b/devtools/client/inspector/markup/test/doc_markup_update-on-navigtion_1.html
@@ -0,0 +1,1 @@
+<div id='one' style='color:red;'>ONE</div>
new file mode 100644
--- /dev/null
+++ b/devtools/client/inspector/markup/test/doc_markup_update-on-navigtion_2.html
@@ -0,0 +1,1 @@
+<div id='two' style='color:green;'>TWO</div>
new file mode 100644
--- /dev/null
+++ b/devtools/shared/heapsnapshot/tests/mochitest/file_saveHeapSnapshot_e10s_01.html
@@ -0,0 +1,1 @@
+<!DOCTYPE HTML><html><body></body></html>
--- a/devtools/shared/heapsnapshot/tests/mochitest/mochitest.ini
+++ b/devtools/shared/heapsnapshot/tests/mochitest/mochitest.ini
@@ -1,6 +1,7 @@
 [DEFAULT]
 tags = devtools devtools-memory
 support-files =
+  file_saveHeapSnapshot_e10s_01.html
 
 [test_saveHeapSnapshot_e10s_01.html]
 
--- a/devtools/shared/heapsnapshot/tests/mochitest/test_saveHeapSnapshot_e10s_01.html
+++ b/devtools/shared/heapsnapshot/tests/mochitest/test_saveHeapSnapshot_e10s_01.html
@@ -14,17 +14,17 @@ Bug 1201597 - Sanity test that we can ta
     <script type="application/javascript">
     "use strict";
     window.onerror = function (msg, url, line, col, err) {
       ok(false, "@" + url + ":" + line + ":" + col + ": " + msg + "\n" + err.stack);
     };
 
     SimpleTest.waitForExplicitFinish();
 
-    var childFrameURL = "data:text/html,<!DOCTYPE HTML><html><body></body></html>";
+    var childFrameURL = "file_saveHeapSnapshot_e10s_01.html";
 
     // This function is stringified and loaded in the child process as a frame
     // script.
     function childFrameScript() {
       try {
         ChromeUtils.saveHeapSnapshot({ runtime: true });
       } catch (err) {
         sendAsyncMessage("testSaveHeapSnapshot:error",