Workaround for Mac video flickering

ui/gl/yuv_to_rgb_converter.h

Index: ui/gl/yuv_to_rgb_converter.h
diff --git a/ui/gl/yuv_to_rgb_converter.h b/ui/gl/yuv_to_rgb_converter.h
index 794b61511a61b8001428780c22a765e26bbacd51..0905c3343b1e8dd50fba523c8141c59cdec3fabd 100644
--- a/ui/gl/yuv_to_rgb_converter.h
+++ b/ui/gl/yuv_to_rgb_converter.h
@@ -19,6 +19,15 @@ class YUVToRGBConverter {
                        const gfx::Size& size,
                        unsigned rgb_texture);
 
+  // Workaround for https://crbug.com/611511:
+  // Destroying a GL texture object every frame will cause an implicit
+  // glFlush, which exposes a bug whereby incompletely-rendered compositor
+  // frames will appear briefly on-screen.
+  // TODO(ccameron): Fix the underlying but whereby inoppurtune glFlushes

[Daniele Castagna, 2016/05/17 19:29:37]

I think this patch makes sense by itself even if it weren't fixing the bug.
So, while this comment and TODO are useful, I don't think they're necessary
here.
Up to you if you want to leave them or remove them.

+  // cause incorrect rendering results.
+  unsigned y_texture() const { return y_texture_; }
+  unsigned uv_texture() const { return uv_texture_; }
+
  private:
   unsigned framebuffer_ = 0;
   unsigned vertex_shader_ = 0;
@@ -26,6 +35,9 @@ class YUVToRGBConverter {
   unsigned program_ = 0;
   int size_location_ = -1;
   unsigned vertex_buffer_ = 0;
+

[Daniele Castagna, 2016/05/17 19:29:37]

nit: no need for the blank line.

+  unsigned y_texture_ = 0;
+  unsigned uv_texture_ = 0;
 };
 
 }  // namespace gl