KEMBAR78
Add WPTs for negotiating H265 with minimal level-id in SDP answers. by chromium-wpt-export-bot · Pull Request #49990 · web-platform-tests/wpt · GitHub
Skip to content

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented Jan 9, 2025

Add test coverage for w3c/webrtc-pc#3023 which
unblocks merging the PR (it has "editors can integrate"). Specifically
it should be possible to negotiate H265 even if the level-id is
downgraded in the SDP answer (not throwing on level-id mismatch).

  • H265 is behind flags so a virtual test suite is added.
  • If bot lacks HW capabilities to do H265, the tests will
    PRECONDITION_FAILED. To avoid reverting tests we will allow the bots
    to both pass and fail, I filed crbug.com/388299759.

Some of the tests FAIL even when they run: passing these tests will be
covered by crbug.com/381407888.

Bug: chromium:381407888, webrtc:41480904, chromium:388299759
Change-Id: Ie017560b4310418a4c9ed854dcb09136b9b446e7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6148192
Reviewed-by: David Baron <dbaron@chromium.org>
Commit-Queue: Henrik Boström <hbos@chromium.org>
Reviewed-by: Evan Shrubsole <eshr@google.com>
Cr-Commit-Position: refs/heads/main@{#1404052}

Add test coverage for w3c/webrtc-pc#3023 which
unblocks merging the PR (it has "editors can integrate"). Specifically
it should be possible to negotiate H265 even if the level-id is
downgraded in the SDP answer (not throwing on level-id mismatch).

- H265 is behind flags so a virtual test suite is added.
- If bot lacks HW capabilities to do H265, the tests will
  PRECONDITION_FAILED. To avoid reverting tests we will allow the bots
  to both pass and fail, I filed crbug.com/388299759.

Some of the tests FAIL even when they run: passing these tests will be
covered by crbug.com/381407888.

Bug: chromium:381407888, webrtc:41480904, chromium:388299759
Change-Id: Ie017560b4310418a4c9ed854dcb09136b9b446e7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6148192
Reviewed-by: David Baron <dbaron@chromium.org>
Commit-Queue: Henrik Boström <hbos@chromium.org>
Reviewed-by: Evan Shrubsole <eshr@google.com>
Cr-Commit-Position: refs/heads/main@{#1404052}
Copy link
Collaborator

@wpt-pr-bot wpt-pr-bot left a comment

Choose a reason for hiding this comment

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

The review process for this patch is being conducted in the Chromium project.

@chromium-wpt-export-bot chromium-wpt-export-bot merged commit 545a2ed into master Jan 9, 2025
21 checks passed
@chromium-wpt-export-bot chromium-wpt-export-bot deleted the chromium-export-cl-6148192 branch January 9, 2025 10:05
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.

3 participants