cc: Remove DrawQuad::IterateResoruces

cc/layers/tiled_layer_impl_unittest.cc

 // Copyright 2012 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/tiled_layer_impl.h"
 
 #include "cc/layers/append_quads_data.h"
 #include "cc/quads/tile_draw_quad.h"
 #include "cc/test/fake_impl_proxy.h"
 #include "cc/test/fake_layer_tree_host_impl.h"
@@ skipping 251 matching lines @@
            tile_size,
            layer_size,
            LayerTilingData::NO_BORDER_TEXELS,
            gfx::Rect(layer_size));
 
   for (auto iter = render_pass->quad_list.cbegin();
        iter != render_pass->quad_list.cend();
        ++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
                                              << iter.index();
   }
 }
 
 TEST_F(TiledLayerImplTest, GPUMemoryUsage) {
   gfx::Size tile_size(20, 30);
   int num_tiles_x = 12;
@@ skipping 94 matching lines @@
     LayerTestCommon::VerifyQuadsAreOccluded(
         impl.quad_list(), occluded, &partially_occluded_count);
     // The layer outputs one quad, which is partially occluded.
     EXPECT_EQ(100u - 10u, impl.quad_list().size());
     EXPECT_EQ(10u + 10u, partially_occluded_count);
   }
 }
 
 }  // namespace
 }  // namespace cc