Bug 1529762 - Update browser_page_style.js test to use a normal web page rather than a data URI. r=gl
authorMike Conley <mconley@mozilla.com>
Fri, 08 Mar 2019 20:22:52 +0000
changeset 521182 8ac8ed5ac772
parent 521181 92aa678ac36b
child 521183 b3d74a35dd1b
push id10862
push userffxbld-merge
push dateMon, 11 Mar 2019 13:01:11 +0000
treeherdermozilla-beta@a2e7f5c935da [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersgl
bugs1529762
milestone67.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 1529762 - Update browser_page_style.js test to use a normal web page rather than a data URI. r=gl The PageStyleChild now only attaches to normal web pages. Differential Revision: https://phabricator.services.mozilla.com/D22614
devtools/client/responsive.html/test/browser/browser.ini
devtools/client/responsive.html/test/browser/browser_page_style.js
devtools/client/responsive.html/test/browser/page_style.html
--- a/devtools/client/responsive.html/test/browser/browser.ini
+++ b/devtools/client/responsive.html/test/browser/browser.ini
@@ -9,16 +9,17 @@ support-files =
   devices.json
   doc_page_state.html
   doc_toolbox_rule_view.css
   doc_toolbox_rule_view.html
   favicon.html
   favicon.ico
   geolocation.html
   head.js
+  page_style.html
   touch.html
   !/devtools/client/inspector/test/shared-head.js
   !/devtools/client/shared/test/shared-head.js
   !/devtools/client/shared/test/shared-redux-head.js
   !/devtools/client/shared/test/telemetry-test-helpers.js
   !/devtools/client/shared/test/test-actor.js
   !/devtools/client/shared/test/test-actor-registry.js
 
--- a/devtools/client/responsive.html/test/browser/browser_page_style.js
+++ b/devtools/client/responsive.html/test/browser/browser_page_style.js
@@ -1,17 +1,17 @@
 /* Any copyright is dedicated to the Public Domain.
 http://creativecommons.org/publicdomain/zero/1.0/ */
 
 "use strict";
 
 // Test that the Page Style browser menu actions make it to the viewport, instead of
 // applying to the RDM UI.
 
-const TEST_URL = "data:text/html;charset=utf-8,<style>body{color:red}</style>Hello RDM";
+const TEST_URL = `${URL_ROOT}page_style.html`;
 
 addRDMTask(TEST_URL, async function({ ui, manager }) {
   // Store the RDM body text color for later.
   const rdmWindow = ui.toolWindow;
   const rdmTextColor = rdmWindow.getComputedStyle(rdmWindow.document.body).color;
 
   info("Trigger the no page style action and wait for the text color to change");
   let onPageColorChanged = waitForContentPageTextColor(ui, "rgb(0, 0, 0)");
new file mode 100644
--- /dev/null
+++ b/devtools/client/responsive.html/test/browser/page_style.html
@@ -0,0 +1,7 @@
+<style>
+body{
+  color: red;
+}
+</style>
+
+Hello RDM