cc: Remove DrawQuad::IterateResoruces

cc/quads/draw_quad.h

 // 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.
 
 #ifndef CC_QUADS_DRAW_QUAD_H_
 #define CC_QUADS_DRAW_QUAD_H_
 
 #include "base/callback.h"
 #include "cc/base/cc_export.h"
 #include "cc/base/resource_id.h"
@@ skipping 78 matching lines @@
   bool IsDebugQuad() const { return material == DEBUG_BORDER; }
 
   bool ShouldDrawWithBlending() const {
     if (needs_blending || shared_quad_state->opacity < 1.0f)
       return true;
     if (visible_rect.IsEmpty())
       return false;
     return !opaque_rect.Contains(visible_rect);
   }
 
-  typedef base::Callback<ResourceId(ResourceId)> ResourceIteratorCallback;
-  virtual void IterateResources(const ResourceIteratorCallback& callback) = 0;
-
   // Is the left edge of this tile aligned with the originating layer's
   // left edge?
   bool IsLeftEdge() const { return !rect.x(); }
 
   // Is the top edge of this tile aligned with the originating layer's
   // top edge?
   bool IsTopEdge() const { return !rect.y(); }
 
   // Is the right edge of this tile aligned with the originating layer's
   // right edge?
   bool IsRightEdge() const {
     return rect.right() == shared_quad_state->content_bounds.width();
   }
 
   // Is the bottom edge of this tile aligned with the originating layer's
   // bottom edge?
   bool IsBottomEdge() const {
     return rect.bottom() == shared_quad_state->content_bounds.height();
   }
 
   // Is any edge of this tile aligned with the originating layer's
   // corresponding edge?
   bool IsEdge() const {
     return IsLeftEdge() || IsTopEdge() || IsRightEdge() || IsBottomEdge();
   }
 
   void AsValueInto(base::trace_event::TracedValue* value) const;
 
+  struct CC_EXPORT Resources {
+    enum : size_t { kMaxResourceIdCount = 4 };

[Nico, 2015/07/23 18:04:41]

(drive-by since I just saw this: explicit types on unnamed enums don't really
make sense. You're just using this line to tell the compiler "yo,
kMaxResourceIdCount is 4", the size_t bit doesn't add anything)

(also, all other fields in this class are at the beginning of this class; it's a
bit weird that resources is tacked on at the end)

+    Resources();
+
+    ResourceId* begin() { return ids; }
+    ResourceId* end() {
+      DCHECK_LE(count, kMaxResourceIdCount);
+      return ids + count;
+    }
+
+    size_t count;
+    ResourceId ids[kMaxResourceIdCount];
+  };
+
+  Resources resources;
+
  protected:
   DrawQuad();
 
   void SetAll(const SharedQuadState* shared_quad_state,
               Material material,
               const gfx::Rect& rect,
               const gfx::Rect& opaque_rect,
               const gfx::Rect& visible_rect,
               bool needs_blending);
   virtual void ExtendValue(base::trace_event::TracedValue* value) const = 0;
 };
 
 }  // namespace cc
 
 #endif  // CC_QUADS_DRAW_QUAD_H_