[webkit-changes] [WebKit/WebKit] 7e8331: [MotionMark] Live version of 4 tests just show a 4...

Myles C. Maxfield noreply at github.com
Thu Mar 30 17:43:12 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 7e8331b4cad9e69af27575d0ada29e950e33f059
      https://github.com/WebKit/WebKit/commit/7e8331b4cad9e69af27575d0ada29e950e33f059
  Author: Myles C. Maxfield <mmaxfield at apple.com>
  Date:   2023-03-30 (Thu, 30 Mar 2023)

  Changed paths:
    M Websites/browserbench.org/MotionMark1.2/resources/debug-runner/tests.js
    M Websites/browserbench.org/MotionMark1.2/tests/bouncing-particles/resources/bouncing-canvas-images.js
    M Websites/browserbench.org/MotionMark1.2/tests/bouncing-particles/resources/bouncing-svg-images.js

  Log Message:
  -----------
  [MotionMark] Live version of 4 tests just show a 403 Access Denied page
https://bugs.webkit.org/show_bug.cgi?id=254782
rdar://107208247

Reviewed by Alexey Proskuryakov.

Apparently, the firewall that the live MotionMark site is behind doesn't like it when paths that include
the "../" string are part of query parameters in URLs. Luckily, MotionMark is already set up to do the right
thing when the query parameter is omitted, so this patch just deletes those query parameters and uses the
test-specific logic to fall back to the correct string anyway.

* Websites/browserbench.org/MotionMark1.2/resources/debug-runner/tests.js:
* Websites/browserbench.org/MotionMark1.2/tests/bouncing-particles/resources/bouncing-canvas-images.js:
(BouncingCanvasParticlesStage.call.initialize):
* Websites/browserbench.org/MotionMark1.2/tests/bouncing-particles/resources/bouncing-svg-images.js:
(BouncingSvgParticlesStage.call.initialize):

Canonical link: https://commits.webkit.org/262382@main




More information about the webkit-changes mailing list