Fix use_image_texture_target inconsistencies

content/renderer/render_thread_impl.h

Index: content/renderer/render_thread_impl.h
diff --git a/content/renderer/render_thread_impl.h b/content/renderer/render_thread_impl.h
index 477c555c756ee78cc2c418420529fe9d5e5bbcd0..5ac171784397311f3fe7faf32bc815ce01c700a9 100644
--- a/content/renderer/render_thread_impl.h
+++ b/content/renderer/render_thread_impl.h
@@ -206,7 +206,8 @@ class CONTENT_EXPORT RenderThreadImpl
   bool IsPartialRasterEnabled() override;
   bool IsGpuMemoryBufferCompositorResourcesEnabled() override;
   bool IsElasticOverscrollEnabled() override;
-  std::vector<unsigned> GetImageTextureTargets() override;
+  const cc::RendererSettings::ImageTextureTargetsMap& GetImageTextureTargets()
+      override;
   scoped_refptr<base::SingleThreadTaskRunner>
   GetCompositorMainThreadTaskRunner() override;
   scoped_refptr<base::SingleThreadTaskRunner>
@@ -658,7 +659,7 @@ class CONTENT_EXPORT RenderThreadImpl
   bool is_gpu_memory_buffer_compositor_resources_enabled_;
   bool is_partial_raster_enabled_;
   bool is_elastic_overscroll_enabled_;
-  std::vector<unsigned> use_image_texture_targets_;
+  cc::RendererSettings::ImageTextureTargetsMap use_image_texture_targets_;

[enne (OOO), 2016/07/15 21:39:32]

This is maybe a readability nit, but I expect something named "use_foo" to be a
boolean.  This is a usage/format => target map, right?  I realize
"use_video_frame_image_targets" is right below, following the same pattern, but
this is a really weird name to my ears.

[ericrk, 2016/07/19 20:22:43]

Renamed this to "buffer_to_texture_target_map"

   std::vector<unsigned> use_video_frame_image_texture_targets_;
   bool are_image_decode_tasks_enabled_;
   bool is_threaded_animation_enabled_;