Improved docs

src/imageLoader.js

@@ -34,16 +34,7 @@
 
 (function( $ ){
 
-/**
- * @private
- * @class ImageJob
- * @classdesc Handles loading a single image for use in a single {@link OpenSeadragon.Tile}.
- *
- * @memberof OpenSeadragon
- * @param {String} source - URL of image to download.
- * @param {String} crossOriginPolicy - CORS policy to use for downloads
- * @param {Function} callback - Called once image has finished downloading.
- */
+// private class
 function ImageJob ( options ) {
 
     $.extend( true, this, {
@@ -60,11 +51,6 @@ function ImageJob ( options ) {
 }
 
 ImageJob.prototype = {
-
-    /**
-     * Initiates downloading of associated image.
-     * @method
-     */
     start: function(){
         var _this = this;
 
@@ -104,8 +90,10 @@ ImageJob.prototype = {
 };
 
 /**
- * @class
+ * @class ImageLoader
+ * @memberof OpenSeadragon
  * @classdesc Handles downloading of a set of images using asynchronous queue pattern.
+ * You generally won't have to interact with the ImageLoader directly.
  */
 $.ImageLoader = function() {
 
@@ -117,7 +105,7 @@ $.ImageLoader = function() {
 
 };
 
-$.ImageLoader.prototype = {
+$.ImageLoader.prototype = /** @lends OpenSeadragon.ImageLoader.prototype */{
 
     /**
      * Add an unloaded image to the loader queue.

src/tilecache.js

@@ -34,7 +34,7 @@
 
 (function( $ ){
 
-// private
+// private class
 var TileRecord = function( options ) {
     $.console.assert( options, "[TileCache.cacheTile] options is required" );
     $.console.assert( options.tile, "[TileCache.cacheTile] options.tile is required" );
@@ -43,7 +43,7 @@ var TileRecord = function( options ) {
     this.tiledImage = options.tiledImage;
 };
 
-// private
+// private class
 var ImageRecord = function(options) {
     $.console.assert( options, "[ImageRecord] options is required" );
     $.console.assert( options.image, "[ImageRecord] options.image is required" );
@@ -71,6 +71,7 @@ ImageRecord.prototype = {
     },
 
     addTile: function(tile) {
+        $.console.assert(tile, '[ImageRecord.addTile] tile is required');
         this._tiles.push(tile);
     },
 
@@ -94,6 +95,7 @@ ImageRecord.prototype = {
  * @class TileCache
  * @memberof OpenSeadragon
  * @classdesc Stores all the tiles displayed in a {@link OpenSeadragon.Viewer}.
+ * You generally won't have to interact with the TileCache directly.
  * @param {Object} options - Configuration for this TileCache.
  * @param {Number} [options.maxImageCacheCount] - See maxImageCacheCount in
  * {@link OpenSeadragon.Options} for details.
@@ -196,8 +198,10 @@ $.TileCache.prototype = /** @lends OpenSeadragon.TileCache.prototype */{
 
     /**
      * Clears all tiles associated with the specified tiledImage.
+     * @param {OpenSeadragon.TiledImage} tiledImage
      */
     clearTilesFor: function( tiledImage ) {
+        $.console.assert(tiledImage, '[TileCache.clearTilesFor] tiledImage is required');
         var tileRecord;
         for ( var i = 0; i < this._tilesLoaded.length; ++i ) {
             tileRecord = this._tilesLoaded[ i ];
@@ -209,12 +213,15 @@ $.TileCache.prototype = /** @lends OpenSeadragon.TileCache.prototype */{
         }
     },
 
+    // private
     getImageRecord: function(url) {
+        $.console.assert(url, '[TileCache.getImageRecord] url is required');
         return this._imagesLoaded[url];
     },
 
     // private
     _unloadTile: function(tile) {
+        $.console.assert(tile, '[TileCache._unloadTile] tile is required');
         tile.unload();
         tile.cacheImageRecord = null;
 

src/tiledimage.js

@@ -35,6 +35,8 @@
 (function( $ ){
 
 /**
+ * You shouldn't have to create a TiledImage directly; use {@link OpenSeadragon.Viewer#open}
+ * or {@link OpenSeadragon.Viewer#addTiledImage} instead.
  * @class TiledImage
  * @memberof OpenSeadragon
  * @classdesc Handles rendering of tiles for an {@link OpenSeadragon.Viewer}.