Use D3DChecks::DoesAlphaTextureSharingWork again

REOPENED
Assigned to

Status

()

Core
Graphics: Layers
REOPENED
2 years ago
a year ago

People

(Reporter: mattwoodrow, Assigned: mattwoodrow, NeedInfo)

Tracking

(Depends on: 1 bug)

unspecified
mozilla52
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox51+ wontfix, firefox52 wontfix)

Details

Attachments

(1 attachment)

(Assignee)

Description

2 years ago
Created attachment 8794641 [details] [diff] [review]
use-alpha-texture-sharing-works-again

We used to check this as part of setting up the ImageBridge device, but it went away as part of removing that device (bug 1289640).

We should check it again before trying to use alpha texture sharing.
Attachment #8794641 - Flags: review?(dvander)
Attachment #8794641 - Flags: review?(dvander) → review+

Comment 1

2 years ago
Pushed by mwoodrow@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/d98c1bfa1caf
Use D3DChecks::DoesAlphaTextureSharingWork again. r=dvander
[Tracking Requested - why for this release]: bug 1289640 changed the behaviour of video significantly (under the covers) - we want to make sure it's complete.
status-firefox51: --- → affected
tracking-firefox51: --- → ?

Comment 4

2 years ago
Pushed by mwoodrow@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/f5a4eadc760e
Use D3DChecks::DoesAlphaTextureSharingWork again. r=dvander

Comment 5

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/f5a4eadc760e
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox52: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla52

Comment 6

2 years ago
Backout by cbook@mozilla.com:
https://hg.mozilla.org/mozilla-central/rev/89b7b2412095
Backed out changeset f5a4eadc760e
had to back this out again from m-c seems this caused together with a what it seems incomplete backout reftest failures like https://treeherder.mozilla.org/logviewer.html#?job_id=5175443&repo=mozilla-central and so got backout from m-c and the integration branches
(Assignee)

Comment 8

2 years ago
This got backed out for making bug 1300355 more frequent. I am investigating.
Depends on: 1300355
Track 51+ as the behaviour of video is changed in 51.
tracking-firefox51: ? → +
reopen because this got backed out
Status: RESOLVED → REOPENED
status-firefox52: fixed → affected
Resolution: FIXED → ---
How important is this to fix for 51? Should it be a blocker?  (Emailing so that Matt sees my question)
This is waiting for work from bug 1315871 and bug 1213961 and is less of a priority right now but still on Matt's radar.
status-firefox51: affected → fix-optional
Mass wontfix for bugs affecting firefox 52.
status-firefox52: affected → wontfix
status-firefox51: fix-optional → wontfix
You need to log in before you can comment on or make changes to this bug.