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

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

Issue 2735943002: cc: Clarify that color spaces are for raster (Closed)
Patch Set: Rebase (a few times) Created 3 years, 9 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_impl.h ('k') | cc/trees/layer_tree_impl_unittest.cc » ('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 467 matching lines...) Expand 10 before | Expand all | Expand 10 after
478 target_tree->set_top_controls_height(top_controls_height_); 478 target_tree->set_top_controls_height(top_controls_height_);
479 target_tree->set_bottom_controls_height(bottom_controls_height_); 479 target_tree->set_bottom_controls_height(bottom_controls_height_);
480 target_tree->PushBrowserControls(nullptr); 480 target_tree->PushBrowserControls(nullptr);
481 481
482 // Active tree already shares the page_scale_factor object with pending 482 // Active tree already shares the page_scale_factor object with pending
483 // tree so only the limits need to be provided. 483 // tree so only the limits need to be provided.
484 target_tree->PushPageScaleFactorAndLimits(nullptr, min_page_scale_factor(), 484 target_tree->PushPageScaleFactorAndLimits(nullptr, min_page_scale_factor(),
485 max_page_scale_factor()); 485 max_page_scale_factor());
486 target_tree->SetDeviceScaleFactor(device_scale_factor()); 486 target_tree->SetDeviceScaleFactor(device_scale_factor());
487 target_tree->set_painted_device_scale_factor(painted_device_scale_factor()); 487 target_tree->set_painted_device_scale_factor(painted_device_scale_factor());
488 target_tree->SetDeviceColorSpace(device_color_space_); 488 target_tree->SetRasterColorSpace(raster_color_space_);
489 target_tree->elastic_overscroll()->PushPendingToActive(); 489 target_tree->elastic_overscroll()->PushPendingToActive();
490 490
491 target_tree->set_content_source_id(content_source_id()); 491 target_tree->set_content_source_id(content_source_id());
492 492
493 target_tree->pending_page_scale_animation_ = 493 target_tree->pending_page_scale_animation_ =
494 std::move(pending_page_scale_animation_); 494 std::move(pending_page_scale_animation_);
495 495
496 target_tree->SetViewportLayersFromIds( 496 target_tree->SetViewportLayersFromIds(
497 overscroll_elasticity_layer_id_, page_scale_layer_id_, 497 overscroll_elasticity_layer_id_, page_scale_layer_id_,
498 inner_viewport_scroll_layer_id_, outer_viewport_scroll_layer_id_); 498 inner_viewport_scroll_layer_id_, outer_viewport_scroll_layer_id_);
(...skipping 431 matching lines...) Expand 10 before | Expand all | Expand 10 after
930 if (device_scale_factor == device_scale_factor_) 930 if (device_scale_factor == device_scale_factor_)
931 return; 931 return;
932 device_scale_factor_ = device_scale_factor; 932 device_scale_factor_ = device_scale_factor;
933 933
934 set_needs_update_draw_properties(); 934 set_needs_update_draw_properties();
935 if (IsActiveTree()) 935 if (IsActiveTree())
936 layer_tree_host_impl_->SetFullViewportDamage(); 936 layer_tree_host_impl_->SetFullViewportDamage();
937 layer_tree_host_impl_->SetNeedUpdateGpuRasterizationStatus(); 937 layer_tree_host_impl_->SetNeedUpdateGpuRasterizationStatus();
938 } 938 }
939 939
940 void LayerTreeImpl::SetDeviceColorSpace( 940 void LayerTreeImpl::SetRasterColorSpace(
941 const gfx::ColorSpace& device_color_space) { 941 const gfx::ColorSpace& raster_color_space) {
942 if (device_color_space == device_color_space_) 942 if (raster_color_space == raster_color_space_)
943 return; 943 return;
944 device_color_space_ = device_color_space; 944 raster_color_space_ = raster_color_space;
945 } 945 }
946 946
947 SyncedProperty<ScaleGroup>* LayerTreeImpl::page_scale_factor() { 947 SyncedProperty<ScaleGroup>* LayerTreeImpl::page_scale_factor() {
948 return page_scale_factor_.get(); 948 return page_scale_factor_.get();
949 } 949 }
950 950
951 const SyncedProperty<ScaleGroup>* LayerTreeImpl::page_scale_factor() const { 951 const SyncedProperty<ScaleGroup>* LayerTreeImpl::page_scale_factor() const {
952 return page_scale_factor_.get(); 952 return page_scale_factor_.get();
953 } 953 }
954 954
(...skipping 1194 matching lines...) Expand 10 before | Expand all | Expand 10 after
2149 2149
2150 void LayerTreeImpl::ResetAllChangeTracking() { 2150 void LayerTreeImpl::ResetAllChangeTracking() {
2151 layers_that_should_push_properties_.clear(); 2151 layers_that_should_push_properties_.clear();
2152 // Iterate over all layers, including masks. 2152 // Iterate over all layers, including masks.
2153 for (auto& layer : *layers_) 2153 for (auto& layer : *layers_)
2154 layer->ResetChangeTracking(); 2154 layer->ResetChangeTracking();
2155 property_trees_.ResetAllChangeTracking(); 2155 property_trees_.ResetAllChangeTracking();
2156 } 2156 }
2157 2157
2158 } // namespace cc 2158 } // namespace cc
OLDNEW
« no previous file with comments | « cc/trees/layer_tree_impl.h ('k') | cc/trees/layer_tree_impl_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698