Reuse GLFenceNV when an existing CommandsCompletedQuery is reused

Reuse GLFenceNV when an existing CommandsCompletedQuery is reused

This is a first part of the fix for 573120. When Begin/End
are called again for a completed query that already has an
instance of GLFenceNV, the existing GLFenceNV instance is
reused rather than creating a new one.
For each frame that saves one call to glGenFencesNV and
one call to glDeleteFencesNV .
My estimation is that that saves about 0.2 Wt when playing
1080p @ 60Hz h.264 content.

The second part of the fix will add reusing instances of
CommandsCompletedQuery.

BUG=573120
Committed: https://crrev.com/1f3c24312efb7d76ccb331f170aa0941e6fdfda9
Cr-Commit-Position: refs/heads/master@{#371898}

[stanisc, 2016-01-27 19:28:16]

Description was changed from

==========
GLFence Prototype

BUG=
==========

to

==========
Reuse GLFenceNV when an existing CommandsCompletedQuery is reused

This is a first part of the fix for 573120. When Begin/End
are called again for a completed query that already has an
instance of GLFenceNV, the existing instance is reused rather
than creating a new one.
For each frame that saves one call to glGenFencesNV and
one call to glDeleteFencesNV .
My estimation is that that saves about 0.2 Wt when playing
1080p @ 60Hz h.264 content.

The second part of the fix will add reusing instances of
CommandsCompletedQuery.

BUG=573120
==========

[stanisc, 2016-01-27 19:28:16]

stanisc@chromium.org changed reviewers:
+ jbauman@chromium.org

[stanisc, 2016-01-27 19:29:15]

Description was changed from

==========
Reuse GLFenceNV when an existing CommandsCompletedQuery is reused

This is a first part of the fix for 573120. When Begin/End
are called again for a completed query that already has an
instance of GLFenceNV, the existing instance is reused rather
than creating a new one.
For each frame that saves one call to glGenFencesNV and
one call to glDeleteFencesNV .
My estimation is that that saves about 0.2 Wt when playing
1080p @ 60Hz h.264 content.

The second part of the fix will add reusing instances of
CommandsCompletedQuery.

BUG=573120
==========

to

==========
Reuse GLFenceNV when an existing CommandsCompletedQuery is reused

This is a first part of the fix for 573120. When Begin/End
are called again for a completed query that already has an
instance of GLFenceNV, the existing GLFenceNV instance is 
reused rather than creating a new one.
For each frame that saves one call to glGenFencesNV and
one call to glDeleteFencesNV .
My estimation is that that saves about 0.2 Wt when playing
1080p @ 60Hz h.264 content.

The second part of the fix will add reusing instances of
CommandsCompletedQuery.

BUG=573120
==========

[stanisc, 2016-01-27 19:40:18]

stanisc@chromium.org changed reviewers:
+ kbr@chromium.org

[stanisc, 2016-01-27 19:40:19]

kbr@: Please review changes in ui/gl.
jbaman@: Please review the rest.

[Ken Russell (switch to Gerrit), 2016-01-27 20:02:33]

lgtm

[jbauman, 2016-01-27 20:09:02]

lgtm

[stanisc, 2016-01-27 22:21:31]

The CQ bit was checked by stanisc@chromium.org

[commit-bot: I haz the power, 2016-01-27 22:22:40]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1637293002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1637293002/20001

[commit-bot: I haz the power, 2016-01-27 22:29:48]

Description was changed from

==========
Reuse GLFenceNV when an existing CommandsCompletedQuery is reused

This is a first part of the fix for 573120. When Begin/End
are called again for a completed query that already has an
instance of GLFenceNV, the existing GLFenceNV instance is 
reused rather than creating a new one.
For each frame that saves one call to glGenFencesNV and
one call to glDeleteFencesNV .
My estimation is that that saves about 0.2 Wt when playing
1080p @ 60Hz h.264 content.

The second part of the fix will add reusing instances of
CommandsCompletedQuery.

BUG=573120
==========

to

==========
Reuse GLFenceNV when an existing CommandsCompletedQuery is reused

This is a first part of the fix for 573120. When Begin/End
are called again for a completed query that already has an
instance of GLFenceNV, the existing GLFenceNV instance is 
reused rather than creating a new one.
For each frame that saves one call to glGenFencesNV and
one call to glDeleteFencesNV .
My estimation is that that saves about 0.2 Wt when playing
1080p @ 60Hz h.264 content.

The second part of the fix will add reusing instances of
CommandsCompletedQuery.

BUG=573120
==========

[commit-bot: I haz the power, 2016-01-27 22:29:49]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-01-27 22:31:53]

Description was changed from

==========
Reuse GLFenceNV when an existing CommandsCompletedQuery is reused

This is a first part of the fix for 573120. When Begin/End
are called again for a completed query that already has an
instance of GLFenceNV, the existing GLFenceNV instance is 
reused rather than creating a new one.
For each frame that saves one call to glGenFencesNV and
one call to glDeleteFencesNV .
My estimation is that that saves about 0.2 Wt when playing
1080p @ 60Hz h.264 content.

The second part of the fix will add reusing instances of
CommandsCompletedQuery.

BUG=573120
==========

to

==========
Reuse GLFenceNV when an existing CommandsCompletedQuery is reused

This is a first part of the fix for 573120. When Begin/End
are called again for a completed query that already has an
instance of GLFenceNV, the existing GLFenceNV instance is
reused rather than creating a new one.
For each frame that saves one call to glGenFencesNV and
one call to glDeleteFencesNV .
My estimation is that that saves about 0.2 Wt when playing
1080p @ 60Hz h.264 content.

The second part of the fix will add reusing instances of
CommandsCompletedQuery.

BUG=573120

Committed: https://crrev.com/1f3c24312efb7d76ccb331f170aa0941e6fdfda9
Cr-Commit-Position: refs/heads/master@{#371898}
==========

[commit-bot: I haz the power, 2016-01-27 22:31:54]

Patchset 2 (id:??) landed as
https://crrev.com/1f3c24312efb7d76ccb331f170aa0941e6fdfda9
Cr-Commit-Position: refs/heads/master@{#371898}