[Webkit-unassigned] [Bug 260109] New: JetStream2/JetStream3 should report results in a better way

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Aug 11 17:35:48 PDT 2023


            Bug ID: 260109
           Summary: JetStream2/JetStream3 should report results in a
                    better way
           Product: WebKit
           Version: WebKit Nightly Build
          Hardware: Unspecified
                OS: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: Tools / Tests
          Assignee: webkit-unassigned at lists.webkit.org
          Reporter: hysu at apple.com

The JetStream2 and JetStream3 harnesses currently report results using `fetch`. Because the header is manually constructed, and there is a typo in both harnesses ("headers" -> "heeaders").
JS2: https://github.com/WebKit/WebKit/blob/ec6f02c2f3242338888639458e4b258cda0e4b77/PerformanceTests/JetStream2/JetStreamDriver.js#L514
JS3: https://github.com/WebKit/WebKit/blob/ec6f02c2f3242338888639458e4b258cda0e4b77/PerformanceTests/JetStream3/JetStreamDriver.js#L514

This isn't an issue at the moment, since somehow the twistd server that run-benchmarks is using is able to work around the typo, but if we ever switch to a different HTTP server implementation (https://bugs.webkit.org/show_bug.cgi?id=259835), this will probably come back to bite us. We should fix this - or even better, switch to using XHR like we patch Speedometer and MotionMark to do: https://github.com/WebKit/WebKit/blob/ec6f02c2f3242338888639458e4b258cda0e4b77/Tools/Scripts/webkitpy/benchmark_runner/data/patches/webserver/Speedometer2.patch#L60

You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20230812/7abef4c3/attachment.htm>

More information about the webkit-unassigned mailing list