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

Test OffscreenCanvas.transferToImageBitmap compositing steps (#3612) #3643

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

kkinnunen-apple
Copy link
Contributor

Test following spec clarifications:

  • the value of preserveDrawingBuffer will have no effect
  • the contents of any depth and/or stencil buffers will be cleared to initial values

@kkinnunen-apple kkinnunen-apple force-pushed the offscreen-transfer-to-image-bitmap-1 branch from a00b946 to 1d65f4f Compare April 18, 2024 17:05
@kkinnunen-apple kkinnunen-apple changed the title Test OffscreenCanvas.transferToImageBitmap compositing steps #3612 Test OffscreenCanvas.transferToImageBitmap compositing steps (#3612) Apr 18, 2024
@kkinnunen-apple kkinnunen-apple force-pushed the offscreen-transfer-to-image-bitmap-1 branch 2 times, most recently from 3044345 to 44dde59 Compare April 18, 2024 17:12
…Group#3612)

Test following spec clarifications:
  * the value of preserveDrawingBuffer will have no effect
  * the contents of any depth and/or stencil buffers will be cleared to
    initial values
gl.bindAttribLocation(p, 0, "a_pos");
gl.linkProgram(p);
gl.useProgram(p);
gl.viewport(0, 0, 128, 128);
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
gl.viewport(0, 0, 128, 128);
gl.viewport(0, 0, gl.canvas.width, gl.canvas.height);

Copy link
Contributor

Choose a reason for hiding this comment

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

It's nice to hardcode fewer things.

Copy link
Member

@kenrussell kenrussell left a comment

Choose a reason for hiding this comment

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

Sorry for dropping this - looks good as long as it's testing the agreed-upon spec.

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.

None yet

3 participants