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

cc/output/renderer_settings.cc

 // 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.
 
 #include "cc/output/renderer_settings.h"
 
 #include <limits>
 
 #include "base/logging.h"
+#include "cc/resources/platform_color.h"
 
 namespace cc {
 
 RendererSettings::RendererSettings()
     : allow_antialiasing(true),
       force_antialiasing(false),
       force_blending_with_shaders(false),
       partial_swap_enabled(false),
       finish_rendering_on_resize(false),
       should_clear_root_render_pass(true),
       disable_display_vsync(false),
       delay_releasing_overlay_resources(false),
       refresh_rate(60.0),
       highp_threshold_min(0),
-      use_rgba_4444_textures(false),
       texture_id_allocation_chunk_size(64),
-      use_gpu_memory_buffer_resources(false) {}
+      use_gpu_memory_buffer_resources(false),
+      preferred_tile_format(PlatformColor::BestTextureFormat(true)) {}

[reveman, 2015/12/02 18:27:18]

nit: I find calling BestTextureFormat with supports_bgra8888=true in a bunch of
places a bit awkward. Can you add a new function instead? E.g. remove  the
argument from BestTextureFormat and add a BestSupportedTextureFormat function
that still takes an argument if we even need that now that we have
IsResourceFormatSupported?

Btw, the name should be Best(Supported)ResourceFormat and not TextureFormat. No
need to but feel free to fix that as part of touching this code if you like.

[christiank, 2015/12/03 12:57:59]

I was thinking along the same lines. Fixed.

I think it makes sense to keep BestSupportedTextureFormat since there is a
different condition for the best render buffer format
(caps.gpu.render_buffer_format_bgra8888).

 
 RendererSettings::~RendererSettings() {
 }
 
 }  // namespace cc