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

Unified Diff: components/html_viewer/web_layer_tree_view_impl.cc

Issue 1181013010: Support impl-side painting in Mandoline. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix presubmit warning Created 5 years, 6 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
Index: components/html_viewer/web_layer_tree_view_impl.cc
diff --git a/components/html_viewer/web_layer_tree_view_impl.cc b/components/html_viewer/web_layer_tree_view_impl.cc
index 85cd8aff1aa2bf73aa5c55871d68a1453839ed2c..c47f094b43ac0f2e131caff3757d878f14c4d1c7 100644
--- a/components/html_viewer/web_layer_tree_view_impl.cc
+++ b/components/html_viewer/web_layer_tree_view_impl.cc
@@ -20,6 +20,8 @@ namespace html_viewer {
WebLayerTreeViewImpl::WebLayerTreeViewImpl(
scoped_refptr<base::SingleThreadTaskRunner> compositor_task_runner,
+ gpu::GpuMemoryBufferManager* gpu_memory_buffer_manager,
+ cc::TaskGraphRunner* task_graph_runner,
mojo::SurfacePtr surface,
mojo::GpuPtr gpu_service)
: widget_(NULL),
@@ -30,13 +32,17 @@ WebLayerTreeViewImpl::WebLayerTreeViewImpl(
cc::LayerTreeSettings settings;
+ settings.impl_side_painting = true;
+ settings.use_image_texture_target = GL_TEXTURE_2D;
+ settings.use_one_copy = true;
+ // TODO(jam): use multiple compositor raster threads and set gather_pixel_refs
+ // accordingly (see content).
+
// For web contents, layer transforms should scale up the contents of layers
// to keep content always crisp when possible.
settings.layer_transforms_should_scale_layer_contents = true;
cc::SharedBitmapManager* shared_bitmap_manager = nullptr;
- gpu::GpuMemoryBufferManager* gpu_memory_buffer_manager = nullptr;
- cc::TaskGraphRunner* task_graph_runner = nullptr;
cc::LayerTreeHost::InitParams params;
params.client = this;

Powered by Google App Engine
This is Rietveld 408576698