Skip to content

Fix #59047 - fix SVG fresh specs links#59066

Merged
jgraham merged 1 commit intomasterfrom
59047-svg2-spec-links-fix
Apr 13, 2026
Merged

Fix #59047 - fix SVG fresh specs links#59066
jgraham merged 1 commit intomasterfrom
59047-svg2-spec-links-fix

Conversation

@karlcow
Copy link
Copy Markdown
Contributor

@karlcow karlcow commented Apr 9, 2026

The domain name doesn't host anymore the fresh version of the SVG WG specifications.
This fixes the links to the current version of the spec and to avoid people double checking prose that would be eventually misleading with recent spec changes.

Copy link
Copy Markdown
Contributor

@schenney-chromium schenney-chromium left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Github's interface for reviewing svg files is not good.

I presume this was automated. Was it?

Regardless, I do not detect any problems. The Checks failures for Chrome and Firefox appear to be pre-existing.

@karlcow
Copy link
Copy Markdown
Contributor Author

karlcow commented Apr 9, 2026

Hi @schenney-chromium

I presume this was automated. Was it?

Search and replace for most of it.
And then checked individually for some of the cases.

but let's see the error for chrome.

25:16.29 INFO |                                 Test                                |                    Subtest                    |            Results            | Messages |
25:16.29 INFO |---------------------------------------------------------------------|-----------------------------------------------|-------------------------------|----------|
25:16.29 INFO | `/svg/coordinate-systems/transform-translate-single-parameter.html` |                                               | **CRASH: 1/10, OK: 9/10**     |          |
25:16.29 INFO | `/svg/coordinate-systems/transform-translate-single-parameter.html` | `Dynamic update of transform; setAttribute()` | **PASS: 9/10, MISSING: 1/10** |          |
25:16.29 INFO 
25:16.29 INFO ::: Running tests in a loop with restarts 10 times : FAIL
25:16.29 INFO :::
25:16.29 ERROR ::: Test verification FAIL
25:16.29 INFO :::
/usr/lib/python3.10/multiprocessing/resource_tracker.py:224: UserWarning: resource_tracker: There appear to be 3 leaked semaphore objects to clean up at shutdown
  warnings.warn('resource_tracker: There appear to be %d '
[taskcluster 2026-04-09 14:33:03.461Z] === Task Finished ===
[taskcluster 2026-04-09 14:33:03.857Z] Unsuccessful task run with exit code: 1 completed in 1665.864 seconds
[V](https://community-tc.services.mozilla.com/tasks/NZPDqANFQ4G_qQCIFgF7SQ)

so this is this file:

<!DOCTYPE HTML>
<link rel="help" href="https://w3c.github.io/svgwg/svg2-draft/coords.html#InterfaceSVGTransform">
<script src="/resources/testharness.js"></script>
<script src="/resources/testharnessreport.js"></script>
<body>

<svg width="200" height="100" xmlns="http://www.w3.org/2000/svg">
  <rect x="10" y="10" width="180" height="80" fill="red" />
</svg>

<script>
test(() => {
const rectElement = document.querySelector("rect");

// Apply single parameter translate - should behave as translate(100, 0)
rectElement.setAttribute("transform", "translate(100)");

assert_equals(rectElement.getAttribute('transform'), "translate(100)");

assert_equals(rectElement.transform.baseVal.getItem(0).matrix.e, 100);
assert_equals(rectElement.transform.baseVal.getItem(0).matrix.f, 0);

}, 'Dynamic update of transform; setAttribute()');
</script>
</body>
</html>

and the only thing changed there is the link

diff --git a/svg/coordinate-systems/transform-translate-single-parameter.html b/svg/coordinate-systems/transform-translate-single-parameter.html
index 880d65e353..26babbc627 100644
--- a/svg/coordinate-systems/transform-translate-single-parameter.html
+++ b/svg/coordinate-systems/transform-translate-single-parameter.html
@@ -1,5 +1,5 @@
 <!DOCTYPE HTML>
-<link rel="help" href="https://svgwg.org/svg2-draft/coords.html#InterfaceSVGTransform">
+<link rel="help" href="https://w3c.github.io/svgwg/svg2-draft/coords.html#InterfaceSVGTransform">
 <script src="/resources/testharness.js"></script>
 <script src="/resources/testharnessreport.js"></script>
 <body>

so yes pre-existing bot failures not my change.

For firefox this is on a different file,

17:21.50 INFO ## Unstable results ##

17:21.50 INFO |                        Test                       |                                 Subtest                                 |          Results           |                               Messages                              |
17:21.50 INFO |---------------------------------------------------|-------------------------------------------------------------------------|----------------------------|---------------------------------------------------------------------|
17:21.50 INFO | `/svg/animations/scripted/onhover-syncbases.html` | `Check if onhover events reset correctly when triggered multiple times` | **FAIL: 1/10, PASS: 9/10** | `assert_equals: expected "rgb(255, 0, 0)" but got "rgb(0, 255, 0)"` |
17:21.50 INFO 
17:21.50 INFO ::: Running tests in a loop with restarts 10 times : FAIL
17:21.50 INFO :::
17:21.50 ERROR ::: Test verification FAIL
17:21.50 INFO :::
[taskcluster 2026-04-09 14:25:59.374Z] === Task Finished ===
[taskcluster 2026-04-09 14:25:59.770Z] Unsuccessful task run with exit code: 1 completed in 1112.085 seconds

which seems a failure on the results, unrelated to my change again. :)

Copy link
Copy Markdown
Contributor

@schenney-chromium schenney-chromium left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Chrome still report 9/10 passing 1/10 failing but the reported failure type is different. I find this rather odd but I agree seems to be unrelated to anything you've changed.

@karlcow karlcow enabled auto-merge (squash) April 10, 2026 23:53
@karlcow
Copy link
Copy Markdown
Contributor Author

karlcow commented Apr 13, 2026

How do we merge this?
or fix before merging?

@community-tc-integration
Copy link
Copy Markdown

Uh oh! Looks like an error!

Client ID static/taskcluster/github does not have sufficient scopes and is missing the following scopes:

{
  "AnyOf": [
    "queue:rerun-task:taskcluster-github/EywtgiyfSyCoyO6HmGfg5Q/NZPDqANFQ4G_qQCIFgF7SQ",
    "queue:rerun-task-in-project:none",
    {
      "AllOf": [
        "queue:rerun-task",
        "assume:scheduler-id:taskcluster-github/EywtgiyfSyCoyO6HmGfg5Q"
      ]
    }
  ]
}

This request requires the client to satisfy the following scope expression:

{
  "AnyOf": [
    "queue:rerun-task:taskcluster-github/EywtgiyfSyCoyO6HmGfg5Q/NZPDqANFQ4G_qQCIFgF7SQ",
    "queue:rerun-task-in-project:none",
    {
      "AllOf": [
        "queue:rerun-task",
        "assume:scheduler-id:taskcluster-github/EywtgiyfSyCoyO6HmGfg5Q"
      ]
    }
  ]
}

  • method: rerunTask
  • errorCode: InsufficientScopes
  • statusCode: 403
  • time: 2026-04-13T00:04:57.281Z

@community-tc-integration
Copy link
Copy Markdown

Uh oh! Looks like an error!

Client ID static/taskcluster/github does not have sufficient scopes and is missing the following scopes:

{
  "AnyOf": [
    "queue:rerun-task:taskcluster-github/EywtgiyfSyCoyO6HmGfg5Q/IDvguFXtSKOup8aPiKWe1A",
    "queue:rerun-task-in-project:none",
    {
      "AllOf": [
        "queue:rerun-task",
        "assume:scheduler-id:taskcluster-github/EywtgiyfSyCoyO6HmGfg5Q"
      ]
    }
  ]
}

This request requires the client to satisfy the following scope expression:

{
  "AnyOf": [
    "queue:rerun-task:taskcluster-github/EywtgiyfSyCoyO6HmGfg5Q/IDvguFXtSKOup8aPiKWe1A",
    "queue:rerun-task-in-project:none",
    {
      "AllOf": [
        "queue:rerun-task",
        "assume:scheduler-id:taskcluster-github/EywtgiyfSyCoyO6HmGfg5Q"
      ]
    }
  ]
}

  • method: rerunTask
  • errorCode: InsufficientScopes
  • statusCode: 403
  • time: 2026-04-13T00:05:06.354Z

@karlcow karlcow disabled auto-merge April 13, 2026 01:35
@jgraham jgraham merged commit bc058f3 into master Apr 13, 2026
23 of 26 checks passed
@jgraham jgraham deleted the 59047-svg2-spec-links-fix branch April 13, 2026 08:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants