cc: Remove DrawQuad::IterateResoruces

This patch removes IterateResources function in favor of
iterating resources directly on the quad. In order to
accomplish this, each derived quad uses new resources
object on the base class to store all of the resources
it needs. This allows the calling code that used to call
IterateResources with a callback to instead directly
iterate all of the ids and manipulate them in any way
that is required.

This improves the performance of the IterateResources
test by ~30%.

BUG=492765
R=danakj, piman@chromium.org
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Review URL: https://codereview.chromium.org/1152473006

Cr-Commit-Position: refs/heads/master@{#332476}

1 files changed, 4 insertions, 3 deletions

diff --git a/cc/quads/stream_video_draw_quad.h b/cc/quads/stream_video_draw_quad.h
index 45c28f2..faffd20 100644
--- a/cc/quads/stream_video_draw_quad.h
+++ b/cc/quads/stream_video_draw_quad.h
@@ -31,14 +31,15 @@ class CC_EXPORT StreamVideoDrawQuad : public DrawQuad {
               unsigned resource_id,
               const gfx::Transform& matrix);
 
-  unsigned resource_id;
   gfx::Transform matrix;
 
-  void IterateResources(const ResourceIteratorCallback& callback) override;
-
   static const StreamVideoDrawQuad* MaterialCast(const DrawQuad*);
 
+  ResourceId resource_id() const { return resources.ids[kResourceIdIndex]; }
+
  private:
+  static const size_t kResourceIdIndex = 0;
+
   void ExtendValue(base::trace_event::TracedValue* value) const override;
 };