From 00136cf6d818b9bc6e4d77504e3ce1ed3d23d611 Mon Sep 17 00:00:00 2001 From: Vojta Jina Date: Mon, 25 Nov 2013 15:08:53 -0800 Subject: [PATCH] fix: do not execute already executing browsers This was causing iframe reloads on IE8/IE9. These browsers use xhr-polling and socket.io does re-connects pretty often. We need to start execution only when the browser connects for the first time, no other re-connects. --- lib/server.js | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/lib/server.js b/lib/server.js index 789a57fc7..b558b5227 100644 --- a/lib/server.js +++ b/lib/server.js @@ -105,15 +105,15 @@ var start = function(injector, config, launcher, globalEmitter, preprocess, file }]).instantiate(browser.Browser); newBrowser.init(); + + // execute in this browser immediately + if (config.singleRun) { + newBrowser.execute(config.client); + singleRunBrowsers.add(newBrowser); + } } replySocketEvents(); - - // execute in this browser immediately - if (config.singleRun) { - newBrowser.execute(config.client); - singleRunBrowsers.add(newBrowser); - } }); });