Old video frames seemed to be rendered on Windows

RESOLVED FIXED in Firefox 59

Status

()

enhancement
P1
normal
RESOLVED FIXED
a year ago
a year ago

People

(Reporter: sotaro, Assigned: sotaro)

Tracking

unspecified
mozilla59
Unspecified
Windows
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox59 fixed)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Assignee

Description

a year ago
During youtube video playback on Windows like the following, some video order seemed to be reverted. We might to recycle video frames too early.

  https://www.youtube.com/watch?v=gu6JunBsyUQ
Assignee

Updated

a year ago
Assignee: nobody → sotaro.ikeda.g
Assignee

Comment 1

a year ago
Add RenderCompositorANGLE to split ANGLE specific handling and added ID3D11Query handling by referring MLGDeviceD3D11.

By it, we expect that when WebRenderTextureHost is recycled, its usage on GPU is already ended.
Assignee

Comment 2

a year ago
We could get ID3D11Device from  gfx::DeviceManagerDx::Get()->GetCompositorDevice() since Bug 1395107 is addressed.
Assignee

Updated

a year ago
Attachment #8943182 - Flags: review?(nical.bugzilla)
Attachment #8943182 - Flags: review?(nical.bugzilla) → review+

Comment 4

a year ago
Pushed by sikeda@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/70c2890fb875
Use ID3D11Query to wait previous frame completion r=nical
Assignee

Comment 6

a year ago
Sorry, explicit keyword was missed and Comment 3 did not catch it :(
Flags: needinfo?(sotaro.ikeda.g)
Assignee

Comment 7

a year ago
Fixed the static analysis problem.
Attachment #8943182 - Attachment is obsolete: true
Attachment #8943807 - Flags: review+

Comment 9

a year ago
Pushed by sikeda@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/29e471159178
Use ID3D11Query to wait previous frame completion r=nical

Comment 10

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/29e471159178
Status: NEW → RESOLVED
Last Resolved: a year ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
You need to log in before you can comment on or make changes to this bug.