Bug 1243963 - e10s fixes for browser_console_error_source_click.js;r=linclark draft
authorBrian Grinstead <bgrinstead@mozilla.com>
Wed, 03 Feb 2016 17:34:48 -0800
changeset 328739 5d2f732d5931a6add6795d2b86ccb5ef69bd38d2
parent 328738 7717f64ecce91fa11e4d430e5b7a320f9f4ab424
child 513855 41972847eb55f40a3533742afac90fd5cafc8959
push id10411
push userbgrinstead@mozilla.com
push dateThu, 04 Feb 2016 01:34:54 +0000
reviewerslinclark
bugs1243963
milestone47.0a1
Bug 1243963 - e10s fixes for browser_console_error_source_click.js;r=linclark
devtools/client/webconsole/test/browser.ini
devtools/client/webconsole/test/browser_console_error_source_click.js
--- a/devtools/client/webconsole/test/browser.ini
+++ b/devtools/client/webconsole/test/browser.ini
@@ -158,17 +158,16 @@ skip-if = e10s # Bug 1042253 - webconsol
 [browser_console_clear_on_reload.js]
 [browser_console_click_focus.js]
 [browser_console_consolejsm_output.js]
 [browser_console_copy_command.js]
 [browser_console_dead_objects.js]
 skip-if = e10s # Bug 1042253 - webconsole tests disabled with e10s
 [browser_console_copy_entire_message_context_menu.js]
 [browser_console_error_source_click.js]
-skip-if = e10s # Bug 1042253 - webconsole e10s tests
 [browser_console_filters.js]
 [browser_console_iframe_messages.js]
 [browser_console_keyboard_accessibility.js]
 [browser_console_log_inspectable_object.js]
 [browser_console_native_getters.js]
 [browser_console_navigation_marker.js]
 [browser_console_netlogging.js]
 [browser_console_nsiconsolemessage.js]
--- a/devtools/client/webconsole/test/browser_console_error_source_click.js
+++ b/devtools/client/webconsole/test/browser_console_error_source_click.js
@@ -17,22 +17,25 @@ function test() {
   loadTab(TEST_URI).then(() => {
     HUDService.toggleBrowserConsole().then(browserConsoleOpened);
   });
 
   function browserConsoleOpened(hudConsole) {
     hud = hudConsole;
     ok(hud, "browser console opened");
 
-    let button = content.document.querySelector("button");
-    ok(button, "button element found");
+    // On e10s, the exception is triggered in child process
+    // and is ignored by test harness
+    if (!Services.appinfo.browserTabsRemoteAutostart) {
+      expectUncaughtException();
+    }
 
     info("generate exception and wait for the message");
-    executeSoon(() => {
-      expectUncaughtException();
+    ContentTask.spawn(gBrowser.selectedBrowser, {}, function*() {
+      let button = content.document.querySelector("button");
       button.click();
     });
 
     waitForMessages({
       webconsole: hud,
       messages: [
         {
           text: "ReferenceError: foobar is not defined",