Distinguish ES2/ES3 contexts in GPU command buffer

gpu/command_buffer/common/gles2_cmd_utils.cc

Index: gpu/command_buffer/common/gles2_cmd_utils.cc
diff --git a/gpu/command_buffer/common/gles2_cmd_utils.cc b/gpu/command_buffer/common/gles2_cmd_utils.cc
index 3985193e1167e80c8c53b120b3de60ef559263f1..0950b41df9e5fb70068486747f8e079c03e270dd 100644
--- a/gpu/command_buffer/common/gles2_cmd_utils.cc
+++ b/gpu/command_buffer/common/gles2_cmd_utils.cc
@@ -894,6 +894,7 @@ const int32 kBufferDestroyed = 0x3095;  // EGL_BUFFER_DESTROYED
 const int32 kBindGeneratesResource = 0x10000;
 const int32 kFailIfMajorPerfCaveat = 0x10001;
 const int32 kLoseContextWhenOutOfMemory = 0x10002;
+const int32 kES3ContextRequired = 0x10003;

[Ken Russell (switch to Gerrit), 2015/03/03 23:14:49]

Might want to think about how this will evolve once ES 3.1 comes along. Maybe
two optional integer parameters specifying the minimum major and minor version
of ES? This seems fine for now though.

[Zhenyao Mo, 2015/03/04 00:24:03]

I was thinking about that, but the blink side attributes defines WebGL version
as one integer, so I feel no need to take the extra step in command buffer for
now.  What you suggest sounds good to me when we get to that point.

[piman, 2015/03/04 00:44:50]

Actually I think we should get rid of the EGL-type int-array attrib lists, and
instead explicitly serialize a struct over IPC. It would be much easier to
manage and evolve.

[Zhenyao Mo, 2015/03/04 00:49:44]

I agree.  Will follow up on this.

 
 }  // namespace
 
@@ -909,7 +910,8 @@ ContextCreationAttribHelper::ContextCreationAttribHelper()
       buffer_preserved(true),
       bind_generates_resource(true),
       fail_if_major_perf_caveat(false),
-      lose_context_when_out_of_memory(false) {}
+      lose_context_when_out_of_memory(false),
+      es3_context_required(false) {}
 
 void ContextCreationAttribHelper::Serialize(std::vector<int32>* attribs) const {
   if (alpha_size != -1) {
@@ -952,6 +954,8 @@ void ContextCreationAttribHelper::Serialize(std::vector<int32>* attribs) const {
   attribs->push_back(fail_if_major_perf_caveat ? 1 : 0);
   attribs->push_back(kLoseContextWhenOutOfMemory);
   attribs->push_back(lose_context_when_out_of_memory ? 1 : 0);
+  attribs->push_back(kES3ContextRequired);
+  attribs->push_back(es3_context_required ? 1 : 0);
   attribs->push_back(kNone);
 }
 
@@ -1006,6 +1010,9 @@ bool ContextCreationAttribHelper::Parse(const std::vector<int32>& attribs) {
       case kLoseContextWhenOutOfMemory:
         lose_context_when_out_of_memory = value != 0;
         break;
+      case kES3ContextRequired:
+        es3_context_required = value != 0;
+        break;
       case kNone:
         // Terminate list, even if more attributes.
         return true;