Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add test that "maintain-framerate" does what it says. #30173

Merged
merged 1 commit into from
Oct 3, 2023

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented Aug 25, 2021

Adds a test that measures the bandwidth, cuts the bandwidth in half,
and then measures that framerate is maintained, but resolution is not.

Also adds more entropy to the "getNoiseStream()" video signal; the old
version generated a video stream of 8 Kbits/second in 640x480.

Bug: none
Change-Id: Iefe0a44b9686f1bb4bdd13ab63922c3d460c775e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3117645
Reviewed-by: Philipp Hancke <phancke@microsoft.com>
Commit-Queue: Harald Alvestrand <hta@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1204584}

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 force-pushed the chromium-export-cl-3117645 branch 5 times, most recently from faafc38 to 4258493 Compare August 27, 2021 09:50
@chromium-wpt-export-bot chromium-wpt-export-bot force-pushed the chromium-export-cl-3117645 branch 4 times, most recently from 84866a0 to c172ad9 Compare October 2, 2023 16:15
Adds a test that measures the bandwidth, cuts the bandwidth in half,
and then measures that framerate is maintained, but resolution is not.

Also adds more entropy to the "getNoiseStream()" video signal; the old
version generated a video stream of 8 Kbits/second in 640x480.

Bug: none
Change-Id: Iefe0a44b9686f1bb4bdd13ab63922c3d460c775e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3117645
Reviewed-by: Philipp Hancke <phancke@microsoft.com>
Commit-Queue: Harald Alvestrand <hta@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1204584}
@chromium-wpt-export-bot chromium-wpt-export-bot merged commit 1af7ffe into master Oct 3, 2023
21 checks passed
@chromium-wpt-export-bot chromium-wpt-export-bot deleted the chromium-export-cl-3117645 branch October 3, 2023 13:12
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