Fix the external-image status for the WR video pipleline

RESOLVED FIXED in Firefox 55

Status

()

RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: jerry, Assigned: jerry)

Tracking

unspecified
mozilla55
Points:
---

Firefox Tracking Flags

(firefox55 fixed)

Details

Attachments

(1 attachment)

When gecko go through the reusing case, the aUseExternalImage isn't set to the correct value. It is always false here. Then, gecko will use the non-external-image code path even though it should use the external-image path.
Created attachment 8874270 [details] [diff] [review]
Update the external image status for reusing case.

When gecko go through the reusing case, the aUseExternalImage isn't set to the correct value. It is always false here.
Then, gecko will use the non-external-image code path later. Add a new data member "mUseExternalImage" to save the previous status.

MozReview-Commit-ID: KvzgIzYXNAB
Attachment #8874270 - Flags: review?(sotaro.ikeda.g)
Comment on attachment 8874270 [details] [diff] [review]
Update the external image status for reusing case.

Thanks.
Attachment #8874270 - Flags: review?(sotaro.ikeda.g) → review+

Comment 4

2 years ago
Pushed by hshih@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/15c554f1a294
Update the external image status for reusing case. r=sotaro

Comment 5

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/15c554f1a294
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
status-firefox55: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.