Merge branch 'master' into crbug-1093539

web-platform-tests · Apr 28, 2023 · 1255eb6 · 1255eb6
2 parents c5ed554 + b371b5b
commit 1255eb6
Show file tree

Hide file tree

Showing 27 changed files with 4,582 additions and 917 deletions.
diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md
@@ -1,4 +1,4 @@
 <!--  Thanks for sending a pull request!  Here are some tips for you:
 
 1. If this is your first time, please read our README.md: https://github.com/web-platform-tests/wpt-metadata/blob/master/README.md 
-2. Please label this pull request `do not merge yet` upon creation because the auto-merge feature is enabled in this repository. If this pull request is finished, feel free to assign foolip/kyleju as reviewers, or we will review and merge them automatically.
+2. Please label this pull request `do not merge yet` upon creation because the auto-merge feature is enabled in this repository. Once your PR is created, do not enable auto-merge on it. If this pull request is finished, feel free to assign foolip/kyleju as reviewers, or we will review and merge them automatically.
diff --git a/.github/workflows/auto-approve.yml b/.github/workflows/auto-approve.yml
@@ -1,10 +1,55 @@
 name: Auto approve
-on: pull_request
+on:
+  workflow_run:
+    workflows: ["test"]
+    types:
+      - completed
 
 jobs:
-  build:
+  approve:
+    name: "Approve PR"
     runs-on: ubuntu-20.04
+    if: >
+      github.event.workflow_run.event == 'pull_request' &&
+      github.event.workflow_run.conclusion == 'success'
+    permissions:
+      pull-requests: write
     steps:
-    - uses: hmarr/auto-approve-action@v2.0.0
+    # Currently there is no way to get the pull request number from the event payload in a workflow_run event.
+    # We save it in the test workflow
+    # These first steps extract it.
+    # More information about it:
+    # https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#using-data-from-the-triggering-workflow
+    # https://securitylab.github.com/research/github-actions-preventing-pwn-requests/
+    - name: 'Download PR number'
+      uses: actions/github-script@v3.1.0
       with:
-        github-token: "${{ secrets.GITHUB_TOKEN }}"
+        script: |
+          console.log("Starting to list artifacts");
+          var artifacts = await github.actions.listWorkflowRunArtifacts({
+              owner: context.repo.owner,
+              repo: context.repo.repo,
+              run_id: context.payload.workflow_run.id,
+          });
+          console.log("Filtering artifacts");
+          var matchArtifact = artifacts.data.artifacts.filter((artifact) => {
+            return artifact.name == "pr_number"
+          })[0];
+          console.log("Downloading artifacts");
+          var download = await github.actions.downloadArtifact({
+              owner: context.repo.owner,
+              repo: context.repo.repo,
+              artifact_id: matchArtifact.id,
+              archive_format: 'zip',
+          });
+          console.log("Writing artifacts to disk");
+          var fs = require('fs');
+          fs.writeFileSync('${{github.workspace}}/pr.zip', Buffer.from(download.data));
+    - run: unzip pr.zip
+    - name: Get the PR value and set it in the environment
+      run: |
+        echo "PR_NUMBER=$(<pr_number.txt)" >> "$GITHUB_ENV"
+        echo "Set PR_NUMBER to $PR_NUMBER"
+    - uses: hmarr/auto-approve-action@v3
+      with:
+        pull-request-number: ${{ env.PR_NUMBER }}
diff --git a/.github/workflows/automerge.yml b/.github/workflows/automerge.yml
@@ -26,7 +26,7 @@ jobs:
         # https://github.com/pascalgn/automerge-action#configuration
         env:
           GITHUB_TOKEN: "${{ secrets.GITHUB_TOKEN }}"
-          MERGE_LABELS: "!do not merge yet"
+          MERGE_LABELS: "!do not merge yet,!interop"
           MERGE_FORKS: false
           MERGE_METHOD: "rebase"
           MERGE_RETRIES: "3"

diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -14,3 +14,19 @@ jobs:
     - uses: actions/checkout@v2
     - run: go get -v -t -d ./...
     - run: go test ./...
+  upload:
+    if: ${{ github.event_name == 'pull_request' }}
+    needs: [test]
+    runs-on: ubuntu-latest
+    name: "Save PR number for Auto Approve workflow_run"
+    steps:
+      - name: Save PR number
+        env:
+          PR_NUMBER: ${{ github.event.number }}
+        run: |
+          mkdir -p ./pr
+          echo $PR_NUMBER > ./pr/pr_number.txt
+      - uses: actions/upload-artifact@v3
+        with:
+          name: pr_number
+          path: pr/
diff --git a/.github/workflows/update_wpt_manifest.yml b/.github/workflows/update_wpt_manifest.yml
@@ -4,7 +4,9 @@ on:
     - cron: '0 13 * * *'
   workflow_dispatch:
 jobs:
-  update-wpt-manifes:
+  update-wpt-manifest:
+    outputs:
+      prURL: ${{ steps.cpr.outputs.pull-request-url }}
     runs-on: ubuntu-20.04
     steps:
     - uses: actions/checkout@v2
@@ -20,13 +22,32 @@ jobs:
       run: ./main
 
     - name: Create pull request
-      uses: peter-evans/create-pull-request@v3
+      id: cpr
+      uses: peter-evans/create-pull-request@v5
       with:
-        token: ${{ secrets.GITHUB_TOKEN }}
-        author: wpt-pr-bot <wpt-pr-bot@users.noreply.github.com>
+        token: ${{ secrets.WPTMETADATA_BOT_USER_PAT }}
         title: "Update latest WPT Manifest for go_test"
         commit-message: "Run curl -s -S https://wpt.fyi/api/manifest?sha=latest >> go_test/MANIFEST.json"
         body: |
           This automated pull request updates the MANIFEST.json file to the latest WPT manifest daily. GitHub checks run against this manifest. If checks fail in this PR, the failing metadata should be either renamed or deleted according to the WPT repository.
         branch: actions/wpt-manifest
-        delete-branch: true
+        delete-branch: true
+        push-to-fork: wptmetadata-bot/wpt-metadata
+  # Create a separate job for only the steps that need write permissions.
+  enable-auto-merge:
+    runs-on: ubuntu-latest
+    needs: update-wpt-manifest
+    # Need both permissions to enable auto merge.
+    permissions:
+      pull-requests: write
+      contents: write
+    steps:
+    # Could use https://github.com/peter-evans/enable-pull-request-automerge but it recommends using the CLI now
+    # More about the CLI options. We need --auto so that the PR will automatically merge after:
+    # - One review
+    # - The test action is successful.
+    # https://cli.github.com/manual/gh_pr_merge
+    - name: Enable Pull Request Automerge
+      run: gh pr merge --squash --auto "${{ needs.update-wpt-manifest.outputs.prURL }}"
+      env:
+        GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
diff --git a/accessibility/crashtests/META.yml b/accessibility/crashtests/META.yml
@@ -0,0 +1,8 @@
+links:
+    - product: safari
+      url: https://bugs.webkit.org/show_bug.cgi?id=256006
+      results:
+        - test: computed-accessible-text-node.html
+        - test: computed-accessible-child-of-pseudo-element.html
+        - test: aom-in-destroyed-iframe.html
+        - test: computed-node-checked.html
diff --git a/css/css-animations/parsing/META.yml b/css/css-animations/parsing/META.yml
@@ -466,3 +466,100 @@ links:
       value
     test: animation-range-start-valid.html
   url: https://bugzilla.mozilla.org/show_bug.cgi?id=1824412
+- product: firefox
+  results:
+  - subtest: Property animation-range-end value 'normal'
+    test: animation-range-end-computed.html
+  - subtest: Property animation-range-end value 'COVER 100%'
+    test: animation-range-end-computed.html
+  - subtest: Property animation-range-end value 'cover'
+    test: animation-range-end-computed.html
+  - subtest: Property animation-range-end value 'contain'
+    test: animation-range-end-computed.html
+  - subtest: Property animation-range-end value 'entry'
+    test: animation-range-end-computed.html
+  - subtest: Property animation-range-end value 'exit'
+    test: animation-range-end-computed.html
+  - subtest: e.style['animation-range-end'] = "normal" should set the property value
+    test: animation-range-end-valid.html
+  - subtest: e.style['animation-range-end'] = "cover" should set the property value
+    test: animation-range-end-valid.html
+  - subtest: e.style['animation-range-end'] = "contain" should set the property value
+    test: animation-range-end-valid.html
+  - subtest: e.style['animation-range-end'] = "entry" should set the property value
+    test: animation-range-end-valid.html
+  - subtest: e.style['animation-range-end'] = "exit" should set the property value
+    test: animation-range-end-valid.html
+  - subtest: e.style['animation-range'] = "normal" should set the property value
+    test: animation-range-shorthand.html
+  - subtest: e.style['animation-range'] = "normal normal" should set the property
+      value
+    test: animation-range-shorthand.html
+  - subtest: Property animation-range value 'normal'
+    test: animation-range-shorthand.html
+  - subtest: Property animation-range value 'normal normal'
+    test: animation-range-shorthand.html
+  - subtest: e.style['animation-range'] = "normal" should set animation-range-end
+    test: animation-range-shorthand.html
+  - subtest: e.style['animation-range'] = "normal" should set animation-range-start
+    test: animation-range-shorthand.html
+  - subtest: e.style['animation-range'] = "normal" should not set unrelated longhands
+    test: animation-range-shorthand.html
+  - subtest: e.style['animation-range'] = "normal normal" should set animation-range-end
+    test: animation-range-shorthand.html
+  - subtest: e.style['animation-range'] = "normal normal" should set animation-range-start
+    test: animation-range-shorthand.html
+  - subtest: e.style['animation-range'] = "normal normal" should not set unrelated
+      longhands
+    test: animation-range-shorthand.html
+  - subtest: e.style['animation-range'] = "normal entry 100%" should set animation-range-end
+    test: animation-range-shorthand.html
+  - subtest: e.style['animation-range'] = "normal entry 100%" should set animation-range-start
+    test: animation-range-shorthand.html
+  - subtest: e.style['animation-range'] = "normal entry 100%" should not set unrelated
+      longhands
+    test: animation-range-shorthand.html
+  - subtest: e.style['animation-range'] = "normal entry 10%" should set animation-range-end
+    test: animation-range-shorthand.html
+  - subtest: e.style['animation-range'] = "normal entry 10%" should set animation-range-start
+    test: animation-range-shorthand.html
+  - subtest: e.style['animation-range'] = "normal entry 10%" should not set unrelated
+      longhands
+    test: animation-range-shorthand.html
+  - subtest: e.style['animation-range'] = "entry, exit" should set animation-range-end
+    test: animation-range-shorthand.html
+  - subtest: e.style['animation-range'] = "entry, exit" should set animation-range-start
+    test: animation-range-shorthand.html
+  - subtest: e.style['animation-range'] = "entry 0%, exit" should set animation-range-end
+    test: animation-range-shorthand.html
+  - subtest: e.style['animation-range'] = "entry 0%, exit" should set animation-range-start
+    test: animation-range-shorthand.html
+  - subtest: e.style['animation-range'] = "exit calc(10% + 50px)" should set animation-range-end
+    test: animation-range-shorthand.html
+  - subtest: Property animation-range-start value 'normal'
+    test: animation-range-start-computed.html
+  - subtest: Property animation-range-start value 'exit 1%, cover 2%, contain 0%'
+    test: animation-range-start-computed.html
+  - subtest: Property animation-range-start value 'cover'
+    test: animation-range-start-computed.html
+  - subtest: Property animation-range-start value 'contain'
+    test: animation-range-start-computed.html
+  - subtest: Property animation-range-start value 'entry'
+    test: animation-range-start-computed.html
+  - subtest: Property animation-range-start value 'exit'
+    test: animation-range-start-computed.html
+  - subtest: e.style['animation-range-start'] = "normal" should set the property value
+    test: animation-range-start-valid.html
+  - subtest: e.style['animation-range-start'] = "exit 1%, cover 2%, contain 0%" should
+      set the property value
+    test: animation-range-start-valid.html
+  - subtest: e.style['animation-range-start'] = "cover" should set the property value
+    test: animation-range-start-valid.html
+  - subtest: e.style['animation-range-start'] = "contain" should set the property
+      value
+    test: animation-range-start-valid.html
+  - subtest: e.style['animation-range-start'] = "entry" should set the property value
+    test: animation-range-start-valid.html
+  - subtest: e.style['animation-range-start'] = "exit" should set the property value
+    test: animation-range-start-valid.html
+  url: https://bugzilla.mozilla.org/show_bug.cgi?id=1829320
diff --git a/css/css-backgrounds/META.yml b/css/css-backgrounds/META.yml
@@ -35,14 +35,6 @@ links:
         - test: background-repeat-space-6.html
         - test: border-image-repeat-round-2.html
         - test: border-image-repeat-round-1.html
-    - product: chrome
-      url: https://bugs.chromium.org/p/chromium/issues/detail?id=767352
-      results:
-        - test: background-attachment-fixed-inside-transform-1.html
-        - test: border-image-width-005.xht
-        - test: border-image-width-006.xht
-        - test: border-image-width-007.xht
-        - test: border-image-width-008.html
     - product: chrome
       url: https://bugs.chromium.org/p/chromium/issues/detail?id=1184497
       results:

diff --git a/css/css-backgrounds/animations/META.yml b/css/css-backgrounds/animations/META.yml
@@ -534,9 +534,8 @@ links:
         - test: box-shadow-composition.html
           subtest: 'Compositing: property <box-shadow> underlying [rgb(10, 20, 30) 1px 2px 3px 4px, rgb(20, 40, 60) 2px 4px 6px 8px, rgb(40, 80, 120) 4px 8px 12px 16px] from add [rgb(100, 100, 100) 10px 20px 30px 40px, rgb(200, 200, 200) 20px 40px 60px 80px] to replace [rgb(200, 200, 200) 20px 40px 60px 80px] at (1) should be [rgb(200, 200, 200) 20px 40px 60px 80px]'
     - product: chrome
-      url: https://bugs.chromium.org/p/chromium/issues/detail?id=900581
+      url: https://bugs.chromium.org/p/chromium/issues/detail?id=1419977
       results:
-        - test: box-shadow-interpolation.html
         - test: border-image-source-interpolation.html
     - product: firefox
       url: https://bugzilla.mozilla.org/show_bug.cgi?id=1697383

diff --git a/css/css-box/margin-trim/computed-margin-values/META.yml b/css/css-box/margin-trim/computed-margin-values/META.yml
@@ -35,3 +35,8 @@ links:
   - subtest: flexbox > item 2
     test: flexbox-row-multi-line-block.html
   url: https://bugzilla.mozilla.org/show_bug.cgi?id=1827010
+- product: firefox
+  results:
+  - subtest: grid > item 3
+    test: grid-inline-start.html
+  url: https://bugzilla.mozilla.org/show_bug.cgi?id=1828539
diff --git a/css/css-contain/container-queries/META.yml b/css/css-contain/container-queries/META.yml
@@ -688,7 +688,6 @@ links:
         - test: font-relative-units.html
         - test: container-units-in-at-container-dynamic.html
         - test: grid-container.html
-        - test: inner-first-line-non-matching.html
         - test: container-size-invalidation-after-load.html
         - test: container-type-invalidation.html
         - test: container-computed.html
@@ -718,7 +717,6 @@ links:
         - test: chrome-legacy-skip-recalc.html
         - test: calc-evaluation.html
         - test: container-for-cue.html
-        - test: container-units-typed-om.html
         - test: fieldset-legend-change.html
         - test: font-relative-calc-dynamic.html
         - test: font-relative-units-dynamic.html
@@ -748,7 +746,6 @@ links:
         - test: top-layer-dialog-backdrop.html
         - test: table-inside-container-changing-display.html
         - test: container-inside-multicol-with-table.html
-        - test: container-units-computational-independence.html
         - test: sibling-layout-dependency.html
         - test: transition-style-change-event-002.html
         - test: container-name-parsing.html

diff --git a/css/css-contain/content-visibility/META.yml b/css/css-contain/content-visibility/META.yml
@@ -290,3 +290,8 @@ links:
       results:
         - test: content-visibility-with-top-layer-006.html
           subtest: 'CSS Content Visibility: offscreen c-v auto content is relevant when in top layer'
+    - product: chrome
+      url: https://crbug.com/1395394
+      results:
+        - test: content-visibility-077.html
+          subtest: Content-visibility is not animatable