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

Side by Side Diff: android_webview/browser/hardware_renderer.cc

Issue 134623005: Make SingleThreadProxy a SchedulerClient (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Mac browser compositor fixes Created 6 years, 5 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "android_webview/browser/hardware_renderer.h" 5 #include "android_webview/browser/hardware_renderer.h"
6 6
7 #include "android_webview/browser/aw_gl_surface.h" 7 #include "android_webview/browser/aw_gl_surface.h"
8 #include "android_webview/browser/deferred_gpu_command_service.h" 8 #include "android_webview/browser/deferred_gpu_command_service.h"
9 #include "android_webview/browser/parent_output_surface.h" 9 #include "android_webview/browser/parent_output_surface.h"
10 #include "android_webview/browser/shared_renderer_state.h" 10 #include "android_webview/browser/shared_renderer_state.h"
(...skipping 73 matching lines...) Expand 10 before | Expand all | Expand 10 after
84 84
85 cc::LayerTreeSettings settings; 85 cc::LayerTreeSettings settings;
86 86
87 // Should be kept in sync with compositor_impl_android.cc. 87 // Should be kept in sync with compositor_impl_android.cc.
88 settings.allow_antialiasing = false; 88 settings.allow_antialiasing = false;
89 settings.highp_threshold_min = 2048; 89 settings.highp_threshold_min = 2048;
90 90
91 // Webview does not own the surface so should not clear it. 91 // Webview does not own the surface so should not clear it.
92 settings.should_clear_root_render_pass = false; 92 settings.should_clear_root_render_pass = false;
93 93
94 layer_tree_host_ = cc::LayerTreeHost::CreateSingleThreaded( 94 // TODO(enne): Update this this compositor to use a synchronous scheduler.
95 this, this, NULL, settings, NULL); 95 settings.single_thread_proxy_scheduler = false;
96
97 layer_tree_host_ =
98 cc::LayerTreeHost::CreateSingleThreaded(this, this, NULL, settings, NULL);
96 layer_tree_host_->SetRootLayer(root_layer_); 99 layer_tree_host_->SetRootLayer(root_layer_);
97 layer_tree_host_->SetLayerTreeHostClientReady(); 100 layer_tree_host_->SetLayerTreeHostClientReady();
98 layer_tree_host_->set_has_transparent_background(true); 101 layer_tree_host_->set_has_transparent_background(true);
99 } 102 }
100 103
101 HardwareRenderer::~HardwareRenderer() { 104 HardwareRenderer::~HardwareRenderer() {
102 // Must reset everything before |resource_collection_| to ensure all 105 // Must reset everything before |resource_collection_| to ensure all
103 // resources are returned before resetting |resource_collection_| client. 106 // resources are returned before resetting |resource_collection_| client.
104 layer_tree_host_.reset(); 107 layer_tree_host_.reset();
105 root_layer_ = NULL; 108 root_layer_ = NULL;
(...skipping 118 matching lines...) Expand 10 before | Expand all | Expand 10 after
224 } 227 }
225 228
226 void HardwareRenderer::UnusedResourcesAreAvailable() { 229 void HardwareRenderer::UnusedResourcesAreAvailable() {
227 cc::ReturnedResourceArray returned_resources; 230 cc::ReturnedResourceArray returned_resources;
228 resource_collection_->TakeUnusedResourcesForChildCompositor( 231 resource_collection_->TakeUnusedResourcesForChildCompositor(
229 &returned_resources); 232 &returned_resources);
230 shared_renderer_state_->InsertReturnedResources(returned_resources); 233 shared_renderer_state_->InsertReturnedResources(returned_resources);
231 } 234 }
232 235
233 } // namespace android_webview 236 } // namespace android_webview
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698