1314007 - marionette args ends up with a long list of urls as we keep appending for each dir

Status: NEW

(Testing :: Mochitest, defect)

Description

[Joel Maher ( :jmaher ) (UTC -8)]

for example:
[task 2016-10-30T17:41:35.423023Z] 17:41:35     INFO - 1477849295410	Marionette	TRACE	conn2 -> [0,12,"executeScript",{"scriptTimeout":null,"newSandbox":true,"args":["mochitest","http://mochi.test:8888/tests?autorun=1&closeWhenDone=1&consoleLevel=INFO&manifestFile=tests.json&dumpOutputDirectory=%2Ftmp","mochitest","http://mochi.test:8888/tests?autorun=1&closeWhenDone=1&consoleLevel=INFO&manifestFile=tests.json&dumpOutputDirectory=%2Ftmp","mochitest","http://mochi.test:8888/tests?autorun=1&closeWhenDone=1&consoleLevel=INFO&manifestFile=tests.json&dumpOutputDirectory=%2Ftmp","mochitest","http://mochi.test:8888/tests?autorun=1&closeWhenDone=1&consoleLevel=INFO&manifestFile=tests.json&dumpOutputDirectory=%2Ftmp","mochitest","http://mochi.test:8888/tests?autorun=1&closeWhenDone=1&consoleLevel=INFO&manifestFile=tests.json&dumpOutputDirectory=%2Ftmp","mochitest","http://mochi.test:8888/tests?autorun=1&closeWhenDone=1&consoleLevel=INFO&manifestFile=tests.json&dumpOutputDirectory=%2Ftmp","mochitest","http://mochi.test:8888/tests?autorun=1&closeWhenDone=1&consoleLevel=INFO&manifestFile=tests.json&dumpOutputDirectory=%2Ftmp","mochitest","http://mochi.test:8888/tests?autorun=1&closeWhenDone=1&consoleLevel=INFO&manifestFile=tests.json&dumpOutputDirectory=%2Ftmp","mochitest","http://mochi.test:8888/tests?autorun=1&closeWhenDone=1&consoleLevel=INFO&manifestFile=tests.json&dumpOutputDirectory=%2Ftmp"],"filename":"runtests.py","script":"/* This Source Code Form is subject to the terms of the Mozilla Public\n * License, v. 2.0. If a copy of the MPL was not distributed with this\n * file, You can obtain one at http://mozilla.org/MPL/2.0/. */\n\nconst flavor  = __webDriverArguments[0]\nconst url = __webDriverArguments[1]\n\nlet wm = Cc[\"@mozilla.org/appshell/window-mediator;1\"]\n          .getService(Ci.nsIWindowMediator);\nlet win = wm.getMostRecentWindow(\"navigator:browser\");\n\n// mochikit's bootstrap.js has set up a listener for this event. It's\n// used so bootstrap.js knows which flavor and url to load.\nlet ev = new CustomEvent('mochitest-load', {'detail': [flavor, url]});\nwin.dispatchEvent(ev);\n","sandbox":"default","line":1557}]


we initialize start_script_args, but we never reset it, we keep appending to it:
https://dxr.mozilla.org/mozilla-central/source/testing/mochitest/runtests.py#1947