Bug 1377523 - Part 2: fix for browser_toolbox_*. r=pbro

fixed the tests in devtools/client/framework/test/ to not to use data:
URI.

devtools/client/framework/test/browser.ini

@@ -5,7 +5,10 @@ support-files =
   browser_toolbox_options_disable_js.html
   browser_toolbox_options_disable_js_iframe.html
   browser_toolbox_options_disable_cache.sjs
+  browser_toolbox_sidebar_existing_tabs.xul
+  browser_toolbox_sidebar_events.xul
   browser_toolbox_sidebar_tool.xul
+  browser_toolbox_sidebar_toolURL.xul
   browser_toolbox_window_title_changes_page.html
   browser_toolbox_window_title_frame_select_page.html
   code_binary_search.coffee

devtools/client/framework/test/browser_toolbox_sidebar.js

@@ -7,13 +7,6 @@ function test() {
   const Cu = Components.utils;
   let {ToolSidebar} = require("devtools/client/framework/sidebar");
 
-  const toolURL = "data:text/xml;charset=utf8,<?xml version='1.0'?>" +
-                  "<window xmlns='http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul'>" +
-                  "<hbox flex='1'><description flex='1'>foo</description><splitter class='devtools-side-splitter'/>" +
-                  "<tabbox flex='1' id='sidebar' class='devtools-sidebar-tabs'><tabs/><tabpanels flex='1'/></tabbox>" +
-                  "</hbox>" +
-                  "</window>";
-
   const tab1URL = "data:text/html;charset=utf8,<title>1</title><p>1</p>";
   const tab2URL = "data:text/html;charset=utf8,<title>2</title><p>2</p>";
   const tab3URL = "data:text/html;charset=utf8,<title>3</title><p>3</p>";
@@ -26,7 +19,7 @@ function test() {
   let toolDefinition = {
     id: "fakeTool4242",
     visibilityswitch: "devtools.fakeTool4242.enabled",
-    url: toolURL,
+    url: CHROME_URL_ROOT + "browser_toolbox_sidebar_toolURL.xul",
     label: "FAKE TOOL!!!",
     isTargetSupported: () => true,
     build: function (iframeWindow, toolbox) {

devtools/client/framework/test/browser_toolbox_sidebar_events.js

@@ -7,13 +7,6 @@ function test() {
   const Cu = Components.utils;
   const { ToolSidebar } = require("devtools/client/framework/sidebar");
 
-  const toolURL = "data:text/xml;charset=utf8,<?xml version='1.0'?>" +
-                  "<window xmlns='http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul'>" +
-                  "<hbox flex='1'><description flex='1'>foo</description><splitter class='devtools-side-splitter'/>" +
-                  "<tabbox flex='1' id='sidebar' class='devtools-sidebar-tabs'><tabs/><tabpanels flex='1'/></tabbox>" +
-                  "</hbox>" +
-                  "</window>";
-
   const tab1URL = "data:text/html;charset=utf8,<title>1</title><p>1</p>";
 
   let collectedEvents = [];
@@ -21,7 +14,7 @@ function test() {
   let toolDefinition = {
     id: "testTool1072208",
     visibilityswitch: "devtools.testTool1072208.enabled",
-    url: toolURL,
+    url: CHROME_URL_ROOT + "browser_toolbox_sidebar_events.xul",
     label: "Test tool",
     isTargetSupported: () => true,
     build: function (iframeWindow, toolbox) {

devtools/client/framework/test/browser_toolbox_sidebar_events.xul

@@ -0,0 +1,6 @@
+<?xml version='1.0'?>
+<window xmlns='http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul'>
+<hbox flex='1'><description flex='1'>foo</description><splitter class='devtools-side-splitter'/>
+<tabbox flex='1' id='sidebar' class='devtools-sidebar-tabs'><tabs/><tabpanels flex='1'/></tabbox>
+</hbox>
+</window>

devtools/client/framework/test/browser_toolbox_sidebar_existing_tabs.js

@@ -9,18 +9,9 @@
 
 const {ToolSidebar} = require("devtools/client/framework/sidebar");
 
-const testToolURL = "data:text/xml;charset=utf8,<?xml version='1.0'?>" +
-                "<window xmlns='http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul'>" +
-                "<hbox flex='1'><description flex='1'>test tool</description>" +
-                "<splitter class='devtools-side-splitter'/>" +
-                "<tabbox flex='1' id='sidebar' class='devtools-sidebar-tabs'>" +
-                "<tabs><tab id='tab1' label='tab 1'></tab><tab id='tab2' label='tab 2'></tab></tabs>" +
-                "<tabpanels flex='1'><tabpanel id='tabpanel1'>tab 1</tabpanel><tabpanel id='tabpanel2'>tab 2</tabpanel></tabpanels>" +
-                "</tabbox></hbox></window>";
-
 const testToolDefinition = {
   id: "testTool",
-  url: testToolURL,
+  url: CHROME_URL_ROOT + "browser_toolbox_sidebar_existing_tabs.xul",
   label: "Test Tool",
   isTargetSupported: () => true,
   build: (iframeWindow, toolbox) => {

devtools/client/framework/test/browser_toolbox_sidebar_existing_tabs.xul

@@ -0,0 +1,8 @@
+<?xml version='1.0'?>
+<window xmlns='http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul'>
+<hbox flex='1'><description flex='1'>test tool</description>
+<splitter class='devtools-side-splitter'/>
+<tabbox flex='1' id='sidebar' class='devtools-sidebar-tabs'>
+<tabs><tab id='tab1' label='tab 1'></tab><tab id='tab2' label='tab 2'></tab></tabs>
+<tabpanels flex='1'><tabpanel id='tabpanel1'>tab 1</tabpanel><tabpanel id='tabpanel2'>tab 2</tabpanel></tabpanels>
+</tabbox></hbox></window>

devtools/client/framework/test/browser_toolbox_sidebar_toolURL.xul

@@ -0,0 +1,7 @@
+<?xml version='1.0'?>
+<window xmlns='http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul'>
+<hbox flex='1'><description flex='1'>foo</description><splitter class='devtools-side-splitter'/>
+<tabbox flex='1' id='sidebar' class='devtools-sidebar-tabs'><tabs/><tabpanels flex='1'/></tabbox>
+</hbox>
+</window>
+