media: replace LUMINANCE_F16 by RG_88 for 9/10-bit h264 videos

cc/output/shader.h

 // Copyright 2011 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.
 
 #ifndef CC_OUTPUT_SHADER_H_
 #define CC_OUTPUT_SHADER_H_
 
 #include <string>
 
 #include "base/macros.h"
@@ skipping 745 matching lines @@
   int mask_sampler_location_;
   int alpha_location_;
   int mask_tex_coord_scale_location_;
   int mask_tex_coord_offset_location_;
   int color_matrix_location_;
   int color_offset_location_;
 };
 
 class FragmentShaderYUVVideo : public FragmentTexBlendMode {
  public:
+  enum HighbitTexture {

[hubbe, 2016/09/27 20:49:51]

There is no need for three values here, because the shader doesn't need to know
if we're using F16 or Y8, the shader does exactly the same thing for those two
cases.

[dshwang, 2016/09/28 10:33:27]

|resource_multiplier_location| and |resource_offset_location| are used by only
HIGHBIT_LUMINANCE_F16. To make it easy to remove HIGHBIT_LUMINANCE_F16 path
later, I highlight HIGHBIT_LUMINANCE_F16 specific code.

[hubbe, 2016/09/28 17:57:44]

True I suppose. But you'll need the resource_multiplier for RG88 too, right?

[dshwang, 2016/09/29 18:52:28]

no, RG88 don't need it.

+    HIGHBIT_Y8 = 0,  // either RED_8 or LUMINANCE_8
+    HIGHBIT_LUMINANCE_F16 = 1,
+    HIGHBIT_RG88 = 2,
+    LAST_HIGHBIT_TEXTURE = HIGHBIT_RG88
+  };
+
   FragmentShaderYUVVideo();
   std::string GetShaderString(
       TexCoordPrecision precision, SamplerType sampler) const;
 
-  void SetFeatures(bool use_alpha_texture, bool use_nv12, bool use_color_lut);
+  void SetFeatures(bool use_alpha_texture,
+                   bool use_nv12,
+                   bool use_color_lut,
+                   HighbitTexture highbit_texture);
 
   void Init(gpu::gles2::GLES2Interface* context,
             unsigned program,
             int* base_uniform_index);
   int y_texture_location() const { return y_texture_location_; }
   int u_texture_location() const { return u_texture_location_; }
   int v_texture_location() const { return v_texture_location_; }
   int uv_texture_location() const { return uv_texture_location_; }
   int a_texture_location() const { return a_texture_location_; }
   int lut_texture_location() const { return lut_texture_location_; }
   int alpha_location() const { return alpha_location_; }
   int yuv_matrix_location() const { return yuv_matrix_location_; }
   int yuv_adj_location() const { return yuv_adj_location_; }
+  int ya_size_location() const { return ya_size_location_; }
+  int uv_size_location() const { return uv_size_location_; }
   int ya_clamp_rect_location() const { return ya_clamp_rect_location_; }
   int uv_clamp_rect_location() const { return uv_clamp_rect_location_; }
+  int max_channel_value_location() const { return max_channel_value_location_; }
   int resource_multiplier_location() const {
     return resource_multiplier_location_;
   }
   int resource_offset_location() const { return resource_offset_location_; }
 
  private:
   bool use_alpha_texture_;
   bool use_nv12_;
   bool use_color_lut_;
+  bool highbit_texture_ = HIGHBIT_Y8;
 
   int y_texture_location_;
   int u_texture_location_;
   int v_texture_location_;
   int uv_texture_location_;
   int a_texture_location_;
   int lut_texture_location_;
   int alpha_location_;
   int yuv_matrix_location_;
   int yuv_adj_location_;
+  int max_channel_value_location_;
+  int ya_size_location_;
+  int uv_size_location_;
   int ya_clamp_rect_location_;
   int uv_clamp_rect_location_;
   int resource_multiplier_location_;
   int resource_offset_location_;
 
   DISALLOW_COPY_AND_ASSIGN(FragmentShaderYUVVideo);
 };
 
 class FragmentShaderColor : public FragmentTexBlendMode {
  public:
@@ skipping 29 matching lines @@
 
  private:
   int color_location_;
 
   DISALLOW_COPY_AND_ASSIGN(FragmentShaderColorAA);
 };
 
 }  // namespace cc
 
 #endif  // CC_OUTPUT_SHADER_H_