Add gl tests to make sure when a buffer is unmapped, all access path generates an INVALID_OPERATION.

gpu/command_buffer/service/program_manager.h

Index: gpu/command_buffer/service/program_manager.h
diff --git a/gpu/command_buffer/service/program_manager.h b/gpu/command_buffer/service/program_manager.h
index 36bc94e84d1e7f4ce0f7dcd4f9cc6339aeb2471c..4e48080c6d701e04a1c9a296d2c4371c6710231b 100644
--- a/gpu/command_buffer/service/program_manager.h
+++ b/gpu/command_buffer/service/program_manager.h
@@ -386,12 +386,13 @@ class GPU_EXPORT Program : public base::RefCounted<Program> {
     return bind_attrib_location_map_;
   }
 
-  const std::vector<std::string>& transform_feedback_varyings() const {
-    return transform_feedback_varyings_;
+  const std::vector<std::string>& effective_transform_feedback_varyings()
+      const {
+    return effective_transform_feedback_varyings_;
   }
 
-  GLenum transform_feedback_buffer_mode() const {
-    return transform_feedback_buffer_mode_;
+  GLenum effective_transform_feedback_buffer_mode() const {
+    return effective_transform_feedback_buffer_mode_;
   }
 
   // See member declaration for details.
@@ -553,10 +554,14 @@ class GPU_EXPORT Program : public base::RefCounted<Program> {
   // uniform-location binding map from glBindUniformLocationCHROMIUM() calls.
   LocationMap bind_uniform_location_map_;
 
+  // Set by glTransformFeedbackVaryings().
   std::vector<std::string> transform_feedback_varyings_;
-
   GLenum transform_feedback_buffer_mode_;
 
+  // After a successful link.
+  std::vector<std::string> effective_transform_feedback_varyings_;
+  GLenum effective_transform_feedback_buffer_mode_;

[vmiura, 2016/10/10 18:20:48]

Please initialize effective_transform_feedback_buffer_mode_ to GL_NONE.

[Zhenyao Mo, 2016/10/10 21:02:07]

Done.

+
   // Fragment input-location binding map from
   // glBindFragmentInputLocationCHROMIUM() calls.
   LocationMap bind_fragment_input_location_map_;