cc: Invalidate framebuffer contents when clearing

cc: Invalidate framebuffer contents when clearing

When clearing the contents of a framebuffer, also invalidate/discard it
if the GL implementation supports this functionality. This improves
performance especially on tiling architecture GPUs since the graphics
driver does not need to restore the previous contents of the framebuffer
before proceeding with rendering.

Note that normally we don't clear the contents of framebuffers whose
contents we know we will completely re-render. This patch will still
invalidate such framebuffers to let the driver know that the previous
contents are not relevant.

BUG=274334
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=221425

[piman, 2013-09-04 19:23:01]

LGTM+nit

https://codereview.chromium.org/23601013/diff/1/cc/output/gl_renderer.cc
File cc/output/gl_renderer.cc (right):

https://codereview.chromium.org/23601013/diff/1/cc/output/gl_renderer.cc#newc...
cc/output/gl_renderer.cc:301: GLenum attachments[kNumAttachments] = {
nit: [] and use arraysize

[Sami, 2013-09-05 12:40:45]

On 2013/09/04 19:23:01, piman wrote:
> nit: [] and use arraysize

Thanks, I knew there must be something like that but just didn't know what it's
called :)

[commit-bot: I haz the power, 2013-09-05 12:43:37]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/skyostil@chromium.org/23601013/6001

[commit-bot: I haz the power, 2013-09-05 14:52:42]

Change committed as 221425