镜像站点 · 本页由第三方 GitHub 只读镜像提供,非 GitHub 官方站点,不接受任何登录或凭据输入。前往 github.com
Skip to content

Reset canvas size on determineCanvasSize call - #126

Open
JoelCrumley wants to merge 1 commit into
WebKit:tentative-1-4-testsfrom
JoelCrumley:reset-canvas-sizes
Open

JoelCrumley wants to merge 1 commit into
WebKit:tentative-1-4-testsfrom
JoelCrumley:reset-canvas-sizes

Conversation

@JoelCrumley

Copy link
Copy Markdown

Resets the canvas size when BenchmarkController#determineCanvasSize is called.

This change prevents old canvas sizes taking priority when determineCanvasSize is called multiple times. determineCanvasSize is called multiple times on the /developer.html page:

  • First on page load inside the DebugBenchmarkController#startBenchmarkImmediatelyIfEncoded function.
  • Second on pressing the Run benchmark button.

…s called.

This change prevents old canvas sizes taking priority when determineCanvasSize is called multiple times.
determineCanvasSize is called multiple times on the /developer.html page:
  - First on page load inside the DebugBenchmarkController#startBenchmarkImmediatelyIfEncoded function.
  - Second on pressing the Run benchmark button.
@netlify

netlify Bot commented Sep 30, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for webkit-motionmark-preview ready!

Name Link
🔨 Latest commit 886923d
🔍 Latest deploy log https://app.netlify.com/projects/webkit-motionmark-preview/deploys/6abcfa78ab8c6600082dbda1
😎 Deploy Preview https://deploy-preview-126--webkit-motionmark-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@betasheet

Copy link
Copy Markdown

Should we land this directly into main?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants