Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4217)

Unified Diff: cc/trees/layer_tree_host_impl.cc

Issue 157743007: cc: Prevent usage of rasterize on-demand with delegating renderer. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: one more unit test Created 6 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « cc/trees/layer_tree_host_impl.h ('k') | cc/trees/layer_tree_host_impl_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/trees/layer_tree_host_impl.cc
diff --git a/cc/trees/layer_tree_host_impl.cc b/cc/trees/layer_tree_host_impl.cc
index 6829156565e085b73fe85fd9cbdc9942ca8303fe..84d9f892e52b65bcb1733690778ca64017e97762 100644
--- a/cc/trees/layer_tree_host_impl.cc
+++ b/cc/trees/layer_tree_host_impl.cc
@@ -1744,7 +1744,8 @@ void LayerTreeHostImpl::CreateAndSetRenderer(
void LayerTreeHostImpl::CreateAndSetTileManager(
ResourceProvider* resource_provider,
ContextProvider* context_provider,
- bool using_map_image) {
+ bool using_map_image,
+ bool allow_rasterize_on_demand) {
DCHECK(settings_.impl_side_painting);
DCHECK(resource_provider);
tile_manager_ =
@@ -1753,6 +1754,7 @@ void LayerTreeHostImpl::CreateAndSetTileManager(
context_provider,
rendering_stats_instrumentation_,
using_map_image,
+ allow_rasterize_on_demand,
GetMaxTransferBufferUsageBytes(context_provider),
GetMaxRasterTasksUsageBytes(context_provider),
GetMapImageTextureTarget(context_provider));
@@ -1805,9 +1807,11 @@ bool LayerTreeHostImpl::InitializeRenderer(
return false;
if (settings_.impl_side_painting) {
- CreateAndSetTileManager(resource_provider.get(),
- output_surface->context_provider().get(),
- GetRendererCapabilities().using_map_image);
+ CreateAndSetTileManager(
+ resource_provider.get(),
+ output_surface->context_provider().get(),
+ GetRendererCapabilities().using_map_image,
+ GetRendererCapabilities().allow_rasterize_on_demand);
}
// Setup BeginImplFrameEmulation if it's not supported natively
@@ -1912,7 +1916,8 @@ void LayerTreeHostImpl::ReleaseGL() {
EnforceZeroBudget(true);
CreateAndSetTileManager(resource_provider_.get(),
NULL,
- GetRendererCapabilities().using_map_image);
+ GetRendererCapabilities().using_map_image,
+ GetRendererCapabilities().allow_rasterize_on_demand);
DCHECK(tile_manager_);
SetOffscreenContextProvider(NULL);
« no previous file with comments | « cc/trees/layer_tree_host_impl.h ('k') | cc/trees/layer_tree_host_impl_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698