| Index: cc/layers/layer_unittest.cc
|
| diff --git a/cc/layers/layer_unittest.cc b/cc/layers/layer_unittest.cc
|
| index 32b4b4c322e84ce13cc25c02441873b4f57b27d5..a6ff2e0a8e7066d65cbf73062cbac5c6517d435f 100644
|
| --- a/cc/layers/layer_unittest.cc
|
| +++ b/cc/layers/layer_unittest.cc
|
| @@ -40,7 +40,7 @@ namespace {
|
| class MockLayerTreeHost : public LayerTreeHost {
|
| public:
|
| explicit MockLayerTreeHost(FakeLayerTreeHostClient* client)
|
| - : LayerTreeHost(client, NULL, LayerTreeSettings()) {
|
| + : LayerTreeHost(client, nullptr, LayerTreeSettings()) {
|
| InitializeSingleThreaded(client, base::MessageLoopProxy::current());
|
| }
|
|
|
| @@ -69,15 +69,15 @@ class LayerTest : public testing::Test {
|
| virtual void TearDown() override {
|
| Mock::VerifyAndClearExpectations(layer_tree_host_.get());
|
| EXPECT_CALL(*layer_tree_host_, SetNeedsFullTreeSync()).Times(AnyNumber());
|
| - parent_ = NULL;
|
| - child1_ = NULL;
|
| - child2_ = NULL;
|
| - child3_ = NULL;
|
| - grand_child1_ = NULL;
|
| - grand_child2_ = NULL;
|
| - grand_child3_ = NULL;
|
| -
|
| - layer_tree_host_->SetRootLayer(NULL);
|
| + parent_ = nullptr;
|
| + child1_ = nullptr;
|
| + child2_ = nullptr;
|
| + child3_ = nullptr;
|
| + grand_child1_ = nullptr;
|
| + grand_child2_ = nullptr;
|
| + grand_child3_ = nullptr;
|
| +
|
| + layer_tree_host_->SetRootLayer(nullptr);
|
| layer_tree_host_ = nullptr;
|
| }
|
|
|
| @@ -151,7 +151,7 @@ TEST_F(LayerTest, BasicCreateAndDestroy) {
|
| Mock::VerifyAndClearExpectations(layer_tree_host_.get());
|
|
|
| EXPECT_CALL(*layer_tree_host_, SetNeedsCommit()).Times(0);
|
| - test_layer->SetLayerTreeHost(NULL);
|
| + test_layer->SetLayerTreeHost(nullptr);
|
| }
|
|
|
| TEST_F(LayerTest, AddAndRemoveChild) {
|
| @@ -234,7 +234,7 @@ TEST_F(LayerTest, InsertChild) {
|
| EXPECT_EQ(child4, parent->children()[3]);
|
| EXPECT_EQ(parent.get(), child4->parent());
|
|
|
| - EXPECT_SET_NEEDS_FULL_TREE_SYNC(1, layer_tree_host_->SetRootLayer(NULL));
|
| + EXPECT_SET_NEEDS_FULL_TREE_SYNC(1, layer_tree_host_->SetRootLayer(nullptr));
|
| }
|
|
|
| TEST_F(LayerTest, InsertChildPastEndOfList) {
|
| @@ -283,7 +283,7 @@ TEST_F(LayerTest, InsertSameChildTwice) {
|
| EXPECT_EQ(child2, parent->children()[0]);
|
| EXPECT_EQ(child1, parent->children()[1]);
|
|
|
| - EXPECT_SET_NEEDS_FULL_TREE_SYNC(1, layer_tree_host_->SetRootLayer(NULL));
|
| + EXPECT_SET_NEEDS_FULL_TREE_SYNC(1, layer_tree_host_->SetRootLayer(nullptr));
|
| }
|
|
|
| TEST_F(LayerTest, ReplaceChildWithNewChild) {
|
| @@ -357,11 +357,11 @@ TEST_F(LayerTest, DeleteRemovedScrollParent) {
|
|
|
| child1->reset_needs_push_properties_for_testing();
|
|
|
| - EXPECT_SET_NEEDS_COMMIT(1, child2 = NULL);
|
| + EXPECT_SET_NEEDS_COMMIT(1, child2 = nullptr);
|
|
|
| EXPECT_TRUE(child1->needs_push_properties());
|
|
|
| - EXPECT_SET_NEEDS_FULL_TREE_SYNC(1, layer_tree_host_->SetRootLayer(NULL));
|
| + EXPECT_SET_NEEDS_FULL_TREE_SYNC(1, layer_tree_host_->SetRootLayer(nullptr));
|
| }
|
|
|
| TEST_F(LayerTest, DeleteRemovedScrollChild) {
|
| @@ -386,11 +386,11 @@ TEST_F(LayerTest, DeleteRemovedScrollChild) {
|
|
|
| child2->reset_needs_push_properties_for_testing();
|
|
|
| - EXPECT_SET_NEEDS_COMMIT(1, child1 = NULL);
|
| + EXPECT_SET_NEEDS_COMMIT(1, child1 = nullptr);
|
|
|
| EXPECT_TRUE(child2->needs_push_properties());
|
|
|
| - EXPECT_SET_NEEDS_FULL_TREE_SYNC(1, layer_tree_host_->SetRootLayer(NULL));
|
| + EXPECT_SET_NEEDS_FULL_TREE_SYNC(1, layer_tree_host_->SetRootLayer(nullptr));
|
| }
|
|
|
| TEST_F(LayerTest, ReplaceChildWithSameChild) {
|
| @@ -444,7 +444,7 @@ TEST_F(LayerTest, SetChildren) {
|
| EXPECT_EQ(new_parent.get(), child1->parent());
|
| EXPECT_EQ(new_parent.get(), child2->parent());
|
|
|
| - EXPECT_SET_NEEDS_FULL_TREE_SYNC(1, layer_tree_host_->SetRootLayer(NULL));
|
| + EXPECT_SET_NEEDS_FULL_TREE_SYNC(1, layer_tree_host_->SetRootLayer(nullptr));
|
| }
|
|
|
| TEST_F(LayerTest, HasAncestor) {
|
| @@ -845,15 +845,15 @@ TEST_F(LayerTest, MaskAndReplicaHasParent) {
|
| EXPECT_EQ(replica.get(), replica_mask->parent());
|
|
|
| replica->SetMaskLayer(replica_mask_replacement.get());
|
| - EXPECT_EQ(NULL, replica_mask->parent());
|
| + EXPECT_EQ(nullptr, replica_mask->parent());
|
| EXPECT_EQ(replica.get(), replica_mask_replacement->parent());
|
|
|
| child->SetMaskLayer(mask_replacement.get());
|
| - EXPECT_EQ(NULL, mask->parent());
|
| + EXPECT_EQ(nullptr, mask->parent());
|
| EXPECT_EQ(child.get(), mask_replacement->parent());
|
|
|
| child->SetReplicaLayer(replica_replacement.get());
|
| - EXPECT_EQ(NULL, replica->parent());
|
| + EXPECT_EQ(nullptr, replica->parent());
|
| EXPECT_EQ(child.get(), replica_replacement->parent());
|
|
|
| EXPECT_EQ(replica.get(), replica->mask_layer()->parent());
|
| @@ -978,7 +978,7 @@ TEST(LayerLayerTreeHostTest, EnteringTree) {
|
| child->SetReplicaLayer(replica.get());
|
| replica->SetMaskLayer(replica_mask.get());
|
|
|
| - AssertLayerTreeHostMatchesForSubtree(parent.get(), NULL);
|
| + AssertLayerTreeHostMatchesForSubtree(parent.get(), nullptr);
|
|
|
| LayerTreeHostFactory factory;
|
| scoped_ptr<LayerTreeHost> layer_tree_host = factory.Create();
|
| @@ -990,9 +990,9 @@ TEST(LayerLayerTreeHostTest, EnteringTree) {
|
|
|
| // Clearing the root layer should also clear out the host pointers for all
|
| // layers in the tree.
|
| - layer_tree_host->SetRootLayer(NULL);
|
| + layer_tree_host->SetRootLayer(nullptr);
|
|
|
| - AssertLayerTreeHostMatchesForSubtree(parent.get(), NULL);
|
| + AssertLayerTreeHostMatchesForSubtree(parent.get(), nullptr);
|
| }
|
|
|
| TEST(LayerLayerTreeHostTest, AddingLayerSubtree) {
|
| @@ -1021,7 +1021,7 @@ TEST(LayerLayerTreeHostTest, AddingLayerSubtree) {
|
| parent->AddChild(child);
|
| AssertLayerTreeHostMatchesForSubtree(parent.get(), layer_tree_host.get());
|
|
|
| - layer_tree_host->SetRootLayer(NULL);
|
| + layer_tree_host->SetRootLayer(nullptr);
|
| }
|
|
|
| TEST(LayerLayerTreeHostTest, ChangeHost) {
|
| @@ -1052,7 +1052,7 @@ TEST(LayerLayerTreeHostTest, ChangeHost) {
|
| AssertLayerTreeHostMatchesForSubtree(parent.get(),
|
| second_layer_tree_host.get());
|
|
|
| - second_layer_tree_host->SetRootLayer(NULL);
|
| + second_layer_tree_host->SetRootLayer(nullptr);
|
| }
|
|
|
| TEST(LayerLayerTreeHostTest, ChangeHostInSubtree) {
|
| @@ -1087,8 +1087,8 @@ TEST(LayerLayerTreeHostTest, ChangeHostInSubtree) {
|
| second_grand_child->layer_tree_host());
|
|
|
| // Test over, cleanup time.
|
| - first_layer_tree_host->SetRootLayer(NULL);
|
| - second_layer_tree_host->SetRootLayer(NULL);
|
| + first_layer_tree_host->SetRootLayer(nullptr);
|
| + second_layer_tree_host->SetRootLayer(nullptr);
|
| }
|
|
|
| TEST(LayerLayerTreeHostTest, ReplaceMaskAndReplicaLayer) {
|
| @@ -1113,16 +1113,16 @@ TEST(LayerLayerTreeHostTest, ReplaceMaskAndReplicaLayer) {
|
|
|
| // Replacing the mask should clear out the old mask's subtree's host pointers.
|
| parent->SetMaskLayer(mask_replacement.get());
|
| - EXPECT_EQ(NULL, mask->layer_tree_host());
|
| - EXPECT_EQ(NULL, mask_child->layer_tree_host());
|
| + EXPECT_EQ(nullptr, mask->layer_tree_host());
|
| + EXPECT_EQ(nullptr, mask_child->layer_tree_host());
|
|
|
| // Same for replacing a replica layer.
|
| parent->SetReplicaLayer(replica_replacement.get());
|
| - EXPECT_EQ(NULL, replica->layer_tree_host());
|
| - EXPECT_EQ(NULL, replica_child->layer_tree_host());
|
| + EXPECT_EQ(nullptr, replica->layer_tree_host());
|
| + EXPECT_EQ(nullptr, replica_child->layer_tree_host());
|
|
|
| // Test over, cleanup time.
|
| - layer_tree_host->SetRootLayer(NULL);
|
| + layer_tree_host->SetRootLayer(nullptr);
|
| }
|
|
|
| TEST(LayerLayerTreeHostTest, DestroyHostWithNonNullRootLayer) {
|
|
|