Allow one-copy task tile worker pool to use compressed textures.

cc/output/renderer_settings.h

 // Copyright 2014 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_RENDERER_SETTINGS_H_
 #define CC_OUTPUT_RENDERER_SETTINGS_H_
 
 #include "base/basictypes.h"
 #include "cc/base/cc_export.h"
 
 namespace cc {
 
 class CC_EXPORT RendererSettings {
  public:
   RendererSettings();
   ~RendererSettings();
 
   bool allow_antialiasing;
   bool force_antialiasing;
   bool force_blending_with_shaders;
   bool partial_swap_enabled;
   bool finish_rendering_on_resize;
   bool should_clear_root_render_pass;
   bool disable_display_vsync;
   bool delay_releasing_overlay_resources;
   double refresh_rate;
   int highp_threshold_min;
   bool use_rgba_4444_textures;
   size_t texture_id_allocation_chunk_size;
+  bool use_tile_compression;

[reveman, 2015/09/30 09:55:15]

Can we merge this with use_rgba_4444_textures somehow? E.g. replace both with a
tile_resource_format setting. Maybe
tile_resource_format+tile_resource_format_with_alpha so the embedder can also
chose the format used when alpha is required.

[christiank, 2015/11/26 15:35:35]

This would be good, I agree. I think we'd need a new enum for this since we need
an option for (!use_rgba_444_textures && !use_tile_compression). I don't think
using ResourceFormat for this would work as it has no default format. Replacing
ResourceProvider::best_texture_format() with a setting looks tricky. Perhaps
something like this would work?

enum class MemoryEfficientResourceFormat {
  NONE,
  RGBA_4444,
  ETC1,
};

MemoryEfficientResourceFormat memory_efficient_format;
MemoryEfficientResourceFormat memory_efficient_format_with_alpha;

What do you think?

[reveman, 2015/11/27 16:46:49]

Can we just use ResourceFormat and require it to be set to RGBA_8888 or
equivalent when compressed formats shouldn't be used? 

[christiank, 2015/11/30 15:41:08]

Yes, that would work. I have made this change now, not sure if I should document
this behavior somewhere?

 };
 
 }  // namespace cc
 
 #endif  // CC_OUTPUT_RENDERER_SETTINGS_H_