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

Side by Side Diff: cc/layers/layer_impl.cc

Issue 1135833002: cc: Ensure pending tree needs prop updates when active tree scrolls. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: test Created 5 years, 7 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
OLDNEW
1 // Copyright 2012 The Chromium Authors. All rights reserved. 1 // Copyright 2012 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/layers/layer_impl.h" 5 #include "cc/layers/layer_impl.h"
6 6
7 #include "base/json/json_reader.h" 7 #include "base/json/json_reader.h"
8 #include "base/strings/stringprintf.h" 8 #include "base/strings/stringprintf.h"
9 #include "base/trace_event/trace_event.h" 9 #include "base/trace_event/trace_event.h"
10 #include "base/trace_event/trace_event_argument.h" 10 #include "base/trace_event/trace_event_argument.h"
(...skipping 1203 matching lines...) Expand 10 before | Expand all | Expand 10 after
1214 1214
1215 if (changed) 1215 if (changed)
1216 DidUpdateScrollOffset(false); 1216 DidUpdateScrollOffset(false);
1217 } 1217 }
1218 1218
1219 void LayerImpl::DidUpdateScrollOffset(bool is_from_root_delegate) { 1219 void LayerImpl::DidUpdateScrollOffset(bool is_from_root_delegate) {
1220 if (!is_from_root_delegate) 1220 if (!is_from_root_delegate)
1221 layer_tree_impl()->DidUpdateScrollOffset(id()); 1221 layer_tree_impl()->DidUpdateScrollOffset(id());
1222 NoteLayerPropertyChangedForSubtree(); 1222 NoteLayerPropertyChangedForSubtree();
1223 ScrollbarParametersDidChange(false); 1223 ScrollbarParametersDidChange(false);
1224 // Inform the pending twin that a property changed.
1225 if (layer_tree_impl()->IsActiveTree()) {
1226 LayerImpl* pending_twin = layer_tree_impl()->FindPendingTreeLayerById(id());
1227 if (pending_twin)
1228 pending_twin->NoteLayerPropertyChangedForSubtree();
1229 }
1224 } 1230 }
1225 1231
1226 void LayerImpl::SetDoubleSided(bool double_sided) { 1232 void LayerImpl::SetDoubleSided(bool double_sided) {
1227 if (double_sided_ == double_sided) 1233 if (double_sided_ == double_sided)
1228 return; 1234 return;
1229 1235
1230 double_sided_ = double_sided; 1236 double_sided_ = double_sided;
1231 NoteLayerPropertyChangedForSubtree(); 1237 NoteLayerPropertyChangedForSubtree();
1232 } 1238 }
1233 1239
(...skipping 401 matching lines...) Expand 10 before | Expand all | Expand 10 after
1635 gfx::Transform scaled_draw_transform = 1641 gfx::Transform scaled_draw_transform =
1636 draw_properties_.target_space_transform; 1642 draw_properties_.target_space_transform;
1637 scaled_draw_transform.Scale(SK_MScalar1 / scale, SK_MScalar1 / scale); 1643 scaled_draw_transform.Scale(SK_MScalar1 / scale, SK_MScalar1 / scale);
1638 gfx::Size scaled_content_bounds = 1644 gfx::Size scaled_content_bounds =
1639 gfx::ToCeiledSize(gfx::ScaleSize(content_bounds(), scale)); 1645 gfx::ToCeiledSize(gfx::ScaleSize(content_bounds(), scale));
1640 return MathUtil::MapEnclosingClippedRect(scaled_draw_transform, 1646 return MathUtil::MapEnclosingClippedRect(scaled_draw_transform,
1641 gfx::Rect(scaled_content_bounds)); 1647 gfx::Rect(scaled_content_bounds));
1642 } 1648 }
1643 1649
1644 } // namespace cc 1650 } // namespace cc
OLDNEW
« no previous file with comments | « no previous file | cc/layers/picture_layer_impl_unittest.cc » ('j') | cc/layers/picture_layer_impl_unittest.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698