Refactor ResourceLoadPriorityOptimizer to avoid walking render tree

Source/core/fetch/ResourceLoadPriorityOptimizer.h

Index: Source/core/fetch/ResourceLoadPriorityOptimizer.h
diff --git a/Source/core/fetch/ResourceLoadPriorityOptimizer.h b/Source/core/fetch/ResourceLoadPriorityOptimizer.h
index cb58320a410b54afd1fba7bbe4957f039092f55c..0fcd0ef00e75d51b928828d04752df98acd9eb03 100644
--- a/Source/core/fetch/ResourceLoadPriorityOptimizer.h
+++ b/Source/core/fetch/ResourceLoadPriorityOptimizer.h
@@ -36,6 +36,7 @@
 #include "platform/geometry/LayoutRect.h"
 
 #include "wtf/HashMap.h"
+#include "wtf/HashSet.h"
 #include "wtf/OwnPtr.h"
 
 namespace WebCore {
@@ -46,11 +47,15 @@ public:
         NotVisible,
         Visible,
     };
-    ResourceLoadPriorityOptimizer();
-    ~ResourceLoadPriorityOptimizer();
     void notifyImageResourceVisibility(ImageResource*, VisibilityStatus);
+    void updateAllImageResourcePriorities();
+    void addRenderObject(RenderObject*);
+    void removeRenderObject(RenderObject*);
 
 private:
+    ResourceLoadPriorityOptimizer();
+    ~ResourceLoadPriorityOptimizer();
+
     void updateImageResourcesWithLoadPriority();
 
     struct ResourceAndVisibility {
@@ -62,8 +67,15 @@ private:
 
     typedef HashMap<unsigned long, OwnPtr<ResourceAndVisibility> > ImageResourceMap;
     ImageResourceMap m_imageResources;
+
+    typedef HashSet<RenderObject*> RenderObjectSet;
+    RenderObjectSet m_objectCache;
+
+    friend ResourceLoadPriorityOptimizer* resourceLoadPriorityOptimizer();

[esprehn, 2014/01/15 01:27:45]

Can this just be a static thing instead? Why does it need to be a friend?

[shatch, 2014/01/15 19:28:52]

Done.

 };
 
+ResourceLoadPriorityOptimizer* resourceLoadPriorityOptimizer();
+
 }
 
 #endif