Removing the assumption that child->SetParent will not change Drawability

cc/layers/ui_resource_layer_unittest.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "cc/layers/ui_resource_layer.h"
 
 #include "cc/resources/prioritized_resource_manager.h"
 #include "cc/resources/resource_provider.h"
 #include "cc/resources/resource_update_queue.h"
 #include "cc/resources/scoped_ui_resource.h"
@@ skipping 84 matching lines @@
 
   bool is_opaque = false;
   scoped_ptr<ScopedUIResource> resource = ScopedUIResource::Create(
       layer_tree_host_.get(), UIResourceBitmap(gfx::Size(10, 10), is_opaque));
   test_layer->SetUIResourceId(resource->id());
   test_layer->Update(&queue, &occlusion_tracker);
 
   EXPECT_TRUE(test_layer->DrawsContent());
 }
 
+TEST_F(UIResourceLayerTest, SetUIResourceIdOrdering) {
+  scoped_refptr<Layer> test_root = Layer::Create();
+  layer_tree_host_->SetRootLayer(test_root);
+
+  scoped_refptr<UIResourceLayer> test_layer = UIResourceLayer::Create();
+  ASSERT_TRUE(test_layer.get());
+  test_layer->SetIsDrawable(true);
+  test_layer->SetBounds(gfx::Size(100, 100));
+
+  bool is_opaque = false;
+  scoped_ptr<ScopedUIResource> resource = ScopedUIResource::Create(
+      layer_tree_host_.get(), UIResourceBitmap(gfx::Size(10, 10), is_opaque));
+  test_layer->SetUIResourceId(resource->id());
+
+  EXPECT_TRUE(test_layer->DrawsContent());
+  test_root->AddChild(test_layer);
+  EXPECT_FALSE(test_layer->DrawsContent());

[danakj, 2014/08/14 22:00:14]

Can you do this test with a test-only override of cc::Layer that changes its
drawability in SetLTH similarly to UIResourceLayer (or add this capability to
cc::FakeLayer), and move to layer_unittest? And can you verify the number of
descendants that draw content on the parent as well?

+}
+
 }  // namespace
 }  // namespace cc