clean up tests

src/canvasdrawer.js

@@ -101,6 +101,24 @@ class CanvasDrawer extends $.DrawerBase{
                 this._drawTiles(tiledImage);
             }
         }
+
+        // // Fire the frame-drawn event.
+        // if( this.viewer ){
+        //     /**
+        //         * Raised when a tiled image is drawn to the canvas. Only valid
+        //         * for webgl and canvas drawers.
+        //         *
+        //         * @event tiled-image-drawn
+        //         * @memberof OpenSeadragon.Viewer
+        //         * @type {object}
+        //         * @property {OpenSeadragon.Viewer} eventSource - A reference to the Viewer which raised the event.
+        //         * @property {HTMLElement} canvas - The canvas which has been drawn onto
+        //         * @property {?Object} userData - Arbitrary subscriber-defined object.
+        //         */
+        //     this.viewer.raiseEvent( 'frame-drawn', {
+        //         canvas: this.canvas
+        //     });
+        // }
     }
 
     /**

src/webgldrawer.js

@@ -380,6 +380,24 @@
                 this._outputContext.drawImage(this._renderingCanvas, 0, 0);
             }
 
+            // // Fire the frame-drawn event.
+            // if( this.viewer ){
+            //     /**
+            //         * Raised when a tiled image is drawn to the canvas. Only valid
+            //         * for webgl and canvas drawers.
+            //         *
+            //         * @event tiled-image-drawn
+            //         * @memberof OpenSeadragon.Viewer
+            //         * @type {object}
+            //         * @property {OpenSeadragon.Viewer} eventSource - A reference to the Viewer which raised the event.
+            //         * @property {HTMLElement} canvas - The canvas which has been drawn onto
+            //         * @property {?Object} userData - Arbitrary subscriber-defined object.
+            //         */
+            //     this.viewer.raiseEvent( 'frame-drawn', {
+            //         canvas: this.canvas
+            //     });
+            // }
+
         }
 
         // Public API required by all Drawer implementations

test/demo/basic.html

@@ -20,16 +20,6 @@
     <div id="contentDiv" class="openseadragon1"></div>
     <script type="text/javascript">
 
-        const url = new URL(window.location.href);
-        let drawer = url.searchParams.get("drawer");
-        if (!drawer) {
-            drawer = 'canvas';
-            url.searchParams.set('drawer', drawer);
-            if ("undefined" !== typeof history.replaceState) {
-                history.replaceState(null, window.location.title, url.toString());
-            }
-        }
-
         var viewer = OpenSeadragon({
             // debugMode: true,
             id: "contentDiv",

test/demo/collections/main.js

@@ -131,7 +131,8 @@
                     var box = new OpenSeadragon.Rect(margins.left, margins.top,
                         $('#contentDiv').width() - (margins.left + margins.right),
                         $('#contentDiv').height() - (margins.top + margins.bottom));
-                    // if drawDebuggingRect is implemented, use it to show the box
+                    // If drawDebuggingRect is implemented, use it to show the box.
+                    // This is not implemented by all drawers however.
                     self.viewer.drawer.drawDebuggingRect(box);
                 });
             }

test/demo/tilesource-swap.html

@@ -80,12 +80,12 @@
                             // The changeover will look better if we wait for the first tile to be drawn.
                             var tileDrawnHandler = function(event) {
                                 if (event.tiledImage === fullImage) {
-                                    viewer.removeHandler('tiled-image-drawn', tileDrawnHandler);
+                                    viewer.removeHandler('update-viewport', tileDrawnHandler);
                                     viewer.world.removeItem(standin);
                                 }
                             };
 
-                            viewer.addHandler('tiled-image-drawn', tileDrawnHandler);
+                            viewer.addHandler('update-viewport', tileDrawnHandler);
                         }
                     });
                 }

test/modules/ajax-tiles.js

@@ -53,10 +53,7 @@
             });
         },
         afterEach: function() {
-            if (viewer && viewer.close) {
+            if (viewer){
-                viewer.close();
-            }
-            if (viewer && viewer.destroy){
                 viewer.destroy();
             }
 

test/modules/basic.js

@@ -16,10 +16,7 @@
             });
         },
         afterEach: function () {
-            if (viewer && viewer.close) {
+            if (viewer){
-                viewer.close();
-            }
-            if (viewer && viewer.destroy){
                 viewer.destroy();
             }
 

test/modules/controls.js

@@ -19,9 +19,6 @@
             if (viewer && viewer.close) {
                 viewer.close();
             }
-            // if (viewer && viewer.destroy){
-            //     viewer.destroy();
-            // }
 
             viewer = null;
         }

test/modules/drawer.js

@@ -10,10 +10,7 @@
             testLog.reset();
         },
         afterEach: function () {
-            if (viewer && viewer.close) {
+            if (viewer){
-                viewer.close();
-            }
-            if (viewer && viewer.destroy){
                 viewer.destroy();
             }
             viewer = null;

test/modules/events.js

@@ -17,10 +17,7 @@
             } );
         },
         afterEach: function () {
-            if ( viewer && viewer.close ) {
+            if (viewer){
-                viewer.close();
-            }
-            if (viewer && viewer.destroy){
                 viewer.destroy();
             }
 

test/modules/formats.js

@@ -14,10 +14,7 @@
             document.getElementById("qunit-fixture").appendChild(example);
         },
         afterEach: function () {
-            if ( viewer && viewer.close ) {
+            if (viewer){
-                viewer.close();
-            }
-            if (viewer && viewer.destroy){
                 viewer.destroy();
             }
 

test/modules/imageloader.js

@@ -15,10 +15,7 @@
             testLog.reset();
         },
         afterEach: function () {
-            if (viewer && viewer.close) {
+            if (viewer){
-                viewer.close();
-            }
-            if (viewer && viewer.destroy){
                 viewer.destroy();
             }
 

test/modules/multi-image.js

@@ -16,10 +16,7 @@
             });
         },
         afterEach: function() {
-            if ( viewer && viewer.close ) {
+            if (viewer){
-                viewer.close();
-            }
-            if (viewer && viewer.destroy){
                 viewer.destroy();
             }
 

test/modules/navigator.js

@@ -42,10 +42,7 @@
 
             resetTestVariables();
 
-            // if ( viewer && viewer.close ) {
+            if (viewer){
-            //     viewer.close();
-            // }
-            if (viewer && viewer.destroy){
                 viewer.destroy();
             }
 

test/modules/overlays.js

@@ -16,10 +16,8 @@
         },
         afterEach: function() {
             resetTestVariables();
-            if ( viewer && viewer.close ) {
+
-                viewer.close();
+            if (viewer){
-            }
-            if (viewer && viewer.destroy){
                 viewer.destroy();
             }
 

test/modules/referencestrip.js

@@ -10,10 +10,7 @@
             testLog.reset();
         },
         afterEach: function () {
-            if (viewer && viewer.close) {
+            if (viewer){
-                viewer.close();
-            }
-            if (viewer && viewer.destroy){
                 viewer.destroy();
             }
 

test/modules/tiledimage.js

@@ -17,10 +17,7 @@
             });
         },
         afterEach: function() {
-            if (viewer && viewer.close) {
+            if (viewer){
-                viewer.close();
-            }
-            if (viewer && viewer.destroy){
                 viewer.destroy();
             }
 

test/modules/tilesource-dynamic-url.js

@@ -130,10 +130,7 @@
         afterEach: function () {
             ASSERT = null;
 
-            if (viewer && viewer.close) {
+            if (viewer){
-                viewer.close();
-            }
-            if (viewer && viewer.destroy){
                 viewer.destroy();
             }
 

test/modules/units.js

@@ -19,10 +19,7 @@
             });
         },
         afterEach: function () {
-            if (viewer && viewer.close) {
+            if (viewer){
-                viewer.close();
-            }
-            if (viewer && viewer.destroy){
                 viewer.destroy();
             }
 

test/modules/viewport.js

@@ -21,10 +21,7 @@
             });
         },
         afterEach: function () {
-            if (viewer && viewer.close) {
+            if (viewer){
-                viewer.close();
-            }
-            if (viewer && viewer.destroy){
                 viewer.destroy();
             }
 

test/modules/world.js

@@ -16,10 +16,7 @@
             });
         },
         afterEach: function () {
-            if (viewer && viewer.close) {
+            if (viewer){
-                viewer.close();
-            }
-            if (viewer && viewer.destroy){
                 viewer.destroy();
             }