Enabled EXT_sRGB WebGL extension support in the command buffer.

Enabled EXT_sRGB WebGL extension support in the command buffer.

BUG=386048
Committed: https://crrev.com/2a2ea235a39a4b0bcad1f75934bdc09fb4d41733
Cr-Commit-Position: refs/heads/master@{#299371}

[bajones, 2014-10-09 21:53:18]

bajones@chromium.org changed reviewers:
+ kbr@chromium.org, zmo@chromium.org

[bajones, 2014-10-09 21:53:19]

PTAL!

[Zhenyao Mo, 2014-10-09 22:04:55]

The code looks good in general, but could you please add a test to
feature_info_unittests.cc?

Similar to this one:
https://code.google.com/p/chromium/codesearch#chromium/src/gpu/command_buffer...

[bajones, 2014-10-10 23:19:53]

On 2014/10/09 at 22:04:55, zmo wrote:
> The code looks good in general, but could you please add a test to
feature_info_unittests.cc?
> 
> Similar to this one:
https://code.google.com/p/chromium/codesearch#chromium/src/gpu/command_buffer...

Very good suggestion, thanks for catching that! Implemented.

[Zhenyao Mo, 2014-10-10 23:45:36]

On 2014/10/10 23:19:53, bajones wrote:
> On 2014/10/09 at 22:04:55, zmo wrote:
> > The code looks good in general, but could you please add a test to
> feature_info_unittests.cc?
> > 
> > Similar to this one:
>
https://code.google.com/p/chromium/codesearch#chromium/src/gpu/command_buffer...
> 
> Very good suggestion, thanks for catching that! Implemented.

LGTM

[Ken Russell (switch to Gerrit), 2014-10-11 03:20:43]

LGTM too.

https://codereview.chromium.org/638143003/diff/40001/gpu/command_buffer/servi...
File gpu/command_buffer/service/texture_manager.cc (right):

https://codereview.chromium.org/638143003/diff/40001/gpu/command_buffer/servi...
gpu/command_buffer/service/texture_manager.cc:1645: if
(gfx::GetGLImplementation() == gfx::kGLImplementationDesktopGL) {
Is this going to have to change for the coming GLES3 upgrade too? If so is a
TODO warranted?

[bajones, 2014-10-13 20:09:47]

The CQ bit was checked by bajones@chromium.org

[commit-bot: I haz the power, 2014-10-13 20:10:30]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/638143003/160001

[commit-bot: I haz the power, 2014-10-13 21:39:07]

Committed patchset #3 (id:160001)

[commit-bot: I haz the power, 2014-10-13 21:39:53]

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