Bug 1248916 - Ignore mozdebug.requiresEscapedArgs in mochitest, r=ted.mielczarek

Mochitest no longer passes in debug_args in via the commandline. Instead, it passes
them in via a marionette chrome script, meaning they no longer need to be escaped.
Since web-platform-tests still passes debug_args on the commandline, we can't simply
remove the feature entirely. Instead, just make mochitest ignore requiresEscapedArgs.

MozReview-Commit-ID: KBVviGpkvQS
This commit is contained in:
Andrew Halberstadt 2016-02-17 14:33:33 -05:00
parent 20fe44ef03
commit cb72de71d7

View File

@ -1928,12 +1928,7 @@ class MochitestDesktop(MochitestBase):
# TODO: mozrunner should use -foreground at least for mac
# https://bugzilla.mozilla.org/show_bug.cgi?id=916512
args.append('-foreground')
if testUrl:
if debuggerInfo and debuggerInfo.requiresEscapedArgs:
testUrl = testUrl.replace("&", "\\&")
self.start_script_args.append(testUrl)
else:
self.start_script_args.append('about:blank')
self.start_script_args.append(testUrl or 'about:blank')
if detectShutdownLeaks:
shutdownLeaks = ShutdownLeaks(self.log)