Closed Bug 1370117 Opened 6 years ago Closed 6 years ago

Fix the external-image status for the WR video pipleline

Categories

(Core :: Graphics: WebRender, enhancement)

Product:
Core
Component:
Graphics: WebRender
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla55
Tracking Flags:
Tracking Status
firefox55 --- fixed

People

(Reporter: jerry, Assigned: jerry)

References

Details

Attachments

(1 file)

Update the external image status for reusing case.
2.77 KB, patch
sotaro
: review+
Details | Diff | Splinter Review 
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.
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+
Pushed by hshih@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/15c554f1a294
Update the external image status for reusing case. r=sotaro
https://hg.mozilla.org/mozilla-central/rev/15c554f1a294
Status: ASSIGNED → RESOLVED
Closed: 6 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.