Magic-Mirror/gecko-dev
1
0
Fork 0
mirror of https://github.com/mozilla/gecko-dev.git synced 2024-11-25 05:41:12 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Bug 1523435 - Do not open two tabs for talos startup tests if the profiler is enabled r=perftest-reviewers,sparky

Browse Source 
It looks like this is a regression from Bug 1121571. This was opening two tabs
instead of one because we are already adding the `url` to the command line
arguments below:
https://searchfox.org/mozilla-central/rev/40d51bef58b8e901d6ab4b60dd280f372a0e417d/testing/talos/talos/utils.py#165-166

Since we have this addition below, we don't have to do it again inside of this
if branch.

Differential Revision: https://phabricator.services.mozilla.com/D192067
This commit is contained in:
Nazım Can Altınova 2023-10-30 10:29:53 +00:00
parent dbb3940438
commit 932fd143f7
1 changed files with 0 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
testing/talos/talos/utils.py
View File

@ -159,7 +159,6 @@ def GenerateBrowserCommandLine(
url += "&" + urllib.parse.urlencode(profiling_info)
else:
url += "?" + urllib.parse.urlencode(profiling_info)
command_args.extend(url.split(" "))
# if there's a url i.e. startup test / non-manifest test, add it to the cmd line args
if url is not None: