Closed
Bug 1616132
Opened 4 years ago
Closed 4 years ago
getProgramParameter(ATTACHED_SHADERS) always returns 2
Categories
(Core :: Graphics: CanvasWebGL, defect, P1)
Core
Graphics: CanvasWebGL
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr68 | --- | unaffected |
firefox73 | --- | unaffected |
firefox74 | + | fixed |
firefox75 | --- | fixed |
People
(Reporter: jgilbert, Assigned: jgilbert)
References
Details
(Keywords: regression)
Attachments
(1 file)
47 bytes,
text/x-phabricator-request
|
RyanVM
:
approval-mozilla-beta+
|
Details | Review |
There are always two valid slots, but we need to check the slot contents for non-nulls.
Assignee | ||
Comment 1•4 years ago
|
||
It was always returning 2.
Assignee | ||
Updated•4 years ago
|
Severity: normal → critical
status-firefox73:
--- → unaffected
status-firefox74:
--- → affected
status-firefox75:
--- → affected
Assignee | ||
Updated•4 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•4 years ago
|
||
Added a test upstream:
https://github.com/KhronosGroup/WebGL/pull/3023
Updated•4 years ago
|
See Also: → https://webcompat.com/issues/47886
Assignee | ||
Comment 3•4 years ago
|
||
[Tracking Requested - why for this release]: This fixes YouTube 360 videos.
tracking-firefox74:
--- → ?
Pushed by jgilbert@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/c906f7b0f8dc ATTACHED_SHADERS must count non-null slots. r=lsalzman
Updated•4 years ago
|
Comment 5•4 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla75
Updated•4 years ago
|
Webcompat Priority: --- → P2
Assignee | ||
Comment 7•4 years ago
|
||
Comment on attachment 9127154 [details]
Bug 1616132 - ATTACHED_SHADERS must count non-null slots.
Beta/Release Uplift Approval Request
- User impact if declined: YouTube 360 video broken in Firefox
- Is this code covered by automated tests?: No
- Has the fix been verified in Nightly?: Yes
- Needs manual test from QE?: No
- If yes, steps to reproduce:
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): Really simple fix, and we're adding tests for it so we don't regress again.
- String changes made/needed: none
Attachment #9127154 -
Flags: approval-mozilla-beta?
Comment 8•4 years ago
|
||
Comment on attachment 9127154 [details]
Bug 1616132 - ATTACHED_SHADERS must count non-null slots.
Fixes a webgl regression causing real-world site issues. Approved for 74.0b5.
Attachment #9127154 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 9•4 years ago
|
||
bugherder uplift |
Comment 10•4 years ago
|
||
Bugbug thinks this bug is a regression, but please revert this change in case of error.
Keywords: regression
Updated•4 years ago
|
status-firefox-esr68:
--- → unaffected
You need to log in
before you can comment on or make changes to this bug.
Description
•