cc: Changes to use GL API for GpuMemoryBuffers

cc/trees/layer_tree_host_impl.cc

 // 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.
 
 #include "cc/trees/layer_tree_host_impl.h"
 
 #include <algorithm>
 
 #include "base/basictypes.h"
 #include "base/json/json_writer.h"
@@ skipping 1371 matching lines @@
       output_surface.get(), settings_.highp_threshold_min);
   if (!resource_provider)
     return false;
 
   if (settings_.impl_side_painting) {
     tile_manager_.reset(new TileManager(this,
                                         resource_provider.get(),
                                         settings_.num_raster_threads,
                                         settings_.use_color_estimator,
                                         settings_.prediction_benchmarking,
-                                        rendering_stats_instrumentation_));
+                                        rendering_stats_instrumentation_,
+                                        settings_.use_map_image));

[reveman, 2013/05/22 18:56:41]

settings_.use_map_image doesn't seem to take output surface capabilities() into
account. am I missing something or should you be using
output_surface->capabilities().using_map_image here?

[kaanb, 2013/05/22 19:13:03]

We plan to use a command flag to initially control this feature. After that the
plan is to enable it by default for WebView. 

[reveman, 2013/05/22 19:51:19]

but shouldn't it be "settings_.use_map_image && capabilities().using_map_image"?
I don't understand what the capabilities part is about otherwise.

[kaanb, 2013/05/22 20:21:01]

Okay, I'm using RendererCapabilities, not OutputSurface capabilities.

Please take another look.

     UpdateTileManagerMemoryPolicy(managed_memory_policy_);
   }
 
   if (output_surface->capabilities().has_parent_compositor) {
     renderer_ = DelegatingRenderer::Create(this, output_surface.get(),
                                            resource_provider.get());
   } else if (output_surface->context3d()) {
     renderer_ = GLRenderer::Create(this,
                                    output_surface.get(),
                                    resource_provider.get(),
@@ skipping 760 matching lines @@
 }
 
 void LayerTreeHostImpl::SetDebugState(const LayerTreeDebugState& debug_state) {
   if (debug_state_.continuous_painting != debug_state.continuous_painting)
     paint_time_counter_->ClearHistory();
 
   debug_state_ = debug_state;
 }
 
 }  // namespace cc