Occlusion query feature is already in GL ES3.0 spec, not an extension.

gpu/command_buffer/service/feature_info.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "gpu/command_buffer/service/feature_info.h"
 
 #include <stddef.h>
 
 #include <set>
 #include <vector>
@@ skipping 1026 matching lines @@
         validators_.texture_internal_format_storage.AddValue(GL_RGBA16F_EXT);
         validators_.texture_internal_format_storage.AddValue(GL_RGB16F_EXT);
         validators_.texture_internal_format_storage.AddValue(GL_ALPHA16F_EXT);
         validators_.texture_internal_format_storage.AddValue(
             GL_LUMINANCE16F_EXT);
         validators_.texture_internal_format_storage.AddValue(
             GL_LUMINANCE_ALPHA16F_EXT);
     }
   }
 
+  bool have_es3_occlusion_query =
+      gl_version_info_->IsAtLeastGLES(3, 0);
   bool have_ext_occlusion_query_boolean =
       extensions.Contains("GL_EXT_occlusion_query_boolean");
   bool have_arb_occlusion_query2 =
       extensions.Contains("GL_ARB_occlusion_query2");
   bool have_arb_occlusion_query =
       extensions.Contains("GL_ARB_occlusion_query");
 
-  if (have_ext_occlusion_query_boolean ||
+  if (have_es3_occlusion_query ||
+      have_ext_occlusion_query_boolean ||
       have_arb_occlusion_query2 ||
       have_arb_occlusion_query) {
-    AddExtensionString("GL_EXT_occlusion_query_boolean");
+    if (have_ext_occlusion_query_boolean ||

[Zhenyao Mo, 2016/07/04 16:06:05]

You should use context_type_ is ES3 or WEBGL2 here.

Even if we are on top of ES3, extension names may still be included in the
EXTENSIONS string.

[xinghua.cao, 2016/07/05 14:21:23]

Zhenyao, sorry, I don't understand you clearly.
I think I did not change the code original logical flow. What I do is setting
"occlusion_query_boolean" to true only when context is ES3 and has not extension
names. And if we are on top of ES3 and extension names are included in the
EXTENSIONS string, AddExtensionString function will be called.
Do you mean that if context_type_ is ES3 or WebGL2, AddExtensionString cannot be
called? Could you give me more details, thank you.

[Zhenyao Mo, 2016/07/05 21:32:06]

The outside if conditions make sure we actually have the functionality in the
underlying driver, and the inside if conditions decide if we need to report the
extension string.  We only need to report the extension string in ES2 context. I
don't even think we need to report it if it's WebGL1 context. (Sorry my original
comment said the opposite thing)

+        have_arb_occlusion_query2 ||
+        have_arb_occlusion_query) {
+      AddExtensionString("GL_EXT_occlusion_query_boolean");
+    }
     feature_flags_.occlusion_query_boolean = true;
     feature_flags_.use_arb_occlusion_query2_for_occlusion_query_boolean =
         !have_ext_occlusion_query_boolean && have_arb_occlusion_query2;
     feature_flags_.use_arb_occlusion_query_for_occlusion_query_boolean =
         !have_ext_occlusion_query_boolean && have_arb_occlusion_query &&
         !have_arb_occlusion_query2;
   }
 
   if (!workarounds_.disable_angle_instanced_arrays &&
       (extensions.Contains("GL_ANGLE_instanced_arrays") ||
@@ skipping 344 matching lines @@
   if (pos == std::string::npos) {
     extensions_ += (extensions_.empty() ? "" : " ") + str;
   }
 }
 
 FeatureInfo::~FeatureInfo() {
 }
 
 }  // namespace gles2
 }  // namespace gpu