cc: Remove DrawQuad::IterateResoruces

cc/layers/tiled_layer_impl_unittest.cc

Index: cc/layers/tiled_layer_impl_unittest.cc
diff --git a/cc/layers/tiled_layer_impl_unittest.cc b/cc/layers/tiled_layer_impl_unittest.cc
index 79ecfa3e31bd3903e2d99a05f075b0ae7e0209f9..7ef93f66e515b9ab12c5cd1b2c0fa98588c70c4d 100644
--- a/cc/layers/tiled_layer_impl_unittest.cc
+++ b/cc/layers/tiled_layer_impl_unittest.cc
@@ -269,8 +269,8 @@ TEST_F(TiledLayerImplTest, TextureInfoForLayerNoBorders) {
        ++iter) {
     const TileDrawQuad* quad = TileDrawQuad::MaterialCast(*iter);
 
-    EXPECT_NE(0u, quad->resource_id) << LayerTestCommon::quad_string
-                                     << iter.index();
+    EXPECT_NE(0u, quad->resource_id()) << LayerTestCommon::quad_string

[danakj, 2015/05/28 17:10:05]

You could check resources.count == 1 and resources[0] != 0 ? Then you don't need
to add things to the subclasses except index constants for quads with > 1
resource?

[vmpstr, 2015/05/28 18:38:45]

I kind of prefer having this type of accessor... I feel like it's more burden on
the caller(s) to do the count/resources[0] check.. since it's really up to the
quad where the resource is stored. Wdyt?

[danakj, 2015/05/28 19:02:29]

What if each DrawQuad subclass had a public kNumResources?

[vmpstr, 2015/05/28 19:18:50]

It's not really a constant (it's determined when SetAll/SetNew is called
sometimes).. I mean that's what resources.count is meant to represent.

+                                       << iter.index();
     EXPECT_EQ(gfx::RectF(gfx::PointF(), tile_size), quad->tex_coord_rect)
         << LayerTestCommon::quad_string << iter.index();
     EXPECT_EQ(tile_size, quad->texture_size) << LayerTestCommon::quad_string