Hookup clear_uniforms_before_first_program_use workaround with implementation.

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 0da039dc121ec2051a55d98f5933d2edd2f6b984..ece436af13fc58de1ae819248aa394552a15815d 100644
--- a/gpu/command_buffer/service/program_manager.h
+++ b/gpu/command_buffer/service/program_manager.h
@@ -20,7 +20,6 @@
 namespace gpu {
 namespace gles2 {
 
-class FeatureInfo;
 class ProgramCache;
 class ProgramManager;
 class Shader;
@@ -157,7 +156,6 @@ class GPU_EXPORT Program : public base::RefCounted<Program> {
   bool Link(ShaderManager* manager,
             ShaderTranslator* vertex_translator,
             ShaderTranslator* fragment_shader,
-            FeatureInfo* feature_info,
             const ShaderCacheCallback& shader_callback);
 
   // Performs glValidateProgram and related activities.
@@ -378,7 +376,7 @@ class GPU_EXPORT ProgramManager {
 
   void DoCompileShader(Shader* shader,
                        ShaderTranslator* translator,
-                       FeatureInfo* feature_info);
+                       bool angle_translated_shader_source);

[Ken Russell (switch to Gerrit), 2014/02/04 22:15:11]

This argument name is confusing now. Please rename it to something like
"preserve_translated_shader_source" or "save_translated_shader_source".

[Zhenyao Mo, 2014/02/05 02:02:41]

Changed to enum type.

 
   uint32 max_varying_vectors() const {
     return max_varying_vectors_;
@@ -404,8 +402,6 @@ class GPU_EXPORT ProgramManager {
 
   bool have_context_;
 
-  bool disable_workarounds_;
-
   // Used to clear uniforms.
   std::vector<uint8> zero_;