Bug 1676499 - Remove application name from benchmark perfherder extraOptions. r=perftest-reviewers,kimberlythegeek

Differential Revision: https://phabricator.services.mozilla.com/D98359

testing/mozharness/external_tools/performance-artifact-schema.json

@@ -123,7 +123,7 @@
                     "description": "Similar to extraOptions, except it does not break existing performance data series",
                     "items": {
                         "type": "string",
-                        "pattern": "^[a-zA-Z0-9]{1,24}$"
+                        "pattern": "^[a-zA-Z0-9-]{1,24}$"
                     },
                     "uniqueItems": true,
                     "maxItems": 14

testing/raptor/raptor/results.py

@@ -772,9 +772,6 @@ class BrowsertimeResultsHandler(PerftestResultsHandler):
                     new_result = _new_standard_result(
                         new_result, subtest_unit=test.get("subtest_unit", "ms")
                     )
-                    # XXX Is this still needed?
-                    if self.app != "firefox":
-                        new_result["extra_options"].append(self.app)
 
                     LOG.info("parsed new benchmark result: %s" % str(new_result))
                     return new_result