Closed Bug 708125 Opened 8 years ago Closed 8 years ago

OGL Layers thrashes current context in UpdateSurface and RenderLayer

Categories

(Core :: Canvas: WebGL, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla11

People

(Reporter: jgilbert, Assigned: jgilbert)

References

Details

Attachments

(1 file)

For the shared-texture path:
In UpdateSurface we currently switch to the layer GLContext, even if it's not necessary.
In RenderLayer, we switch to the WebGL GLContext to resolve, then back again to the layer GLContext for rendering.

This is possibly adding an extra context switch, when we could be doing all the work in the WebGL GLContext before moving on to the work on the layer GLContext.
This also moves the UpdateSurface code for resolving the buffer back to UpdateSurface, instead of where it is now in RenderLayer.
Attachment #579520 - Flags: review?(bjacob)
Attachment #579520 - Attachment is patch: true
Comment on attachment 579520 [details] [diff] [review]
Reduce current context thrashing in OGL Layers

Review of attachment 579520 [details] [diff] [review]:
-----------------------------------------------------------------

Patch no longer necessary after bug 711649.
Attachment #579520 - Flags: review?(bjacob) → review-
Depends on: 711649
Fixed by bug 711649
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla11
You need to log in before you can comment on or make changes to this bug.