Add user preference for rendering cache size

(bzr r10347.1.34)

4 files changed, 53 insertions, 19 deletions

diff --git a/src/display/canvas-arena.cpp b/src/display/canvas-arena.cpp
index b254a55c8..ac2704895 100644
--- a/src/display/canvas-arena.cpp
+++ b/src/display/canvas-arena.cpp
@@ -21,6 +21,7 @@
 #include "display/drawing-item.h"
 #include "display/drawing-group.h"
 #include "display/drawing-surface.h"
+#include "preferences.h"
 
 using namespace Inkscape;
 
@@ -48,6 +49,22 @@ static void sp_canvas_arena_request_render (SPCanvasArena *ca, Geom::IntRect con
 static SPCanvasItemClass *parent_class;
 static guint signals[LAST_SIGNAL] = {0};
 
+struct CacheBudgetObserver : public Inkscape::Preferences::Observer {
+    CacheBudgetObserver(SPCanvasArena *arena)
+        : Inkscape::Preferences::Observer("/options/renderingcache/size")
+        , _arena(arena)
+    {
+        Inkscape::Preferences *prefs = Inkscape::Preferences::get();
+        Inkscape::Preferences::Entry v = prefs->getEntry(observed_path);
+        notify(v);
+        prefs->addObserver(*this);
+    }
+    void notify(Preferences::Entry const &v) {
+        _arena->drawing.setCacheBudget((1 << 20) * v.getIntLimited(128, 0, 4096));
+    }
+    SPCanvasArena *_arena;
+};
+
 GType
 sp_canvas_arena_get_type (void)
 {
@@ -102,6 +119,7 @@ sp_canvas_arena_init (SPCanvasArena *arena)
     arena->sticky = FALSE;
 
     new (&arena->drawing) Inkscape::Drawing(arena);
+    arena->observer = new CacheBudgetObserver(arena);
 
     Inkscape::DrawingGroup *root = new DrawingGroup(arena->drawing);
     root->setPickChildren(true);
@@ -129,6 +147,7 @@ sp_canvas_arena_destroy (GtkObject *object)
 {
     SPCanvasArena *arena = SP_CANVAS_ARENA (object);
 
+    delete arena->observer;
     arena->drawing.~Drawing();
 
     if (GTK_OBJECT_CLASS (parent_class)->destroy)
diff --git a/src/display/canvas-arena.h b/src/display/canvas-arena.h
index 6c65bb0e5..463dc1bc3 100644
--- a/src/display/canvas-arena.h
+++ b/src/display/canvas-arena.h
@@ -31,6 +31,7 @@ G_BEGIN_DECLS
 
 typedef struct _SPCanvasArena      SPCanvasArena;
 typedef struct _SPCanvasArenaClass SPCanvasArenaClass;
+struct CacheBudgetObserver;
 
 struct _SPCanvasArena {
     SPCanvasItem item;
@@ -45,6 +46,7 @@ struct _SPCanvasArena {
     Inkscape::DrawingItem *active;
     /* fixme: */
     Inkscape::DrawingItem *picked;
+    CacheBudgetObserver *observer;
     double delta;
 };
 
diff --git a/src/display/drawing.cpp b/src/display/drawing.cpp
index e1a17edf1..06183fed2 100644
--- a/src/display/drawing.cpp
+++ b/src/display/drawing.cpp
@@ -26,7 +26,7 @@ Drawing::Drawing(SPCanvasArena *arena)
     , _blur_quality(BLUR_QUALITY_BEST)
     , _filter_quality(Filters::FILTER_QUALITY_BEST)
     , _cache_score_threshold(50000.0)
-    , _cache_budget(128 << 20) // 128 MiB
+    , _cache_budget(0)
     , _canvasarena(arena)
 {
 
@@ -128,6 +128,12 @@ Drawing::setCacheLimit(Geom::OptIntRect const &r)
         (*i)->_markForUpdate(DrawingItem::STATE_CACHE, false);
     }
 }
+void
+Drawing::setCacheBudget(size_t bytes)
+{
+    _cache_budget = bytes;
+    _pickItemsForCaching();
+}
 
 void
 Drawing::update(Geom::IntRect const &area, UpdateContext const &ctx, unsigned flags, unsigned reset)
@@ -136,6 +142,29 @@ Drawing::update(Geom::IntRect const &area, UpdateContext const &ctx, unsigned fl
         _root->update(area, ctx, flags, reset);
     }
     // process the updated cache scores
+    _pickItemsForCaching();
+}
+
+void
+Drawing::render(DrawingContext &ct, Geom::IntRect const &area, unsigned flags)
+{
+    if (_root) {
+        _root->render(ct, area, flags);
+    }
+}
+
+DrawingItem *
+Drawing::pick(Geom::Point const &p, double delta, unsigned flags)
+{
+    if (_root) {
+        return _root->pick(p, delta, flags);
+    }
+    return NULL;
+}
+
+void
+Drawing::_pickItemsForCaching()
+{
     // we cache the objects with the highest score until the budget is exhausted
     _candidate_items.sort(std::greater<CacheRecord>());
     size_t used = 0;
@@ -162,23 +191,6 @@ Drawing::update(Geom::IntRect const &area, UpdateContext const &ctx, unsigned fl
     }
 }
 
-void
-Drawing::render(DrawingContext &ct, Geom::IntRect const &area, unsigned flags)
-{
-    if (_root) {
-        _root->render(ct, area, flags);
-    }
-}
-
-DrawingItem *
-Drawing::pick(Geom::Point const &p, double delta, unsigned flags)
-{
-    if (_root) {
-        return _root->pick(p, delta, flags);
-    }
-    return NULL;
-}
-
 } // end namespace Inkscape
 
 /*
diff --git a/src/display/drawing.h b/src/display/drawing.h
index 011bf35a6..cfba4ebe6 100644
--- a/src/display/drawing.h
+++ b/src/display/drawing.h
@@ -55,6 +55,7 @@ public:
 
     Geom::OptIntRect const &cacheLimit() const;
     void setCacheLimit(Geom::OptIntRect const &r);
+    void setCacheBudget(size_t bytes);
 
     OutlineColors const &colors() const { return _colors; }
 
@@ -67,7 +68,7 @@ public:
     sigc::signal<void, DrawingItem *> signal_item_deleted;
 
 private:
-    void _reportCacheScore(CacheRecord const &);
+    void _pickItemsForCaching();
 
     typedef std::list<CacheRecord> CandidateList;