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

Side by Side Diff: cc/trees/layer_tree_impl.cc

Issue 2693703010: cc: Remove support for disabling non-root render surfaces. (Closed)
Patch Set: softwaredraw-remove-no-surfaces: rebase Created 3 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 unified diff | Download patch
« no previous file with comments | « cc/trees/layer_tree_host_common_unittest.cc ('k') | cc/trees/property_tree.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2011 The Chromium Authors. All rights reserved. 1 // Copyright 2011 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 "cc/trees/layer_tree_impl.h" 5 #include "cc/trees/layer_tree_impl.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include <algorithm> 10 #include <algorithm>
(...skipping 969 matching lines...) Expand 10 before | Expand all | Expand 10 after
980 render_surface_layer_list_.clear(); 980 render_surface_layer_list_.clear();
981 981
982 if (layer_list_.empty()) 982 if (layer_list_.empty())
983 return false; 983 return false;
984 984
985 { 985 {
986 base::ElapsedTimer timer; 986 base::ElapsedTimer timer;
987 TRACE_EVENT2( 987 TRACE_EVENT2(
988 "cc", "LayerTreeImpl::UpdateDrawProperties::CalculateDrawProperties", 988 "cc", "LayerTreeImpl::UpdateDrawProperties::CalculateDrawProperties",
989 "IsActive", IsActiveTree(), "SourceFrameNumber", source_frame_number_); 989 "IsActive", IsActiveTree(), "SourceFrameNumber", source_frame_number_);
990 // TODO(crbug.com/692780): Remove this option entirely once this get to
991 // stable and proves it works.
992 bool can_render_to_separate_surface = true;
993
994 // We verify visible rect calculations whenever we verify clip tree 990 // We verify visible rect calculations whenever we verify clip tree
995 // calculations except when this function is explicitly passed a flag asking 991 // calculations except when this function is explicitly passed a flag asking
996 // us to skip it. 992 // us to skip it.
997 bool verify_visible_rect_calculations = 993 bool verify_visible_rect_calculations =
998 force_skip_verify_visible_rect_calculations 994 force_skip_verify_visible_rect_calculations
999 ? false 995 ? false
1000 : settings().verify_clip_tree_calculations; 996 : settings().verify_clip_tree_calculations;
1001 LayerTreeHostCommon::CalcDrawPropsImplInputs inputs( 997 LayerTreeHostCommon::CalcDrawPropsImplInputs inputs(
1002 layer_list_[0], DrawViewportSize(), 998 layer_list_[0], DrawViewportSize(),
1003 layer_tree_host_impl_->DrawTransform(), device_scale_factor(), 999 layer_tree_host_impl_->DrawTransform(), device_scale_factor(),
1004 current_page_scale_factor(), PageScaleLayer(), 1000 current_page_scale_factor(), PageScaleLayer(),
1005 InnerViewportScrollLayer(), OuterViewportScrollLayer(), 1001 InnerViewportScrollLayer(), OuterViewportScrollLayer(),
1006 elastic_overscroll()->Current(IsActiveTree()), 1002 elastic_overscroll()->Current(IsActiveTree()),
1007 OverscrollElasticityLayer(), resource_provider()->max_texture_size(), 1003 OverscrollElasticityLayer(), resource_provider()->max_texture_size(),
1008 can_render_to_separate_surface,
1009 settings().layer_transforms_should_scale_layer_contents, 1004 settings().layer_transforms_should_scale_layer_contents,
1010 settings().verify_clip_tree_calculations, 1005 settings().verify_clip_tree_calculations,
1011 verify_visible_rect_calculations, 1006 verify_visible_rect_calculations,
1012 &render_surface_layer_list_, &property_trees_); 1007 &render_surface_layer_list_, &property_trees_);
1013 LayerTreeHostCommon::CalculateDrawProperties(&inputs); 1008 LayerTreeHostCommon::CalculateDrawProperties(&inputs);
1014 if (const char* client_name = GetClientNameForMetrics()) { 1009 if (const char* client_name = GetClientNameForMetrics()) {
1015 UMA_HISTOGRAM_COUNTS( 1010 UMA_HISTOGRAM_COUNTS(
1016 base::StringPrintf( 1011 base::StringPrintf(
1017 "Compositing.%s.LayerTreeImpl.CalculateDrawPropertiesUs", 1012 "Compositing.%s.LayerTreeImpl.CalculateDrawPropertiesUs",
1018 client_name), 1013 client_name),
(...skipping 1075 matching lines...) Expand 10 before | Expand all | Expand 10 after
2094 2089
2095 void LayerTreeImpl::ResetAllChangeTracking() { 2090 void LayerTreeImpl::ResetAllChangeTracking() {
2096 layers_that_should_push_properties_.clear(); 2091 layers_that_should_push_properties_.clear();
2097 // Iterate over all layers, including masks. 2092 // Iterate over all layers, including masks.
2098 for (auto& layer : *layers_) 2093 for (auto& layer : *layers_)
2099 layer->ResetChangeTracking(); 2094 layer->ResetChangeTracking();
2100 property_trees_.ResetAllChangeTracking(); 2095 property_trees_.ResetAllChangeTracking();
2101 } 2096 }
2102 2097
2103 } // namespace cc 2098 } // namespace cc
OLDNEW
« no previous file with comments | « cc/trees/layer_tree_host_common_unittest.cc ('k') | cc/trees/property_tree.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698