Added support for EXT_frag_depth

ANGLE DEPS roll 2245:2249

Added support for EXT_frag_depth

BUG=167037
R=gman@chromium.org, kbr@chromium.org, piman@chromium.org, zmo@chromium.org

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

[bajones, 2013-05-24 18:08:39]

Will require an ANGLE roll before it can land, in the process of landing the
ANGLE changes now.

[Zhenyao Mo, 2013-05-28 18:14:58]

On 2013/05/24 18:08:39, bajones wrote:
> Will require an ANGLE roll before it can land, in the process of landing the
> ANGLE changes now.

LGTM

[greggman, 2013-05-28 18:17:58]

a couple of things then lgtm

https://codereview.chromium.org/15846014/diff/1/gpu/command_buffer/service/fe...
File gpu/command_buffer/service/feature_info.cc (right):

https://codereview.chromium.org/15846014/diff/1/gpu/command_buffer/service/fe...
gpu/command_buffer/service/feature_info.cc:700:
AddExtensionString("GL_EXT_frag_depth");
needs feature_flags_.ext_frag_depth = true;

https://codereview.chromium.org/15846014/diff/1/gpu/command_buffer/service/fe...
File gpu/command_buffer/service/feature_info_unittest.cc (right):

https://codereview.chromium.org/15846014/diff/1/gpu/command_buffer/service/fe...
gpu/command_buffer/service/feature_info_unittest.cc:820:
EXPECT_THAT(info_->extensions(), HasSubstr("GL_EXT_frag_depth"));
needs

 EXPECT_TRUE(info_->feature_flags().ext_frag_depth);

[Ken Russell (switch to Gerrit), 2013-05-29 15:16:44]

LGTM with gman's feedback addressed.

https://codereview.chromium.org/15846014/diff/4002/content/public/common/cont...
File content/public/common/content_switches.cc (right):

https://codereview.chromium.org/15846014/diff/4002/content/public/common/cont...
content/public/common/content_switches.cc:845: // Enabled WebGL extensions not
yet approved by the community.
Enabled -> Enables?

[commit-bot: I haz the power, 2013-06-02 22:38:12]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/bajones@chromium.org/15846014/9001

[commit-bot: I haz the power, 2013-06-02 22:47:17]

Retried try job too often on chromium_presubmit for step(s) presubmit
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_p...

[bajones, 2013-06-03 16:43:08]

Need LGTM from joi@ for content/public and an owner of content/renderer

[piman, 2013-06-03 18:06:47]

One thing, after that LGTM.

https://codereview.chromium.org/15846014/diff/9001/content/public/common/cont...
File content/public/common/content_switches.h (right):

https://codereview.chromium.org/15846014/diff/9001/content/public/common/cont...
content/public/common/content_switches.h:267: CONTENT_EXPORT extern const char
kEnableWebGLDraftExtensions[];
Please also add this flag to kForwardSwitches in
chrome/browser/chromeos/login/chrome_restart_request.cc so that it works in
guest mode on Chrome OS.

[jam, 2013-06-03 20:29:46]

you added me but piman reviewed already, so removing myself

[Jói, 2013-06-03 21:27:45]

Removed myself as well, piman@'s review is sufficient for //content.

[commit-bot: I haz the power, 2013-06-03 21:31:33]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/bajones@chromium.org/15846014/20001

[bajones, 2013-06-04 03:33:53]

Committed patchset #5 manually as r203825 (presubmit successful).