1184327 - Depth buffer should not be cleared in GL compositor

Status: RESOLVED WONTFIX

(Core :: Graphics, defect)

Description

[Andrew Comminos [:acomminos]]

apitrace reports the following warning:

> Failed to fast clear depth due to scissor being enabled. Possible 5% performance win if avoided.

I discussed this with :mattwoodrow in person and we agreed that it doesn't make any sense for the GL compositor to clear the depth buffer.

Comment 1

[Andrew Comminos [:acomminos]]

Attachment: Disable GL scissor test before clearing.

Talos try push for possible OS X 10.10 regression:

https://treeherder.mozilla.org/#/jobs?repo=try&revision=a36de385f031

Comment 2

[Andrew Comminos [:acomminos]]

Comment on attachment 8634405 [details] [diff] [review]
Disable GL scissor test before clearing.

This shouldn't be necessary with the fix for bug 1184534.

Comment 3

[Andrew Comminos [:acomminos]]

Closing as wontfix- GLX defaults to a GLX_DEPTH_SIZE of 0, falling back to FBConfigs with larger depth sizes if necessary. If we're getting a depth buffer at all, it's because we couldn't find an FBConfig without one.