summaryrefslogtreecommitdiffstats
path: root/cc/tree_synchronizer_unittest.cc
diff options
context:
space:
mode:
authorenne@chromium.org <enne@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2012-10-19 20:58:10 +0000
committerenne@chromium.org <enne@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2012-10-19 20:58:10 +0000
commit184fac5add06d5f4670fa6be4a4d3b39232d51bb (patch)
tree7efc1d20146fcafdbe408fa67292117c41e9393f /cc/tree_synchronizer_unittest.cc
parentfb7ffba75e81a79436434bab0715be0d6e67bc07 (diff)
downloadchromium_src-184fac5add06d5f4670fa6be4a4d3b39232d51bb.zip
chromium_src-184fac5add06d5f4670fa6be4a4d3b39232d51bb.tar.gz
chromium_src-184fac5add06d5f4670fa6be4a4d3b39232d51bb.tar.bz2
cc: Rename cc classes and members to match filenames
BUG=155413 Review URL: https://codereview.chromium.org/11189043 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@163052 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'cc/tree_synchronizer_unittest.cc')
-rw-r--r--cc/tree_synchronizer_unittest.cc316
1 files changed, 158 insertions, 158 deletions
diff --git a/cc/tree_synchronizer_unittest.cc b/cc/tree_synchronizer_unittest.cc
index 288cc20..8344704 100644
--- a/cc/tree_synchronizer_unittest.cc
+++ b/cc/tree_synchronizer_unittest.cc
@@ -20,64 +20,64 @@ using namespace WebKitTests;
namespace {
-class MockCCLayerImpl : public CCLayerImpl {
+class MockLayerImpl : public LayerImpl {
public:
- static scoped_ptr<MockCCLayerImpl> create(int layerId)
+ static scoped_ptr<MockLayerImpl> create(int layerId)
{
- return make_scoped_ptr(new MockCCLayerImpl(layerId));
+ return make_scoped_ptr(new MockLayerImpl(layerId));
}
- virtual ~MockCCLayerImpl()
+ virtual ~MockLayerImpl()
{
- if (m_ccLayerDestructionList)
- m_ccLayerDestructionList->append(id());
+ if (m_layerImplDestructionList)
+ m_layerImplDestructionList->append(id());
}
- void setCCLayerDestructionList(Vector<int>* list) { m_ccLayerDestructionList = list; }
+ void setLayerImplDestructionList(Vector<int>* list) { m_layerImplDestructionList = list; }
private:
- MockCCLayerImpl(int layerId)
- : CCLayerImpl(layerId)
- , m_ccLayerDestructionList(0)
+ MockLayerImpl(int layerId)
+ : LayerImpl(layerId)
+ , m_layerImplDestructionList(0)
{
}
- Vector<int>* m_ccLayerDestructionList;
+ Vector<int>* m_layerImplDestructionList;
};
-class MockLayerChromium : public LayerChromium {
+class MockLayer : public Layer {
public:
- static scoped_refptr<MockLayerChromium> create(Vector<int>* ccLayerDestructionList)
+ static scoped_refptr<MockLayer> create(Vector<int>* layerImplDestructionList)
{
- return make_scoped_refptr(new MockLayerChromium(ccLayerDestructionList));
+ return make_scoped_refptr(new MockLayer(layerImplDestructionList));
}
- virtual scoped_ptr<CCLayerImpl> createCCLayerImpl() OVERRIDE
+ virtual scoped_ptr<LayerImpl> createLayerImpl() OVERRIDE
{
- return MockCCLayerImpl::create(m_layerId).PassAs<CCLayerImpl>();
+ return MockLayerImpl::create(m_layerId).PassAs<LayerImpl>();
}
- virtual void pushPropertiesTo(CCLayerImpl* ccLayer) OVERRIDE
+ virtual void pushPropertiesTo(LayerImpl* layerImpl) OVERRIDE
{
- LayerChromium::pushPropertiesTo(ccLayer);
+ Layer::pushPropertiesTo(layerImpl);
- MockCCLayerImpl* mockCCLayer = static_cast<MockCCLayerImpl*>(ccLayer);
- mockCCLayer->setCCLayerDestructionList(m_ccLayerDestructionList);
+ MockLayerImpl* mockLayerImpl = static_cast<MockLayerImpl*>(layerImpl);
+ mockLayerImpl->setLayerImplDestructionList(m_layerImplDestructionList);
}
private:
- MockLayerChromium(Vector<int>* ccLayerDestructionList)
- : LayerChromium()
- , m_ccLayerDestructionList(ccLayerDestructionList)
+ MockLayer(Vector<int>* layerImplDestructionList)
+ : Layer()
+ , m_layerImplDestructionList(layerImplDestructionList)
{
}
- virtual ~MockLayerChromium() { }
+ virtual ~MockLayer() { }
- Vector<int>* m_ccLayerDestructionList;
+ Vector<int>* m_layerImplDestructionList;
};
-class FakeLayerAnimationController : public CCLayerAnimationController {
+class FakeLayerAnimationController : public LayerAnimationController {
public:
- static scoped_ptr<FakeLayerAnimationController> create(CCLayerAnimationControllerClient* client)
+ static scoped_ptr<FakeLayerAnimationController> create(LayerAnimationControllerClient* client)
{
return make_scoped_ptr(new FakeLayerAnimationController(client));
}
@@ -85,46 +85,46 @@ public:
bool synchronizedAnimations() const { return m_synchronizedAnimations; }
private:
- explicit FakeLayerAnimationController(CCLayerAnimationControllerClient* client)
- : CCLayerAnimationController(client)
+ explicit FakeLayerAnimationController(LayerAnimationControllerClient* client)
+ : LayerAnimationController(client)
, m_synchronizedAnimations(false)
{
}
- virtual void pushAnimationUpdatesTo(CCLayerAnimationController* controllerImpl)
+ virtual void pushAnimationUpdatesTo(LayerAnimationController* controllerImpl)
{
- CCLayerAnimationController::pushAnimationUpdatesTo(controllerImpl);
+ LayerAnimationController::pushAnimationUpdatesTo(controllerImpl);
m_synchronizedAnimations = true;
}
bool m_synchronizedAnimations;
};
-void expectTreesAreIdentical(LayerChromium* layer, CCLayerImpl* ccLayer, CCLayerTreeHostImpl* hostImpl)
+void expectTreesAreIdentical(Layer* layer, LayerImpl* layerImpl, LayerTreeHostImpl* hostImpl)
{
ASSERT_TRUE(layer);
- ASSERT_TRUE(ccLayer);
+ ASSERT_TRUE(layerImpl);
- EXPECT_EQ(layer->id(), ccLayer->id());
- EXPECT_EQ(ccLayer->layerTreeHostImpl(), hostImpl);
+ EXPECT_EQ(layer->id(), layerImpl->id());
+ EXPECT_EQ(layerImpl->layerTreeHostImpl(), hostImpl);
- EXPECT_EQ(layer->nonFastScrollableRegion(), ccLayer->nonFastScrollableRegion());
+ EXPECT_EQ(layer->nonFastScrollableRegion(), layerImpl->nonFastScrollableRegion());
- ASSERT_EQ(!!layer->maskLayer(), !!ccLayer->maskLayer());
+ ASSERT_EQ(!!layer->maskLayer(), !!layerImpl->maskLayer());
if (layer->maskLayer())
- expectTreesAreIdentical(layer->maskLayer(), ccLayer->maskLayer(), hostImpl);
+ expectTreesAreIdentical(layer->maskLayer(), layerImpl->maskLayer(), hostImpl);
- ASSERT_EQ(!!layer->replicaLayer(), !!ccLayer->replicaLayer());
+ ASSERT_EQ(!!layer->replicaLayer(), !!layerImpl->replicaLayer());
if (layer->replicaLayer())
- expectTreesAreIdentical(layer->replicaLayer(), ccLayer->replicaLayer(), hostImpl);
+ expectTreesAreIdentical(layer->replicaLayer(), layerImpl->replicaLayer(), hostImpl);
- const std::vector<scoped_refptr<LayerChromium> >& layerChildren = layer->children();
- const ScopedPtrVector<CCLayerImpl>& ccLayerChildren = ccLayer->children();
+ const std::vector<scoped_refptr<Layer> >& layerChildren = layer->children();
+ const ScopedPtrVector<LayerImpl>& layerImplChildren = layerImpl->children();
- ASSERT_EQ(layerChildren.size(), ccLayerChildren.size());
+ ASSERT_EQ(layerChildren.size(), layerImplChildren.size());
for (size_t i = 0; i < layerChildren.size(); ++i)
- expectTreesAreIdentical(layerChildren[i].get(), ccLayerChildren[i], hostImpl);
+ expectTreesAreIdentical(layerChildren[i].get(), layerImplChildren[i], hostImpl);
}
// Attempts to synchronizes a null tree. This should not crash, and should
@@ -133,9 +133,9 @@ TEST(TreeSynchronizerTest, syncNullTree)
{
DebugScopedSetImplThread impl;
- scoped_ptr<CCLayerImpl> ccLayerTreeRoot = TreeSynchronizer::synchronizeTrees(0, scoped_ptr<CCLayerImpl>(), 0);
+ scoped_ptr<LayerImpl> layerImplTreeRoot = TreeSynchronizer::synchronizeTrees(0, scoped_ptr<LayerImpl>(), 0);
- EXPECT_TRUE(!ccLayerTreeRoot.get());
+ EXPECT_TRUE(!layerImplTreeRoot.get());
}
// Constructs a very simple tree and synchronizes it without trying to reuse any preexisting layers.
@@ -143,89 +143,89 @@ TEST(TreeSynchronizerTest, syncSimpleTreeFromEmpty)
{
DebugScopedSetImplThread impl;
- CCLayerTreeSettings settings;
- scoped_ptr<CCLayerTreeHostImpl> hostImpl = CCLayerTreeHostImpl::create(settings, 0);
+ LayerTreeSettings settings;
+ scoped_ptr<LayerTreeHostImpl> hostImpl = LayerTreeHostImpl::create(settings, 0);
- scoped_refptr<LayerChromium> layerTreeRoot = LayerChromium::create();
- layerTreeRoot->addChild(LayerChromium::create());
- layerTreeRoot->addChild(LayerChromium::create());
+ scoped_refptr<Layer> layerTreeRoot = Layer::create();
+ layerTreeRoot->addChild(Layer::create());
+ layerTreeRoot->addChild(Layer::create());
- scoped_ptr<CCLayerImpl> ccLayerTreeRoot = TreeSynchronizer::synchronizeTrees(layerTreeRoot.get(), scoped_ptr<CCLayerImpl>(), hostImpl.get());
+ scoped_ptr<LayerImpl> layerImplTreeRoot = TreeSynchronizer::synchronizeTrees(layerTreeRoot.get(), scoped_ptr<LayerImpl>(), hostImpl.get());
- expectTreesAreIdentical(layerTreeRoot.get(), ccLayerTreeRoot.get(), hostImpl.get());
+ expectTreesAreIdentical(layerTreeRoot.get(), layerImplTreeRoot.get(), hostImpl.get());
}
// Constructs a very simple tree and synchronizes it attempting to reuse some layers
TEST(TreeSynchronizerTest, syncSimpleTreeReusingLayers)
{
DebugScopedSetImplThread impl;
- Vector<int> ccLayerDestructionList;
+ Vector<int> layerImplDestructionList;
- CCLayerTreeSettings settings;
- scoped_ptr<CCLayerTreeHostImpl> hostImpl = CCLayerTreeHostImpl::create(settings, 0);
+ LayerTreeSettings settings;
+ scoped_ptr<LayerTreeHostImpl> hostImpl = LayerTreeHostImpl::create(settings, 0);
- scoped_refptr<LayerChromium> layerTreeRoot = MockLayerChromium::create(&ccLayerDestructionList);
- layerTreeRoot->addChild(MockLayerChromium::create(&ccLayerDestructionList));
- layerTreeRoot->addChild(MockLayerChromium::create(&ccLayerDestructionList));
+ scoped_refptr<Layer> layerTreeRoot = MockLayer::create(&layerImplDestructionList);
+ layerTreeRoot->addChild(MockLayer::create(&layerImplDestructionList));
+ layerTreeRoot->addChild(MockLayer::create(&layerImplDestructionList));
- scoped_ptr<CCLayerImpl> ccLayerTreeRoot = TreeSynchronizer::synchronizeTrees(layerTreeRoot.get(), scoped_ptr<CCLayerImpl>(), hostImpl.get());
- expectTreesAreIdentical(layerTreeRoot.get(), ccLayerTreeRoot.get(), hostImpl.get());
+ scoped_ptr<LayerImpl> layerImplTreeRoot = TreeSynchronizer::synchronizeTrees(layerTreeRoot.get(), scoped_ptr<LayerImpl>(), hostImpl.get());
+ expectTreesAreIdentical(layerTreeRoot.get(), layerImplTreeRoot.get(), hostImpl.get());
- // Add a new layer to the LayerChromium side
- layerTreeRoot->children()[0]->addChild(MockLayerChromium::create(&ccLayerDestructionList));
+ // Add a new layer to the Layer side
+ layerTreeRoot->children()[0]->addChild(MockLayer::create(&layerImplDestructionList));
// Remove one.
layerTreeRoot->children()[1]->removeFromParent();
- int secondCCLayerId = ccLayerTreeRoot->children()[1]->id();
+ int secondLayerImplId = layerImplTreeRoot->children()[1]->id();
- // Synchronize again. After the sync the trees should be equivalent and we should have created and destroyed one CCLayerImpl.
- ccLayerTreeRoot = TreeSynchronizer::synchronizeTrees(layerTreeRoot.get(), ccLayerTreeRoot.Pass(), hostImpl.get());
- expectTreesAreIdentical(layerTreeRoot.get(), ccLayerTreeRoot.get(), hostImpl.get());
+ // Synchronize again. After the sync the trees should be equivalent and we should have created and destroyed one LayerImpl.
+ layerImplTreeRoot = TreeSynchronizer::synchronizeTrees(layerTreeRoot.get(), layerImplTreeRoot.Pass(), hostImpl.get());
+ expectTreesAreIdentical(layerTreeRoot.get(), layerImplTreeRoot.get(), hostImpl.get());
- ASSERT_EQ(1u, ccLayerDestructionList.size());
- EXPECT_EQ(secondCCLayerId, ccLayerDestructionList[0]);
+ ASSERT_EQ(1u, layerImplDestructionList.size());
+ EXPECT_EQ(secondLayerImplId, layerImplDestructionList[0]);
}
// Constructs a very simple tree and checks that a stacking-order change is tracked properly.
TEST(TreeSynchronizerTest, syncSimpleTreeAndTrackStackingOrderChange)
{
DebugScopedSetImplThread impl;
- Vector<int> ccLayerDestructionList;
+ Vector<int> layerImplDestructionList;
- CCLayerTreeSettings settings;
- scoped_ptr<CCLayerTreeHostImpl> hostImpl = CCLayerTreeHostImpl::create(settings, 0);
+ LayerTreeSettings settings;
+ scoped_ptr<LayerTreeHostImpl> hostImpl = LayerTreeHostImpl::create(settings, 0);
// Set up the tree and sync once. child2 needs to be synced here, too, even though we
// remove it to set up the intended scenario.
- scoped_refptr<LayerChromium> layerTreeRoot = MockLayerChromium::create(&ccLayerDestructionList);
- scoped_refptr<LayerChromium> child2 = MockLayerChromium::create(&ccLayerDestructionList);
- layerTreeRoot->addChild(MockLayerChromium::create(&ccLayerDestructionList));
+ scoped_refptr<Layer> layerTreeRoot = MockLayer::create(&layerImplDestructionList);
+ scoped_refptr<Layer> child2 = MockLayer::create(&layerImplDestructionList);
+ layerTreeRoot->addChild(MockLayer::create(&layerImplDestructionList));
layerTreeRoot->addChild(child2);
- scoped_ptr<CCLayerImpl> ccLayerTreeRoot = TreeSynchronizer::synchronizeTrees(layerTreeRoot.get(), scoped_ptr<CCLayerImpl>(), hostImpl.get());
- expectTreesAreIdentical(layerTreeRoot.get(), ccLayerTreeRoot.get(), hostImpl.get());
- ccLayerTreeRoot->resetAllChangeTrackingForSubtree();
+ scoped_ptr<LayerImpl> layerImplTreeRoot = TreeSynchronizer::synchronizeTrees(layerTreeRoot.get(), scoped_ptr<LayerImpl>(), hostImpl.get());
+ expectTreesAreIdentical(layerTreeRoot.get(), layerImplTreeRoot.get(), hostImpl.get());
+ layerImplTreeRoot->resetAllChangeTrackingForSubtree();
// re-insert the layer and sync again.
child2->removeFromParent();
layerTreeRoot->addChild(child2);
- ccLayerTreeRoot = TreeSynchronizer::synchronizeTrees(layerTreeRoot.get(), ccLayerTreeRoot.Pass(), hostImpl.get());
- expectTreesAreIdentical(layerTreeRoot.get(), ccLayerTreeRoot.get(), hostImpl.get());
+ layerImplTreeRoot = TreeSynchronizer::synchronizeTrees(layerTreeRoot.get(), layerImplTreeRoot.Pass(), hostImpl.get());
+ expectTreesAreIdentical(layerTreeRoot.get(), layerImplTreeRoot.get(), hostImpl.get());
// Check that the impl thread properly tracked the change.
- EXPECT_FALSE(ccLayerTreeRoot->layerPropertyChanged());
- EXPECT_FALSE(ccLayerTreeRoot->children()[0]->layerPropertyChanged());
- EXPECT_TRUE(ccLayerTreeRoot->children()[1]->layerPropertyChanged());
+ EXPECT_FALSE(layerImplTreeRoot->layerPropertyChanged());
+ EXPECT_FALSE(layerImplTreeRoot->children()[0]->layerPropertyChanged());
+ EXPECT_TRUE(layerImplTreeRoot->children()[1]->layerPropertyChanged());
}
TEST(TreeSynchronizerTest, syncSimpleTreeAndProperties)
{
DebugScopedSetImplThread impl;
- CCLayerTreeSettings settings;
- scoped_ptr<CCLayerTreeHostImpl> hostImpl = CCLayerTreeHostImpl::create(settings, 0);
+ LayerTreeSettings settings;
+ scoped_ptr<LayerTreeHostImpl> hostImpl = LayerTreeHostImpl::create(settings, 0);
- scoped_refptr<LayerChromium> layerTreeRoot = LayerChromium::create();
- layerTreeRoot->addChild(LayerChromium::create());
- layerTreeRoot->addChild(LayerChromium::create());
+ scoped_refptr<Layer> layerTreeRoot = Layer::create();
+ layerTreeRoot->addChild(Layer::create());
+ layerTreeRoot->addChild(Layer::create());
// Pick some random properties to set. The values are not important, we're just testing that at least some properties are making it through.
FloatPoint rootPosition = FloatPoint(2.3f, 7.4f);
@@ -237,48 +237,48 @@ TEST(TreeSynchronizerTest, syncSimpleTreeAndProperties)
IntSize secondChildBounds = IntSize(25, 53);
layerTreeRoot->children()[1]->setBounds(secondChildBounds);
- scoped_ptr<CCLayerImpl> ccLayerTreeRoot = TreeSynchronizer::synchronizeTrees(layerTreeRoot.get(), scoped_ptr<CCLayerImpl>(), hostImpl.get());
- expectTreesAreIdentical(layerTreeRoot.get(), ccLayerTreeRoot.get(), hostImpl.get());
+ scoped_ptr<LayerImpl> layerImplTreeRoot = TreeSynchronizer::synchronizeTrees(layerTreeRoot.get(), scoped_ptr<LayerImpl>(), hostImpl.get());
+ expectTreesAreIdentical(layerTreeRoot.get(), layerImplTreeRoot.get(), hostImpl.get());
- // Check that the property values we set on the LayerChromium tree are reflected in the CCLayerImpl tree.
- FloatPoint rootCCLayerPosition = ccLayerTreeRoot->position();
- EXPECT_EQ(rootPosition.x(), rootCCLayerPosition.x());
- EXPECT_EQ(rootPosition.y(), rootCCLayerPosition.y());
+ // Check that the property values we set on the Layer tree are reflected in the LayerImpl tree.
+ FloatPoint rootLayerImplPosition = layerImplTreeRoot->position();
+ EXPECT_EQ(rootPosition.x(), rootLayerImplPosition.x());
+ EXPECT_EQ(rootPosition.y(), rootLayerImplPosition.y());
- EXPECT_EQ(firstChildOpacity, ccLayerTreeRoot->children()[0]->opacity());
+ EXPECT_EQ(firstChildOpacity, layerImplTreeRoot->children()[0]->opacity());
- IntSize secondCCLayerChildBounds = ccLayerTreeRoot->children()[1]->bounds();
- EXPECT_EQ(secondChildBounds.width(), secondCCLayerChildBounds.width());
- EXPECT_EQ(secondChildBounds.height(), secondCCLayerChildBounds.height());
+ IntSize secondLayerImplChildBounds = layerImplTreeRoot->children()[1]->bounds();
+ EXPECT_EQ(secondChildBounds.width(), secondLayerImplChildBounds.width());
+ EXPECT_EQ(secondChildBounds.height(), secondLayerImplChildBounds.height());
}
-TEST(TreeSynchronizerTest, reuseCCLayersAfterStructuralChange)
+TEST(TreeSynchronizerTest, reuseLayerImplsAfterStructuralChange)
{
DebugScopedSetImplThread impl;
- Vector<int> ccLayerDestructionList;
+ Vector<int> layerImplDestructionList;
- CCLayerTreeSettings settings;
- scoped_ptr<CCLayerTreeHostImpl> hostImpl = CCLayerTreeHostImpl::create(settings, 0);
+ LayerTreeSettings settings;
+ scoped_ptr<LayerTreeHostImpl> hostImpl = LayerTreeHostImpl::create(settings, 0);
// Set up a tree with this sort of structure:
// root --- A --- B ---+--- C
// |
// +--- D
- scoped_refptr<LayerChromium> layerTreeRoot = MockLayerChromium::create(&ccLayerDestructionList);
- layerTreeRoot->addChild(MockLayerChromium::create(&ccLayerDestructionList));
+ scoped_refptr<Layer> layerTreeRoot = MockLayer::create(&layerImplDestructionList);
+ layerTreeRoot->addChild(MockLayer::create(&layerImplDestructionList));
- scoped_refptr<LayerChromium> layerA = layerTreeRoot->children()[0].get();
- layerA->addChild(MockLayerChromium::create(&ccLayerDestructionList));
+ scoped_refptr<Layer> layerA = layerTreeRoot->children()[0].get();
+ layerA->addChild(MockLayer::create(&layerImplDestructionList));
- scoped_refptr<LayerChromium> layerB = layerA->children()[0].get();
- layerB->addChild(MockLayerChromium::create(&ccLayerDestructionList));
+ scoped_refptr<Layer> layerB = layerA->children()[0].get();
+ layerB->addChild(MockLayer::create(&layerImplDestructionList));
- scoped_refptr<LayerChromium> layerC = layerB->children()[0].get();
- layerB->addChild(MockLayerChromium::create(&ccLayerDestructionList));
- scoped_refptr<LayerChromium> layerD = layerB->children()[1].get();
+ scoped_refptr<Layer> layerC = layerB->children()[0].get();
+ layerB->addChild(MockLayer::create(&layerImplDestructionList));
+ scoped_refptr<Layer> layerD = layerB->children()[1].get();
- scoped_ptr<CCLayerImpl> ccLayerTreeRoot = TreeSynchronizer::synchronizeTrees(layerTreeRoot.get(), scoped_ptr<CCLayerImpl>(), hostImpl.get());
- expectTreesAreIdentical(layerTreeRoot.get(), ccLayerTreeRoot.get(), hostImpl.get());
+ scoped_ptr<LayerImpl> layerImplTreeRoot = TreeSynchronizer::synchronizeTrees(layerTreeRoot.get(), scoped_ptr<LayerImpl>(), hostImpl.get());
+ expectTreesAreIdentical(layerTreeRoot.get(), layerImplTreeRoot.get(), hostImpl.get());
// Now restructure the tree to look like this:
// root --- D ---+--- A
@@ -294,45 +294,45 @@ TEST(TreeSynchronizerTest, reuseCCLayersAfterStructuralChange)
layerB->removeAllChildren();
layerC->addChild(layerB);
- // After another synchronize our trees should match and we should not have destroyed any CCLayerImpls
- ccLayerTreeRoot = TreeSynchronizer::synchronizeTrees(layerTreeRoot.get(), ccLayerTreeRoot.Pass(), hostImpl.get());
- expectTreesAreIdentical(layerTreeRoot.get(), ccLayerTreeRoot.get(), hostImpl.get());
+ // After another synchronize our trees should match and we should not have destroyed any LayerImpls
+ layerImplTreeRoot = TreeSynchronizer::synchronizeTrees(layerTreeRoot.get(), layerImplTreeRoot.Pass(), hostImpl.get());
+ expectTreesAreIdentical(layerTreeRoot.get(), layerImplTreeRoot.get(), hostImpl.get());
- EXPECT_EQ(0u, ccLayerDestructionList.size());
+ EXPECT_EQ(0u, layerImplDestructionList.size());
}
// Constructs a very simple tree, synchronizes it, then synchronizes to a totally new tree. All layers from the old tree should be deleted.
TEST(TreeSynchronizerTest, syncSimpleTreeThenDestroy)
{
DebugScopedSetImplThread impl;
- Vector<int> ccLayerDestructionList;
+ Vector<int> layerImplDestructionList;
- CCLayerTreeSettings settings;
- scoped_ptr<CCLayerTreeHostImpl> hostImpl = CCLayerTreeHostImpl::create(settings, 0);
+ LayerTreeSettings settings;
+ scoped_ptr<LayerTreeHostImpl> hostImpl = LayerTreeHostImpl::create(settings, 0);
- scoped_refptr<LayerChromium> oldLayerTreeRoot = MockLayerChromium::create(&ccLayerDestructionList);
- oldLayerTreeRoot->addChild(MockLayerChromium::create(&ccLayerDestructionList));
- oldLayerTreeRoot->addChild(MockLayerChromium::create(&ccLayerDestructionList));
+ scoped_refptr<Layer> oldLayerTreeRoot = MockLayer::create(&layerImplDestructionList);
+ oldLayerTreeRoot->addChild(MockLayer::create(&layerImplDestructionList));
+ oldLayerTreeRoot->addChild(MockLayer::create(&layerImplDestructionList));
int oldTreeRootLayerId = oldLayerTreeRoot->id();
int oldTreeFirstChildLayerId = oldLayerTreeRoot->children()[0]->id();
int oldTreeSecondChildLayerId = oldLayerTreeRoot->children()[1]->id();
- scoped_ptr<CCLayerImpl> ccLayerTreeRoot = TreeSynchronizer::synchronizeTrees(oldLayerTreeRoot.get(), scoped_ptr<CCLayerImpl>(), hostImpl.get());
- expectTreesAreIdentical(oldLayerTreeRoot.get(), ccLayerTreeRoot.get(), hostImpl.get());
+ scoped_ptr<LayerImpl> layerImplTreeRoot = TreeSynchronizer::synchronizeTrees(oldLayerTreeRoot.get(), scoped_ptr<LayerImpl>(), hostImpl.get());
+ expectTreesAreIdentical(oldLayerTreeRoot.get(), layerImplTreeRoot.get(), hostImpl.get());
- // Remove all children on the LayerChromium side.
+ // Remove all children on the Layer side.
oldLayerTreeRoot->removeAllChildren();
- // Synchronize again. After the sync all CCLayerImpls from the old tree should be deleted.
- scoped_refptr<LayerChromium> newLayerTreeRoot = LayerChromium::create();
- ccLayerTreeRoot = TreeSynchronizer::synchronizeTrees(newLayerTreeRoot.get(), ccLayerTreeRoot.Pass(), hostImpl.get());
- expectTreesAreIdentical(newLayerTreeRoot.get(), ccLayerTreeRoot.get(), hostImpl.get());
+ // Synchronize again. After the sync all LayerImpls from the old tree should be deleted.
+ scoped_refptr<Layer> newLayerTreeRoot = Layer::create();
+ layerImplTreeRoot = TreeSynchronizer::synchronizeTrees(newLayerTreeRoot.get(), layerImplTreeRoot.Pass(), hostImpl.get());
+ expectTreesAreIdentical(newLayerTreeRoot.get(), layerImplTreeRoot.get(), hostImpl.get());
- ASSERT_EQ(3u, ccLayerDestructionList.size());
- EXPECT_TRUE(ccLayerDestructionList.contains(oldTreeRootLayerId));
- EXPECT_TRUE(ccLayerDestructionList.contains(oldTreeFirstChildLayerId));
- EXPECT_TRUE(ccLayerDestructionList.contains(oldTreeSecondChildLayerId));
+ ASSERT_EQ(3u, layerImplDestructionList.size());
+ EXPECT_TRUE(layerImplDestructionList.contains(oldTreeRootLayerId));
+ EXPECT_TRUE(layerImplDestructionList.contains(oldTreeFirstChildLayerId));
+ EXPECT_TRUE(layerImplDestructionList.contains(oldTreeSecondChildLayerId));
}
// Constructs+syncs a tree with mask, replica, and replica mask layers.
@@ -340,64 +340,64 @@ TEST(TreeSynchronizerTest, syncMaskReplicaAndReplicaMaskLayers)
{
DebugScopedSetImplThread impl;
- CCLayerTreeSettings settings;
- scoped_ptr<CCLayerTreeHostImpl> hostImpl = CCLayerTreeHostImpl::create(settings, 0);
+ LayerTreeSettings settings;
+ scoped_ptr<LayerTreeHostImpl> hostImpl = LayerTreeHostImpl::create(settings, 0);
- scoped_refptr<LayerChromium> layerTreeRoot = LayerChromium::create();
- layerTreeRoot->addChild(LayerChromium::create());
- layerTreeRoot->addChild(LayerChromium::create());
- layerTreeRoot->addChild(LayerChromium::create());
+ scoped_refptr<Layer> layerTreeRoot = Layer::create();
+ layerTreeRoot->addChild(Layer::create());
+ layerTreeRoot->addChild(Layer::create());
+ layerTreeRoot->addChild(Layer::create());
// First child gets a mask layer.
- scoped_refptr<LayerChromium> maskLayer = LayerChromium::create();
+ scoped_refptr<Layer> maskLayer = Layer::create();
layerTreeRoot->children()[0]->setMaskLayer(maskLayer.get());
// Second child gets a replica layer.
- scoped_refptr<LayerChromium> replicaLayer = LayerChromium::create();
+ scoped_refptr<Layer> replicaLayer = Layer::create();
layerTreeRoot->children()[1]->setReplicaLayer(replicaLayer.get());
// Third child gets a replica layer with a mask layer.
- scoped_refptr<LayerChromium> replicaLayerWithMask = LayerChromium::create();
- scoped_refptr<LayerChromium> replicaMaskLayer = LayerChromium::create();
+ scoped_refptr<Layer> replicaLayerWithMask = Layer::create();
+ scoped_refptr<Layer> replicaMaskLayer = Layer::create();
replicaLayerWithMask->setMaskLayer(replicaMaskLayer.get());
layerTreeRoot->children()[2]->setReplicaLayer(replicaLayerWithMask.get());
- scoped_ptr<CCLayerImpl> ccLayerTreeRoot = TreeSynchronizer::synchronizeTrees(layerTreeRoot.get(), scoped_ptr<CCLayerImpl>(), hostImpl.get());
+ scoped_ptr<LayerImpl> layerImplTreeRoot = TreeSynchronizer::synchronizeTrees(layerTreeRoot.get(), scoped_ptr<LayerImpl>(), hostImpl.get());
- expectTreesAreIdentical(layerTreeRoot.get(), ccLayerTreeRoot.get(), hostImpl.get());
+ expectTreesAreIdentical(layerTreeRoot.get(), layerImplTreeRoot.get(), hostImpl.get());
// Remove the mask layer.
layerTreeRoot->children()[0]->setMaskLayer(0);
- ccLayerTreeRoot = TreeSynchronizer::synchronizeTrees(layerTreeRoot.get(), ccLayerTreeRoot.Pass(), hostImpl.get());
- expectTreesAreIdentical(layerTreeRoot.get(), ccLayerTreeRoot.get(), hostImpl.get());
+ layerImplTreeRoot = TreeSynchronizer::synchronizeTrees(layerTreeRoot.get(), layerImplTreeRoot.Pass(), hostImpl.get());
+ expectTreesAreIdentical(layerTreeRoot.get(), layerImplTreeRoot.get(), hostImpl.get());
// Remove the replica layer.
layerTreeRoot->children()[1]->setReplicaLayer(0);
- ccLayerTreeRoot = TreeSynchronizer::synchronizeTrees(layerTreeRoot.get(), ccLayerTreeRoot.Pass(), hostImpl.get());
- expectTreesAreIdentical(layerTreeRoot.get(), ccLayerTreeRoot.get(), hostImpl.get());
+ layerImplTreeRoot = TreeSynchronizer::synchronizeTrees(layerTreeRoot.get(), layerImplTreeRoot.Pass(), hostImpl.get());
+ expectTreesAreIdentical(layerTreeRoot.get(), layerImplTreeRoot.get(), hostImpl.get());
// Remove the replica mask.
replicaLayerWithMask->setMaskLayer(0);
- ccLayerTreeRoot = TreeSynchronizer::synchronizeTrees(layerTreeRoot.get(), ccLayerTreeRoot.Pass(), hostImpl.get());
- expectTreesAreIdentical(layerTreeRoot.get(), ccLayerTreeRoot.get(), hostImpl.get());
+ layerImplTreeRoot = TreeSynchronizer::synchronizeTrees(layerTreeRoot.get(), layerImplTreeRoot.Pass(), hostImpl.get());
+ expectTreesAreIdentical(layerTreeRoot.get(), layerImplTreeRoot.get(), hostImpl.get());
}
TEST(TreeSynchronizerTest, synchronizeAnimations)
{
DebugScopedSetImplThread impl;
- CCLayerTreeSettings settings;
- scoped_ptr<CCLayerTreeHostImpl> hostImpl = CCLayerTreeHostImpl::create(settings, 0);
+ LayerTreeSettings settings;
+ scoped_ptr<LayerTreeHostImpl> hostImpl = LayerTreeHostImpl::create(settings, 0);
- scoped_refptr<LayerChromium> layerTreeRoot = LayerChromium::create();
+ scoped_refptr<Layer> layerTreeRoot = Layer::create();
FakeLayerAnimationControllerClient dummy;
- layerTreeRoot->setLayerAnimationController(FakeLayerAnimationController::create(&dummy).PassAs<CCLayerAnimationController>());
+ layerTreeRoot->setLayerAnimationController(FakeLayerAnimationController::create(&dummy).PassAs<LayerAnimationController>());
EXPECT_FALSE(static_cast<FakeLayerAnimationController*>(layerTreeRoot->layerAnimationController())->synchronizedAnimations());
- scoped_ptr<CCLayerImpl> ccLayerTreeRoot = TreeSynchronizer::synchronizeTrees(layerTreeRoot.get(), scoped_ptr<CCLayerImpl>(), hostImpl.get());
- ccLayerTreeRoot = TreeSynchronizer::synchronizeTrees(layerTreeRoot.get(), ccLayerTreeRoot.Pass(), hostImpl.get());
+ scoped_ptr<LayerImpl> layerImplTreeRoot = TreeSynchronizer::synchronizeTrees(layerTreeRoot.get(), scoped_ptr<LayerImpl>(), hostImpl.get());
+ layerImplTreeRoot = TreeSynchronizer::synchronizeTrees(layerTreeRoot.get(), layerImplTreeRoot.Pass(), hostImpl.get());
EXPECT_TRUE(static_cast<FakeLayerAnimationController*>(layerTreeRoot->layerAnimationController())->synchronizedAnimations());
}