Closed Bug 1487825 Opened 6 years ago Closed 6 years ago

Don't WaitForBufferOwnership if the surface is already Available.

Categories

(Core :: Graphics: CanvasWebGL, enhancement)

Unspecified
Android
enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla63
Tracking Status
firefox63 --- fixed

People

(Reporter: mortimergoro, Assigned: mortimergoro)

Details

Attachments

(1 file)

There is an assertion failure when doing dynamic canvas resized during WebVR. It can be reproduced with this sample: https://webvr.info/samples/test-slow-render.html?standardSize=1&renderScale=1.0&canvasResizeInterval=33

The problem is that WaitForBufferOwnership is called when not needed (the surface is already available)
Comment on attachment 9005673 [details]
Bug 1487825 - Don't WaitForBufferOwnership if the surface is already Available

Jeff Gilbert [:jgilbert] has approved the revision.
Attachment #9005673 - Flags: review+
Assignee: nobody → imanol
Keywords: checkin-needed
Pushed by apavel@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/1b09c0419d13
Don't WaitForBufferOwnership if the surface is already Available r=jgilbert
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/1b09c0419d13
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: