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 support for WebGL2 MIN_PROGRAM_TEXEL_OFFSET #26336

Merged
merged 2 commits into from May 1, 2020

Conversation

@mmatyas
Copy link
Contributor

mmatyas commented Apr 28, 2020

Improves the support of the WebGL2 MIN_PROGRAM_TEXEL_OFFSET property (ie. stores it as a signed integer) and adds support for querying it using GetParameter.

See: https://www.khronos.org/registry/webgl/specs/latest/2.0/#3.7.2

cc @jdm @zakorgy

Depends on #26333 because they touch the same test files.


  • ./mach build -d does not report any errors
  • ./mach test-tidy does not report any errors
  • There are tests for these changes
@highfive
Copy link

highfive commented Apr 28, 2020

Heads up! This PR modifies the following files:

  • @asajeffrey: components/script/dom/webgl2renderingcontext.rs
  • @KiChjang: components/script/dom/webgl2renderingcontext.rs
@highfive
Copy link

highfive commented Apr 28, 2020

warning Warning warning

  • These commits modify unsafe code. Please review it carefully!
@mmatyas mmatyas force-pushed the szeged:mmatyas__webgl_fns_getparam4 branch from 73379d9 to 8eb4f91 Apr 28, 2020
@jdm
jdm approved these changes Apr 28, 2020
@bors-servo
Copy link
Contributor

bors-servo commented Apr 30, 2020

The latest upstream changes (presumably #26354) made this pull request unmergeable. Please resolve the merge conflicts.

mmatyas added 2 commits Apr 27, 2020
Adds support for the following new WebGL2 GetParameter values:

- `PACK_ROW_LENGTH`
- `PACK_SKIP_PIXELS`
- `PACK_SKIP_ROWS`
- `RASTERIZER_DISCARD`
- `UNPACK_IMAGE_HEIGHT`
- `UNPACK_ROW_LENGTH`
- `UNPACK_SKIP_IMAGES`
- `UNPACK_SKIP_PIXELS`
- `UNPACK_SKIP_ROWS`

See: https://www.khronos.org/registry/webgl/specs/latest/2.0/#3.7.2
Improves the support of the WebGL2 `MIN_PROGRAM_TEXEL_OFFSET` property
(ie. stores it as a signed integer) and adds support for querying it
using GetParameter.

See: https://www.khronos.org/registry/webgl/specs/latest/2.0/#3.7.2
@mmatyas mmatyas force-pushed the szeged:mmatyas__webgl_fns_getparam4 branch from 8eb4f91 to 1431a38 Apr 30, 2020
@mmatyas
Copy link
Contributor Author

mmatyas commented Apr 30, 2020

Rebased to #26333.

@jdm
Copy link
Member

jdm commented May 1, 2020

@bors-servo
Copy link
Contributor

bors-servo commented May 1, 2020

📌 Commit 1431a38 has been approved by jdm

@bors-servo
Copy link
Contributor

bors-servo commented May 1, 2020

Testing commit 1431a38 with merge d08c4ff...

@bors-servo
Copy link
Contributor

bors-servo commented May 1, 2020

☀️ Test successful - status-taskcluster
Approved by: jdm
Pushing d08c4ff to master...

@bors-servo bors-servo merged commit d08c4ff into servo:master May 1, 2020
2 checks passed
2 checks passed
Community-TC (pull_request) TaskGroup: success
Details
homu Test successful
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

5 participants
You can’t perform that action at this time.