Fix memory leak while destroying the viewer.

This commit is contained in:
Henri Astre 2014-06-17 17:26:10 -07:00
parent e4800a9455
commit 764fa3e34a
4 changed files with 98 additions and 2 deletions

View File

@ -302,7 +302,22 @@ $.Drawer.prototype = /** @lends OpenSeadragon.Drawer.prototype */{
*/
canRotate: function() {
return this.useCanvas;
}
},
/**
* Destroy the drawer (unload current loaded tiles)
*/
destroy: function() {
//unload current loaded tiles (=empty TILE_CACHE)
for (var i=0; i<this.tilesLoaded.length; ++i) {
this.tilesLoaded[i].unload();
}
//force unloading of current canvas (1x1 will be gc later, trick not necessary needed)
this.canvas.width = 1;
this.canvas.height = 1;
}
};
/**

View File

@ -252,6 +252,9 @@
destroy: function () {
stopTracking( this );
this.element = null;
THIS[ this.hash ] = null;
delete THIS[ this.hash ];
},
/**

View File

@ -542,6 +542,10 @@ $.extend( $.Viewer.prototype, $.EventSource.prototype, $.ControlDock.prototype,
this.drawersContainer.innerHTML = "";
this.overlaysContainer.innerHTML = "";
if (this.drawer) {
this.drawer.destroy();
}
this.source = null;
this.drawer = null;
this.drawers = [];
@ -596,7 +600,10 @@ $.extend( $.Viewer.prototype, $.EventSource.prototype, $.ControlDock.prototype,
if (this.outerTracker){
this.outerTracker.destroy();
}
THIS[ this.hash ] = null;
delete THIS[ this.hash ];
// clear all our references to dom objects
this.canvas = null;
this.keyboardCommandArea = null;

View File

@ -0,0 +1,71 @@
<!DOCTYPE html>
<html>
<head>
<title>OpenSeadragon Memory Check Demo</title>
<script type="text/javascript" src='../../build/openseadragon/openseadragon.js'></script>
<style type="text/css">
.openseadragon1 {
width: 800px;
height: 600px;
}
</style>
</head>
<body>
<div>
Simple demo page to monitor OpenSeadragon Memory Usage.
</div>
<button onclick="createViewer()">Create</button>
<button onclick="destroyViewer()">Destroy</button>
<div id="contentDiv" class="openseadragon1"></div>
<script type="text/javascript">
var _viewer;
var generateUniqueHash = (function() {
var counter = 0;
return function() {
return "seadragon_"+(counter++);
};
})();
function createViewer() {
if (_viewer) {
destroyViewer();
}
_viewer = OpenSeadragon({
element: document.getElementById("contentDiv"),
showNavigationControl: false,
springStiffness: 20,
animationTime : 1.5,
visibilityRatio: 1,
hash: "hello",
tileSources: {
Image: {
xmlns: "http://schemas.microsoft.com/deepzoom/2008",
Url: 'http://cdn.photosynth.net/ps2/92fe5de9-dffa-4315-9095-082da8b90642/packet/undistorted/img0014/',
Format: "jpg",
Overlap: 1,
TileSize: 510,
Size: {
Width: 4592,
Height: 3448
}
}
}
});
}
function destroyViewer() {
if (_viewer) {
_viewer.destroy();
}
_viewer = null;
}
</script>
</body>
</html>