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

Unified Diff: cc/test/fake_layer_tree_host.cc

Issue 2251143002: cc: Reland Move data to LayerTree from LayerTreeHost. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: better fix? Created 4 years, 4 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: cc/test/fake_layer_tree_host.cc
diff --git a/cc/test/fake_layer_tree_host.cc b/cc/test/fake_layer_tree_host.cc
index 9d3ae6fd2eb717d3f3a5d9733e916074ef4efca9..8b885b1a886b5f00a9bd1c0b9afb2c6b701cb346 100644
--- a/cc/test/fake_layer_tree_host.cc
+++ b/cc/test/fake_layer_tree_host.cc
@@ -9,12 +9,31 @@
#include "cc/layers/layer.h"
#include "cc/test/fake_image_serialization_processor.h"
#include "cc/test/test_task_graph_runner.h"
+#include "cc/trees/layer_tree.h"
namespace cc {
+
+namespace {
+
+class FakeLayerTree : public LayerTree {
+ public:
+ FakeLayerTree(std::unique_ptr<AnimationHost> animation_host,
vmpstr 2016/08/17 20:28:15 minor nit: this could be templatized with a parame
+ LayerTreeHost* layer_tree_host)
+ : LayerTree(std::move(animation_host), layer_tree_host) {}
+
+ void SetNeedsFullTreeSync() override {}
+};
+
+} // namespace
+
FakeLayerTreeHost::FakeLayerTreeHost(FakeLayerTreeHostClient* client,
LayerTreeHost::InitParams* params,
CompositorMode mode)
- : LayerTreeHost(params, mode),
+ : LayerTreeHost(
+ params,
+ mode,
+ base::MakeUnique<FakeLayerTree>(std::move(params->animation_host),
+ this)),
client_(client),
host_impl_(*params->settings,
&task_runner_provider_,
@@ -96,13 +115,17 @@ LayerImpl* FakeLayerTreeHost::CommitAndCreateLayerImplTree() {
active_tree()->UpdatePropertyTreeScrollOffset(property_trees());
- if (page_scale_layer() && inner_viewport_scroll_layer()) {
+ if (layer_tree_->page_scale_layer() &&
+ layer_tree_->inner_viewport_scroll_layer()) {
active_tree()->SetViewportLayersFromIds(
- overscroll_elasticity_layer() ? overscroll_elasticity_layer()->id()
- : Layer::INVALID_ID,
- page_scale_layer()->id(), inner_viewport_scroll_layer()->id(),
- outer_viewport_scroll_layer() ? outer_viewport_scroll_layer()->id()
- : Layer::INVALID_ID);
+ layer_tree_->overscroll_elasticity_layer()
+ ? layer_tree_->overscroll_elasticity_layer()->id()
+ : Layer::INVALID_ID,
+ layer_tree_->page_scale_layer()->id(),
+ layer_tree_->inner_viewport_scroll_layer()->id(),
+ layer_tree_->outer_viewport_scroll_layer()
+ ? layer_tree_->outer_viewport_scroll_layer()->id()
+ : Layer::INVALID_ID);
}
active_tree()->UpdatePropertyTreesForBoundsDelta();

Powered by Google App Engine
This is Rietveld 408576698